Skip to content

Release fails because opentelemetrybot does not have permission to set PR labels #3444

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

Closed
aabmass opened this issue Apr 25, 2025 · 4 comments · Fixed by #3445
Closed

Release fails because opentelemetrybot does not have permission to set PR labels #3444

aabmass opened this issue Apr 25, 2025 · 4 comments · Fixed by #3445

Comments

@aabmass
Copy link
Member

aabmass commented Apr 25, 2025

@emdneto updated the release workflow to add labels to release PRs in #3424. I just ran a release and it fails because @opentelemetrybot does not have permission to add labels apparently

pull request update failed: GraphQL: opentelemetrybot does not have permission to update the pull request PR_kwDODSTwx86T9CdO. (updatePullRequest)

example run

@aabmass
Copy link
Member Author

aabmass commented Apr 25, 2025

@trask can you help us with @opentelemetrybot permissions?

@trask
Copy link
Member

trask commented Apr 25, 2025

currently @opentelemetrybot has no permissions to any otel repos

there's discussion in open-telemetry/community#2127 how we can create repo-specific bots for cases where more permissions are needed

in this case though I think you could use the default GITHUB_TOKEN to add the label after creating the PR?

@emdneto
Copy link
Member

emdneto commented Apr 25, 2025

currently @opentelemetrybot has no permissions to any otel repos

there's discussion in open-telemetry/community#2127 how we can create repo-specific bots for cases where more permissions are needed

in this case though I think you could use the default GITHUB_TOKEN to add the label after creating the PR?

oh 😓

I can try to add the label after the PR is created. Thanks.

@open-telemetry/python-maintainers I'll revert my changes for now and open a PR with @trask suggestion.

@aabmass
Copy link
Member Author

aabmass commented Apr 25, 2025

Thank you Trask!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants