Skip to content
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

Fix Update Version step in push GHA #10

Closed

Conversation

eb-oss
Copy link

@eb-oss eb-oss commented Nov 22, 2024

We're seeing a 403 when attempting to push with the create-pull- request action. I believe this is being caused by the fact that the checkout is being done using the default action token, but the push is trying to use the onf-bot's PAT. Using the same token for both should fix the error.

We're seeing a 403 when attempting to push with the create-pull-
request action. I believe this is being caused by the fact that the
checkout is being done using the default action token, but the push
is trying to use the onf-bot's PAT. Using the same token for both
should fix the error.

Signed-off-by: Eric Ball <[email protected]>
@eb-oss eb-oss requested a review from gab-arrobo November 22, 2024 01:06
@gab-arrobo
Copy link
Contributor

gab-arrobo commented Nov 22, 2024

We're seeing a 403 when attempting to push with the create-pull- request action. I believe this is being caused by the fact that the checkout is being done using the default action token, but the push is trying to use the onf-bot's PAT. Using the same token for both should fix the error.

Hi @eb-oss,

I think the issue is the permissions for the token and I say this because we are already using this same GitHub Action in all of our active repositories in the omec-project organization. Here you can see an example of it (https://github.com/omec-project/amf/blob/main/.github/workflows/push.yml#L148). My suggestion would be to compare the permissions between the GH_OMEC_PAT and the GH_ONF_BOT_PAT tokens because everything works fine when using the GH_OMEC_PAT token. Here is an example PR: omec-project/amf#337 for your reference

@eb-oss
Copy link
Author

eb-oss commented Nov 22, 2024

Thanks for the additional info. I was able to get it to work by using the same permissions as the OMEC token. I'll close this PR.

@eb-oss eb-oss closed this Nov 22, 2024
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