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

[meta] Update Cirrus GITHUB_TOKEN #924

Merged
merged 1 commit into from
Feb 15, 2024
Merged

Conversation

confused-Techie
Copy link
Member

This PR updates the GITHUB_TOKEN for CirrusCI.
As the previously used token had expired yesterday.

Getting this updated will be a requirement for our next Regular Release

Copy link
Member

@meadowsys meadowsys left a comment

Choose a reason for hiding this comment

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

looks good to me!

edit: I don't think we need to wait for CI here, right? heh

Copy link
Member

@DeeDeeG DeeDeeG left a comment

Choose a reason for hiding this comment

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

Also approving this.

The only way we'd know if this is not working is if we mange to get rate-limited in the vscode-ripgrep download in Cirrus. Hard to prove the opposite (that it is authed to the github.com REST API properly) with just one CI run anyhow.

@meadowsys
Copy link
Member

The only way we'd know if this is not working

now I wonder if the GitHub API just silently proceeds unauthenticated (if it can) on invalid token, or hard errors (I would think its the latter, but I'm not sure, thinking about it)

@confused-Techie
Copy link
Member Author

@meadowsys From my experience GitHub will error out on an invalid token. Since at that point it becomes an authentication failure, rather than a missing parameter it will try to skip. So pretty sure if we incorrectly set this value we would see lots of failures on the next CirrusCI run due to authentication failures in GitHub.

But thanks both of your for your review!

@confused-Techie confused-Techie merged commit 68c88e7 into master Feb 15, 2024
103 checks passed
@confused-Techie confused-Techie deleted the update-cirrus-token branch February 15, 2024 16:07
@meadowsys
Copy link
Member

GitHub will error out on an invalid token

then it would be pretty obvious if it doesn't work, will just error out immediately >w<

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