Skip to content

ci: drop the trivy-sarif artifact upload the Actions budget rejects - #43

Merged
Cramraika merged 2 commits into
mainfrom
ci/drop-dead-artifact-uploads
Sep 14, 2026
Merged

Cramraika merged 2 commits into
mainfrom
ci/drop-dead-artifact-uploads

Conversation

@Cramraika

Copy link
Copy Markdown
Owner

Summary

The Actions account's storage quota now rejects every actions/upload-artifact call ("Artifact storage quota has been hit"). Per the operator ruling (OW-1138), CI evidence should not live in GitHub artifacts.

file:line artifact name class action
.github/workflows/trivy.yml:119 trivy-sarif COPY, no reader deleted the step

Searched (git grep -n "download-artifact\|trivy-sarif\|upload-sarif") — no consumer anywhere in the repo, and no codeql-action/upload-sarif sink either (this repo has a separate codeql.yml for that lane).

Scope

Only the upload step changed. Job logic, runners, triggers and permissions are untouched. Note the trivy scan step that generates trivy-results.sarif for that upload (--exit-code 0, still run) is left as-is per scope — it's now an orphaned producer the coordinator may want to remove separately.

🤖 Generated with Claude Code

The account's Actions storage quota rejects every actions/upload-artifact
call ("Artifact storage quota has been hit"). Per OW-1138, CI evidence
should not live in GitHub artifacts.

- .github/workflows/trivy.yml: deleted "Upload SARIF artifact"
  (trivy-sarif). No reader anywhere in the repo (no codeql-action/
  upload-sarif sink either).

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 14, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 2751c04d-f6d5-444e-8e11-760634f4b1a7


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

…ARIF

With the SARIF artifact gone, the second scan wrote a file nothing read.
It now emits JSON and prints CRITICAL/HIGH counts to the step summary.
The scan stays advisory (--exit-code 0); the verdict is unchanged.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@Cramraika
Cramraika merged commit f5f98ff into main Sep 14, 2026
9 of 12 checks passed
@Cramraika
Cramraika deleted the ci/drop-dead-artifact-uploads branch September 14, 2026 11:21
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.

1 participant