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

feat(release): Replace release bot with GH app #14628

Merged
merged 1 commit into from
Dec 10, 2024

Conversation

Jeffreyhung
Copy link
Member

The getsentry-release is a GitHub bot account that is used in various automation, and the ${{ secrets.GH_RELEASE_PAT }} is a personal access token from that bot account. We are using a regular GitHub account as a bot while it should be a non-human account since there are no humans behind it. Hence, we are replacing it with a GitHub App.

Functionality wise, there will be no difference.

More details: https://www.notion.so/sentry/DACI-Replace-GitHub-bot-accounts-with-GitHub-Apps-getsentry-release-15109965d1204a91b9be71c49e8b66e0?pvs=4

#skip-changelog

@Jeffreyhung Jeffreyhung enabled auto-merge December 9, 2024 23:44
Copy link
Member

@Lms24 Lms24 left a comment

Choose a reason for hiding this comment

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

@Jeffreyhung was this already tested in other repos? Just asking because I'd like to keep the risk of blocking a release from this repo as minimal as possible. (Also, we'd like to cut a release today/tomorrow so we might just wait with merging this in)

Copy link
Member

@AbhiPrasad AbhiPrasad left a comment

Choose a reason for hiding this comment

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

Here's it being done in Sentry getsentry/sentry#81692 - let's get this merged in

@Jeffreyhung Jeffreyhung merged commit 83c6504 into develop Dec 10, 2024
66 checks passed
@Jeffreyhung Jeffreyhung deleted the Replace-release-bot-with-GH-app branch December 10, 2024 17:07
@Jeffreyhung
Copy link
Member Author

@Lms24 yes this has been tested, and we have heard success cases since we start rolling this out!

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.

3 participants