Skip to content

HDDS-14746. gh pr edit fails due to lack of permission in OZONE_WEBSITE_BUILD token#9852

Merged
adoroszlai merged 1 commit intoapache:masterfrom
sarvekshayr:HDDS-14746
Feb 28, 2026
Merged

HDDS-14746. gh pr edit fails due to lack of permission in OZONE_WEBSITE_BUILD token#9852
adoroszlai merged 1 commit intoapache:masterfrom
sarvekshayr:HDDS-14746

Conversation

@sarvekshayr
Copy link
Contributor

What changes were proposed in this pull request?

Error while updating existing PR

GraphQL: Your token has not been granted the required scopes to execute this query. The 'login' field requires one of the following scopes: ['read:org'], but your token has only been granted the: ['public_repo', 'repo:status'] scopes. Please modify your token's scopes at: https://github.com/settings/tokens., Your token has not been granted the required scopes to execute this query. The 'name' field requires one of the following scopes: ['read:org', 'read:discussion'], but your token has only been granted the: ['public_repo', 'repo:status'] scopes. Please modify your token's scopes at: https://github.com/settings/tokens., Your token has not been granted the required scopes to execute this query. The 'slug' field requires one of the following scopes: ['read:org', 'read:discussion'], but your token has only been granted the: ['public_repo', 'repo:status'] scopes. Please modify your token's scopes at: https://github.com/settings/tokens.

Replaced gh pr edit with gh pr comment so it tracks all the commits added to the existing PR.

What is the link to the Apache JIRA

HDDS-14746

How was this patch tested?

Can be tested once the PR is merged.

Copy link
Contributor

@adoroszlai adoroszlai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @sarvekshayr for the patch. This looks good for now, but I think we should tweak the message in the long run, the current one is a bit verbose for each additional comment.

@sarvekshayr
Copy link
Contributor Author

Thanks @sarvekshayr for the patch. This looks good for now, but I think we should tweak the message in the long run, the current one is a bit verbose for each additional comment.

Yes, we should either tweak the message or request permission on the token to edit PR during the next renewal.

@adoroszlai adoroszlai merged commit 8044a55 into apache:master Feb 28, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants