-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
fix(ci): multiple workflows need 'contents: write' permissions #21940
Conversation
Datadog ReportBranch report: ✅ 0 Failed, 7 Passed, 0 Skipped, 25.46s Total Time |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you may need to update this in all of the workflows that call this one, too:
vector/.github/workflows/release.yml
Lines 3 to 5 in 4f14967
permissions: | |
contents: read | |
packages: write |
vector/.github/workflows/nightly.yml
Lines 3 to 5 in 4f14967
permissions: | |
contents: read | |
packages: write |
vector/.github/workflows/custom_builds.yml
Lines 3 to 5 in 4f14967
permissions: | |
contents: read | |
packages: write |
* pin to latest VRL * cargo vdev build release-cue * update versions * fix static img link (need to automate this) * add description * vrl changelog added * fix typo * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * spell checker fixes * remove PartialOrd entry * dd-rust-license-tool write * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * PR feedback and cue fmt * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * fix indentation * fix indentation --------- Co-authored-by: Sandra (neko) <[email protected]> Co-authored-by: Jesse Szwedko <[email protected]> fix(ci): multiple workflows need 'contents: write' permissions (#21940) * fix(ci): publish workflow needs more permissions * update more workflows
* pin to latest VRL * cargo vdev build release-cue * update versions * fix static img link (need to automate this) * add description * vrl changelog added * fix typo * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Sandra (neko) <[email protected]> * spell checker fixes * remove PartialOrd entry * dd-rust-license-tool write * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * PR feedback and cue fmt * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * Update website/cue/reference/releases/0.43.0.cue Co-authored-by: Jesse Szwedko <[email protected]> * fix indentation * fix indentation --------- Co-authored-by: Sandra (neko) <[email protected]> Co-authored-by: Jesse Szwedko <[email protected]> fix(ci): multiple workflows need 'contents: write' permissions (#21940) * fix(ci): publish workflow needs more permissions * update more workflows
Summary
Fixes https://github.com/vectordotdev/vector/actions/runs/12143081309
Change Type
Is this a breaking change?
How did you test this PR?
Does this PR include user facing changes?
Checklist
Cargo.lock
), pleaserun
dd-rust-license-tool write
to regenerate the license inventory and commit the changes (if any). More details here.References
https://docs.github.com/en/actions/writing-workflows/choosing-what-your-workflow-does/controlling-permissions-for-github_token