Skip to content

fix(ci): remove trivy, bump java deps, drop armv7#324

Merged
matifali merged 6 commits intomainfrom
fix/bump-trivy-version
Apr 10, 2026
Merged

fix(ci): remove trivy, bump java deps, drop armv7#324
matifali merged 6 commits intomainfrom
fix/bump-trivy-version

Conversation

@matifali
Copy link
Copy Markdown
Member

@matifali matifali commented Apr 10, 2026

Changes

1. Remove Trivy scanning (.github/workflows/build.yaml)

The Trivy install step was failing because v0.37.1 binaries were deleted during the Trivy supply-chain incident (Feb 28, 2026). A second incident on March 19 published malicious binaries. Removed all Trivy steps until the situation stabilizes.

2. Bump Maven + Gradle in Java image (images/java/ubuntu.Dockerfile)

Tool Before After Why
Maven 3.9.12 3.9.14 dlcdn.apache.org only serves the latest release; 3.9.12 was 404ing
Maven URL dlcdn.apache.org archive.apache.org Archive retains older releases, won't 404 on next bump
Gradle 6.7 8.14.2 6.7 is from 2020; 8.14.2 is the latest 8.x (9.x requires Java 17+, this image uses Java 11)

3. Drop armv7 platform support (images/node/ubuntu.Dockerfile, scripts/build_images.sh)

  • Removed armv7-specific Node.js version logic and simplified to just setup_lts.x.
  • Removed linux/arm/v7 from the Depot build platforms.

🤖 This PR was created with the help of Coder Agents, and needs a human review. 🧑💻

Copy link
Copy Markdown
Member

@johnstcn johnstcn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would suggest we instead migrate to a different tool.

@matifali
Copy link
Copy Markdown
Member Author

matifali commented Apr 10, 2026

@johnstcn, are we removing it altogether for now? builds are broken

@johnstcn
Copy link
Copy Markdown
Member

I'd rather remove it altogether, but I'll defer to @jdomeracki-coder

The Trivy supply-chain compromise (Feb/Mar 2026) deleted releases
v0.27.0 through v0.69.1 and later published malicious binaries.
Remove Trivy scanning entirely until the situation stabilizes.
@matifali matifali changed the title fix(ci): bump trivy from v0.37.1 to v0.69.3 fix(ci): remove trivy scanning Apr 10, 2026
Copy link
Copy Markdown
Member

@johnstcn johnstcn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving, but would like buy-in from JD before merging.

@matifali matifali changed the title fix(ci): remove trivy scanning fix(ci): remove trivy scanning and bump maven to 3.9.14 Apr 10, 2026
- Switch Maven download URL from dlcdn.apache.org to
  archive.apache.org so pinned versions don't 404 when a new
  release is published.
- Bump Maven 3.9.12 -> 3.9.14 (latest stable).
- Bump Gradle 6.7 -> 8.14.2 (latest 8.x; 9.x requires Java 17+
  but this image uses Java 11).
@matifali matifali force-pushed the fix/bump-trivy-version branch from a7ddbe4 to 6e37a9d Compare April 10, 2026 11:28
@matifali matifali changed the title fix(ci): remove trivy scanning and bump maven to 3.9.14 fix(ci): remove trivy scanning, bump maven and gradle Apr 10, 2026
@matifali matifali changed the title fix(ci): remove trivy scanning, bump maven and gradle fix(ci): remove trivy scanning Apr 10, 2026
@matifali matifali changed the title fix(ci): remove trivy scanning fix(ci): remove trivy, bump java deps, drop armv7 Apr 10, 2026
@johnstcn
Copy link
Copy Markdown
Member

:shipit:

@matifali matifali merged commit 260f196 into main Apr 10, 2026
2 checks passed
@matifali matifali deleted the fix/bump-trivy-version branch April 10, 2026 11:53
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