Skip to content

Adds valid PAT testing to existing ec2 pipes #11486

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

Merged
merged 2 commits into from
Mar 29, 2022

Conversation

supertopher
Copy link
Contributor

adds the same workflow seen in gradle.yml to other pipelines

What

grade.yml is running through PAT keys to check for valid keys. lets do that elsewhere

How

adds a need before GitHub token invoking the supertopher branch

adds the same workflow seen in gradle.yml to other pipelines
@supertopher supertopher temporarily deployed to more-secrets March 28, 2022 21:49 Inactive
@supertopher supertopher temporarily deployed to more-secrets March 28, 2022 21:49 Inactive
id: variables
run: |
./tools/bin/find_non_rate_limited_PAT \
${{ secrets.SUPERTOPHER_PAT }} \
Copy link
Contributor

@davinchia davinchia Mar 29, 2022

Choose a reason for hiding this comment

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

we should update this to the token list we spoke about before merging this in.

Copy link
Contributor

Choose a reason for hiding this comment

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

same for the rest of the jobs.

Copy link
Contributor

@davinchia davinchia left a comment

Choose a reason for hiding this comment

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

Nice work! Fingers crossed!

rename SELF_RUNNER_GITHUB_ACCESS_TOKEN_1 ->
DAVINCHIA_PAT

remove LABEL_BOT_USERNAME
Copy link
Contributor Author

@supertopher supertopher left a comment

Choose a reason for hiding this comment

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

LGTM assuming testing passes I'll merge away

@@ -12,4 +12,4 @@ jobs:
- uses: harshithmullapudi/label-actions@75686c2b3de17244526f10a22424f319d0bc134f
with:
github-token: ${{ secrets.OCTAVIA_PAT }}
github-username: ${{ secrets.LABEL_BOT_USERNAME }}
github-username: octavia-squidington-iii
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Whatever we were using before it's Octavia now, I'm killing these secrets after this

@supertopher supertopher temporarily deployed to more-secrets March 29, 2022 14:37 Inactive
@supertopher supertopher temporarily deployed to more-secrets March 29, 2022 14:38 Inactive
@supertopher supertopher merged commit 2693107 into master Mar 29, 2022
@supertopher supertopher deleted the more_valid_PATS_more_places branch March 29, 2022 15:04
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