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

ci: add missing permission to Trivy scan job #4298

Merged
merged 1 commit into from
Dec 29, 2024

Conversation

erikgb
Copy link
Contributor

@erikgb erikgb commented Dec 29, 2024

Hopefully, this should fix the current error on the main branch:

RequestError [HttpError]: Resource not accessible by integration
    at /home/runner/work/_actions/github/codeql-action/04daf0[14](https://github.com/weaveworks/weave-gitops/actions/runs/12536344145/job/34959246645#step:4:15)b50eaf774287bf3f0f1869d4b4c4b913/node_modules/@octokit/request/dist-node/index.js:86:21
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async requestWithGraphqlErrorHandling (/home/runner/work/_actions/github/codeql-action/04daf014b50eaf774287bf3f0f1869d4b4c4b913/node_modules/@octokit/plugin-retry/dist-node/index.js:71:20)
    at async Job.doExecute (/home/runner/work/_actions/github/codeql-action/04daf014b50eaf774287bf3f0f1869d4b4c4b913/node_modules/bottleneck/light.js:405:18) {
  status: 403,
  response: {
    url: 'https://api.github.com/repos/weaveworks/weave-gitops/code-scanning/analysis/status',
    status: 403,

The permission granted in this PR is the same as the one given to the CodeQL scanning job. I have a feeling the two jobs require the same permissions.

@casibbald casibbald merged commit 26e3e65 into weaveworks:main Dec 29, 2024
17 checks passed
casibbald added a commit that referenced this pull request Dec 29, 2024
* chore: address pinned dependencies error

* ci: add missing permission to Trivy scan job (#4298)

* chore: bump Flux module dependencies (#4297)

---------

Co-authored-by: Erik Godding Boye <[email protected]>
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