-
Notifications
You must be signed in to change notification settings - Fork 2.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove Kubeflow Katib and training operators presubmit jobs #19657
Remove Kubeflow Katib and training operators presubmit jobs #19657
Conversation
/hold |
427ca49
to
c8eb833
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we also disable pytorch-operator
, mpi-operator
, ?
c8eb833
to
44de033
Compare
@andreyvelich: GitHub didn't allow me to request PR reviews from the following users: johnugeorge, gaocegege, terrytangyuan, ChanYiLin. Note that only kubernetes members and repo collaborators can review this PR, and authors cannot review their own PRs. 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. |
@andreyvelich pytorch-operator has been deleted previously in a separate PR for POC purpose. mpi-operator is there. Let's update this in slack channel to notify everyone and I will create some issues in those repos to track process of test migration |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can remove MPI's presubmit as well since we are mostly just using Travis for now.
/test pull-test-infra-bazel |
overally lgtm |
/test pull-test-infra-bazel |
rebase master changes. |
44de033
to
e267ff8
Compare
#19662 |
Should we fix the failure then merge it? |
/test pull-test-infra-bazel |
@gaocegege Yeah, seems sig-testing folks fix the issue. |
/lgtm |
@PatrickXYS: changing LGTM is restricted to collaborators 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. |
/hold cancel |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
@andreyvelich: changing LGTM is restricted to collaborators 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. |
/assign @jlewi |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: andreyvelich, Jeffwan, jlewi 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 files:
Approvers can indicate their approval by writing |
@Jeffwan: Updated the
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. |
POC to move presubmit jobs to AWS prow is working.
AWS prow runs presubmit jobs and sig-testing prow tide (kubeflow-ci-bot) merges PR.
Here's the PRs with new bot working.
kubeflow/pytorch-operator#305
kubeflow/katib#1356
/cc @andreyvelich @jlewi
/cc @johnugeorge @gaocegege @terrytangyuan @ChanYiLin
I hold to PR to make sure training operator owners are aware of this.