Add prow jobs for new Kubeflow repositories: #6779
Conversation
jlewi
requested review from
BenTheElder,
cjwagner,
0xmichalis,
krzyzacy and
michelle192837
as code owners
k8s-ci-robot
added
cncf-cla: yes
jlewi
force-pushed
the
kubeflow_repos
branch
from
c791f30 to
85fd6c6
Compare
| preset-service-account: true | ||
| spec: | ||
| containers: | ||
| - image: gcr.io/kubeflow-ci/test-worker:latest |
There was a problem hiding this comment.
can you link to the image Dockerfile?
There was a problem hiding this comment.
The Dockerfile is here
https://github.com/kubeflow/testing/blob/master/images/Dockerfile
What do you mean by link to the Dockerfile? Do you mean add it in a comment?
There was a problem hiding this comment.
nope, was curious how it works with git checkout, and looking if that's possible to integrate with our checkout procedures
There was a problem hiding this comment.
How do your git checkout procedures work?
kubeflow/examples kubeflow/testing kubeflow/tf-operator kubeflow/reporting Remove references to tensorflow/k8s repository.
jlewi
force-pushed
the
kubeflow_repos
branch
from
f1e6b82 to
b03be4e
Compare
|
@krzyzacy I removed references to tensorflow/k8s because that repo has moved to kubeflow/tf-operator. I rebased. PTAL. |
|
@jlewi we are currently using https://github.com/kubernetes/test-infra/blob/master/jenkins/bootstrap.py, and will move to an init container later this should be fine, assume you've configured webhooks already? If so feel free to |
k8s-ci-robot
added
do-not-merge/hold
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jlewi, krzyzacy The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these OWNERS Files:
Approvers can indicate their approval by writing |
k8s-ci-robot
added
the
approved
|
@krzyzacy yes we have webhooks enabled at the org level. |
|
/hold cancel |
k8s-ci-robot
removed
the
do-not-merge/hold
|
@jlewi: Updated the following 2 configmaps:
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Define pre and postsubmit jobs for the following repositories.
kubeflow/examples
kubeflow/testing
kubeflow/tf-operator
kubeflow/reporting
remove tensorflow/k8s because this has been moved to kubeflow/tf-operator.
Move tensorflow/k8s to kubeflow/tf-operator kubeflow/training-operator#350
We don't add periodic jobs right now; we might add these in the future.