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

chore: bump moby/buildkit and opencontainers/runc versions, upgrade go to 1.22 #9364

Merged
merged 8 commits into from Mar 28, 2024

Conversation

alphanota
Copy link
Contributor

@alphanota alphanota commented Mar 28, 2024

Fixes: #9288, #9357, #9358
Related: Relevant tracking issues, for context
Merge before/after: Dependent or prerequisite PRs

Description

  • Bump the go version that skaffold is built on to the latest available version 1.22
  • Bump moby/buildkit and opencontainers/runc to latest available version.
  • This fixes various CVE's present in the images.
  • goconst and nakedret rules for go code style check are disabled at this moment, it seems the new golang-lint version has made those rules more rigid and they're blocking ci, we'll fix that in another issue
  • coverageredesign is disabled with GOEXPERIMENT=nocoverageredesign when running test coverage job due to a bug in go 1.22.1, which error out folder doesn't have tests and it's blocking ci. This is already been fixed in the go code base, we will for another go 1.22 release to turn on this.

User facing changes (remove if N/A)

Follow-up Work (remove if N/A)

Most likely bump up the versions of other dependencies if they exist.

@alphanota alphanota requested a review from a team as a code owner March 28, 2024 13:34
@alphanota alphanota marked this pull request as draft March 28, 2024 13:35
@alphanota alphanota changed the title Moby cherry chore: Upgrade go version to 1.22, upgrade runc and moby packages Mar 28, 2024
Copy link

codecov bot commented Mar 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 63.19%. Comparing base (290280e) to head (f78cd2a).
Report is 1119 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #9364      +/-   ##
==========================================
- Coverage   70.48%   63.19%   -7.30%     
==========================================
  Files         515      642     +127     
  Lines       23150    32983    +9833     
==========================================
+ Hits        16317    20842    +4525     
- Misses       5776    10531    +4755     
- Partials     1057     1610     +553     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ericzzzzzzz ericzzzzzzz added the kokoro:run runs the kokoro jobs on a PR label Mar 28, 2024
@kokoro-team kokoro-team removed the kokoro:run runs the kokoro jobs on a PR label Mar 28, 2024
@alphanota alphanota changed the title chore: Upgrade go version to 1.22, upgrade runc and moby packages chore: bump moby/buildkit and opencontainers/runc versions, upgrade go to 1.22 Mar 28, 2024
@alphanota alphanota marked this pull request as ready for review March 28, 2024 14:31
@ericzzzzzzz ericzzzzzzz merged commit ddf0709 into GoogleContainerTools:main Mar 28, 2024
15 checks passed
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.

Upgrade skaffold go version to 1.22.0
3 participants