Skip to content

deps: address CVEs and add security scans to CI#741

Merged
Fricounet merged 2 commits intocontainerd:mainfrom
fidencio:topic/update-critical-cves-in-deps
Apr 17, 2026
Merged

deps: address CVEs and add security scans to CI#741
Fricounet merged 2 commits intocontainerd:mainfrom
fidencio:topic/update-critical-cves-in-deps

Conversation

@fidencio
Copy link
Copy Markdown
Contributor

Bump Go to 1.25.9 and refresh modules (gRPC, containerd, AWS SDK, golang.org/x/*, go-jose, selinux, docker/cli, OpenTelemetry, CDI). Update optimizer-server Cargo.lock (rustix).

Add a CI job that runs govulncheck and OSV-Scanner on each push and PR.

Bump Go to 1.25.9 and refresh modules (gRPC, containerd, AWS SDK,
golang.org/x/*, go-jose, selinux, docker/cli, OpenTelemetry, CDI).
Update optimizer-server Cargo.lock (rustix).

Add a CI job that runs govulncheck and OSV-Scanner on each push and PR.

Signed-off-by: Fabiano Fidêncio <ffidencio@nvidia.com>
@fidencio fidencio force-pushed the topic/update-critical-cves-in-deps branch from 1bd7577 to 3f15015 Compare April 17, 2026 09:42
@fidencio
Copy link
Copy Markdown
Contributor Author

cc @imeoer

@fidencio
Copy link
Copy Markdown
Contributor Author

cc @bergwolf

Use S3 transfermanager instead of deprecated feature/s3/manager.
Replace deprecated golang.org/x/net/context with the standard library.

Signed-off-by: Fabiano Fidêncio <ffidencio@nvidia.com>
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 17, 2026

Codecov Report

❌ Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 24.45%. Comparing base (fc330cc) to head (e0e73dc).
⚠️ Report is 26 commits behind head on main.

Files with missing lines Patch % Lines
pkg/backend/s3.go 0.00% 4 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #741      +/-   ##
==========================================
+ Coverage   22.02%   24.45%   +2.42%     
==========================================
  Files         130      132       +2     
  Lines       11931    12237     +306     
==========================================
+ Hits         2628     2992     +364     
+ Misses       8960     8881      -79     
- Partials      343      364      +21     
Files with missing lines Coverage Δ
pkg/supervisor/supervisor.go 68.91% <ø> (ø)
pkg/backend/s3.go 22.58% <0.00%> (ø)

... and 16 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@fidencio
Copy link
Copy Markdown
Contributor Author

cc @Fricounet

Copy link
Copy Markdown
Contributor

@Fricounet Fricounet left a comment

Choose a reason for hiding this comment

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

Nice, LGTM thanks for the PR!

@fidencio
Copy link
Copy Markdown
Contributor Author

Ideally we'd like to have a release cut after this one gets merged, so we can ship this on Kata Containers and reduce the amount of vulns there that are not related to Kata itself :-)

@Fricounet Fricounet merged commit c2f1c15 into containerd:main Apr 17, 2026
18 checks passed
@Fricounet
Copy link
Copy Markdown
Contributor

I've published v0.15.15 which should have the fix

@fidencio
Copy link
Copy Markdown
Contributor Author

I've published v0.15.15 which should have the fix

Amazing, I'm already updating it on the Kata Containers side.
Thanks a whole lot, as usual!

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