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.

Sign up for GitHub

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

Jump to bottom

bump-formula-action fails with GH app token #965

Closed
rfratto opened this issue May 30, 2024 · 0 comments · Fixed by #1025
Closed

bump-formula-action fails with GH app token #965

rfratto opened this issue May 30, 2024 · 0 comments · Fixed by #1025
Labels
frozen-due-to-age

Comments

@rfratto
Copy link
Member

rfratto commented May 30, 2024

bump-formula-action does not work now that we use GH app token for all cross-repository actions:

/home/linuxbrew/.linuxbrew/Homebrew/Library/Homebrew/utils/github/api.rb:384:in `raise_error': GitHub API Error: Resource not accessible by integration (GitHub::API::AuthenticationFailedError)
HOMEBREW_GITHUB_API_TOKEN may be invalid or expired; check:
  https://github.com/settings/tokens

	from /home/linuxbrew/.linuxbrew/Homebrew/Library/Homebrew/utils/github/api.rb:285:in `open_rest'
	from /home/runner/work/_actions/dawidd6/action-homebrew-bump-formula/v3/main.rb:61:in `<module:Homebrew>'
	from /home/runner/work/_actions/dawidd6/action-homebrew-bump-formula/v3/main.rb:17:in `<main>'

This is because the action gets user information via /user, which doesn't work with app tokens. See dawidd6/action-homebrew-bump-formula#66
rfratto added a commit that referenced this issue Jun 11, 2024
@rfratto
ci: fix bump-formula-pr action
ec8e7b3 
This commit changes the bump-formula-pr action to directly call `brew
bump-formula-pr` for creating a PR to bump a formula.

The previous action didn't work for GitHub Apps as it used a user-only
API for determining the commit name and email. For now, these fields are
hardcoded for the @grafana-alloybot app.

Closes #965
@rfratto rfratto mentioned this issue Jun 11, 2024
Merged
rfratto added a commit that referenced this issue Jun 11, 2024
@rfratto
ci: fix bump-formula-pr action (#1025)
96aa51c 
This commit changes the bump-formula-pr action to directly call `brew
bump-formula-pr` for creating a PR to bump a formula.

The previous action didn't work for GitHub Apps as it used a user-only
API for determining the commit name and email. For now, these fields are
hardcoded for the @grafana-alloybot app.

Closes #965
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
frozen-due-to-age
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

ci: fix bump-formula-pr action grafana/alloy
1 participant
@rfratto