Skip to content

github actions: update triage bot to use client id#4834

Merged
alecbcs merged 1 commit into
spack:developfrom
alecbcs:ci/update-triage-to-client-id
May 13, 2026
Merged

github actions: update triage bot to use client id#4834
alecbcs merged 1 commit into
spack:developfrom
alecbcs:ci/update-triage-to-client-id

Conversation

@alecbcs

@alecbcs alecbcs commented May 13, 2026

Copy link
Copy Markdown
Member

GitHub now recommends using an app's client ID instead of the app ID to generate short lived tokens. This PR swaps to using the new client ID method. After merging we should remove the app ID from the internally configured secrets.

actions/create-github-app-token#353

Signed-off-by: Alec Scott <alec@llnl.gov>
@spackbot-triage spackbot-triage Bot added the ci Issues related to Continuous Integration label May 13, 2026
@alecbcs alecbcs assigned alecbcs and unassigned alecbcs May 13, 2026

@kwryankrattiger kwryankrattiger left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@alecbcs alecbcs enabled auto-merge (squash) May 13, 2026 19:00
@alecbcs alecbcs disabled auto-merge May 13, 2026 20:47
@alecbcs alecbcs merged commit c1bb64b into spack:develop May 13, 2026
26 of 27 checks passed
@alecbcs

alecbcs commented May 13, 2026

Copy link
Copy Markdown
Member Author

Force merged as this PR does not affect failing GitLab CI

kshea21 pushed a commit that referenced this pull request May 14, 2026
Signed-off-by: Alec Scott <alec@llnl.gov>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Issues related to Continuous Integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants