-
Notifications
You must be signed in to change notification settings - Fork 55
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
Bumper jobs failing #1005
Comments
Added And added it to nwaku&nimbus: |
Found out experimentaly, that |
Codex repo is missing the |
Bumper is green again: @jakubgs, please review this workaround when you are back. Token for |
This is a useful part of security guide: This list describes the recommended approaches for accessing repository data within a workflow, in descending order of preference:
We use 4 approach. And It looks like we can't use 1, because we need write access to other repos. So it's wort to check 2 and 3. |
If possible we should avoid using old "classic" tokens and we should try to create as specific "Fine-grained personal access tokens" as possible. They can be created to target a specific repo with a specific permission only. |
"Fine-grained" token didn't work with user owner - git fails to push:
https://github.com/status-im/nim-libp2p/actions/runs/7252738074/job/20160975446 Organisation wide token worked! We'll need to have 1 token for each of 3 orgs (waku, status and codex) and update the job. |
Looks like Deploy key is actually might be useful for us: |
@yakimant what's the status here? |
No update since January. There are 2 options:
|
is there any pending input from the libp2p team? |
Started to fail after Tanguy left.
After getting access to the repo Settings @diegomrsantos and I discovered the probably personal token was used for
ACTIONS_GITHUB_TOKEN
.Token is setup here:
https://github.com/status-im/nim-libp2p/settings/secrets/actions
See action code:
https://github.com/status-im/nim-libp2p/blob/d2c98bd87d540ec2f2d1a7c244b60386dfda66d3/.github/workflows/bumper.yml#L28
Actions & security docs:
The text was updated successfully, but these errors were encountered: