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

CI: Update Cirrus Rolling upload token #1086

Merged
merged 1 commit into from
Sep 2, 2024

Conversation

DeeDeeG
Copy link
Member

@DeeDeeG DeeDeeG commented Aug 31, 2024

Update the Cirrus Rolling binaries upload token, which had expired.


Some context, for anyone wanting to know more about this:

(Note for context, in case anyone is wondering and is unclear how we set these up: For convenience reasons, this is a fine-grained PAT with just the needed permissions for pulsar-rolling-releases repo, generated on my personal GitHub account. The PAT could just as well be made on anyone's account with adequate permissions to the pulsar-rolling-releases repo. We could further jump through some hoops to make a "GitHub App" to be able to auth as the app or as the org or whichever it is instead, but it appears, to put it mildly, to be a huge hassle and without a whole lot of benefit. So, going to keep doing this as a PAT for now.)

(Second note for more context: I already updated the GitHub Actions Rolling token, which I can do in the settings UI for this repo here on GitHub.com, and doesn't require any changes to repo files, so it didn't need a PR. Updating the Cirrus rolling upload token requires updating .cirrus.yml, and so requires a PR (e.g. this PR) to get that changed file into the repo.)

Copy link
Member

@confused-Techie confused-Techie left a comment

Choose a reason for hiding this comment

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

Thanks for updating this one!

Love the long explanation as if someone has never seen this before, can only imagine that being super helpful in the future checking the diff of these files

@confused-Techie confused-Techie merged commit fec1c7d into master Sep 2, 2024
103 checks passed
@confused-Techie confused-Techie deleted the ci-update-cirrus-rolling-upload-token branch September 2, 2024 21:39
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