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

[SL-ONLY] Change workflow token to use the PAT token with additionnal worklfow permissions #230

Merged
merged 2 commits into from
Jan 24, 2025

Conversation

mkardous-silabs
Copy link
Contributor

@mkardous-silabs mkardous-silabs commented Jan 22, 2025

Description

For security reasons, the github token has no permissions to create or update workflows in a github action and there is no way to configure a workflow to give those permissions.

To do this, we had to create a PAT (Personnal access token) with the right permissions in the SiliconLabsSoftware org and use that token as a repository secret. The PAT can be re-used for multiple secrets.

Tests

Manual test

@mkardous-silabs mkardous-silabs force-pushed the fix/csa_update_workflow branch from 9a899ba to 3366c6c Compare January 24, 2025 16:00
@mkardous-silabs mkardous-silabs changed the title [SL-ONLY] Add action write permissions to the csa update workflow [SL-ONLY] Change workflow token to use the PAT token with additionnal worklfow permissions Jan 24, 2025
@mkardous-silabs mkardous-silabs marked this pull request as ready for review January 24, 2025 16:12
@mkardous-silabs mkardous-silabs requested a review from a team as a code owner January 24, 2025 16:12
@mkardous-silabs mkardous-silabs merged commit 35ad252 into main Jan 24, 2025
57 checks passed
@mkardous-silabs mkardous-silabs deleted the fix/csa_update_workflow branch January 24, 2025 18:13
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