-
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
Migrate pull-kubernetes-bazel-build to k8s-infra-prow-build #19855
Migrate pull-kubernetes-bazel-build to k8s-infra-prow-build #19855
Conversation
Part of kubernetes#19073 Ref: kubernetes#18550 Signed-off-by: Arnaud Meukam <[email protected]>
/lgtm but hold for Aaron's review. |
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.
/approve
/lgtm
/hold
I want us to confirm that ci-kubernetes-bazel-build is fully functional over there before releasing the hold to move forward with presubmits
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ameukam, spiffxp 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 |
Most recent run of the ci-kubernetes-build-canary is still failing: https://prow.k8s.io/view/gcs/kubernetes-jenkins/logs/ci-kubernetes-build-canary/1324380000262033408 |
#19483 (comment) we are writing artifacts, I would be willing to move forward with this However, I'm not going to remove the /hold right now because I can't commit to babysitting these jobs / reverting over the weekend. If someone else wants to, please go for it. Otherwise I will move on this Monday before test freeze |
+1 for remove the /hold Monday. |
/hold cancel |
@ameukam: 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. |
Part of #19073
Ref: #18550
Signed-off-by: Arnaud Meukam [email protected]