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
[JENKINS-61321] Fix permission check #4541
Conversation
🤕 |
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.
oops. We need to do more careful during the reviews. It is not the first time when we had such error, but the PR was fairly small and we should have caught it. OTOH low number of reports/votes suggests that the configuration-as-code/Pipeline-as-code adoption is going well
I plan to merge it tomorrow if no negative feedback.
ugh I was just testing this for sanity, and was very confused when I couldn't re-produce this. Well I'm not surprised I missed it when developing it =/ |
…NKINS-61321 [JENKINS-61321] Fix permission check (cherry picked from commit 692c684)
See JENKINS-61321.
Well, f***.
Manually confirmed working. Needs Project-based Matrix Authorization, and a user with Job/Configure on the per-job level, rather than the global configuration.
Proposed changelog entries
Proposed upgrade guidelines
N/A
Submitter checklist
Proposed changelog entries
section only if there are breaking changes or other changes which may require extra steps from users during the upgradeDesired reviewers
@mention
Maintainer checklist
Before the changes are marked as
ready-for-merge
:Proposed changelog entries
are correctupgrade-guide-needed
label is set and there is aProposed upgrade guidelines
section in the PR title. (example)lts-candidate