-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
robots/commenter: add --include-locked flag #23029
robots/commenter: add --include-locked flag #23029
Conversation
Functions just like --include-archived and --include-closed. Added equivalent test cases, reordered a bit so the order is consistently "archived, closed, locked"
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: 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 |
/lgtm |
I was assuming this was going to push an image but I don't see a new one
|
https://prow.k8s.io/view/gs/kubernetes-jenkins/logs/post-test-infra-push-prow/1419823923347853312 Ah, this is pushed to k8s-prow (why?)
|
Opened #23032 to bump jobs to use this image |
This fixes something I just happened to notice while verifying #22997 as part of #12296
https://testgrid.k8s.io/sig-contribex-k8s-triage-robot#stale - the job is commenting, but it's also failing because it's unable to comment on a locked issue, e.g. https://storage.googleapis.com/kubernetes-jenkins/logs/ci-k8s-triage-robot-stale/1419722568809582592/build-log.txt
In this case the issue is being used as an announcements issue, so they don't want contributors to be able to comment on it.
Rather than add
is:unlocked
to all of the queries, ignore locked issues by default. This is the same thing we do with--include-archived
and--include-closed
.Added equivalent test cases, reordered a bit so the order is consistently "archived, closed, locked"