-
Notifications
You must be signed in to change notification settings - Fork 796
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
Add comment for k8s-infra-prow tide labels #2800
Conversation
Signed-off-by: Eddie Zaneski <eddiezane@gmail.com>
67e42a7
to
52bb9c0
Compare
/hold |
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
Sorry we didn't have something like this here in the first place. Good catch
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: eddiezane, 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 |
So the way to get its tide to stop trying to look at this repo is to remove the tide query entirely. The current status is that both tides report to the same context, so last one to update wins. It's not very friendly to new contributors to this repo. It does let us know that k8s-infra's tide is running and doing stuff though. So I could go either way. |
Suggestion of spiffxp: kubernetes#2800 (comment). Tide is not required for the moment. Signed-off-by: Arnaud Meukam <ameukam@gmail.com>
@eddiezane: PR needs rebase. 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. |
/close |
@spiffxp: Closed this PR. 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. |
This PR is to ensure that #2795 does not happen.
/assign @ameukam