Skip to content

containerd: Switch to 1.25.9#412

Merged
thaJeztah merged 1 commit intodocker:mainfrom
vvoland:c8d-go
Apr 15, 2026
Merged

containerd: Switch to 1.25.9#412
thaJeztah merged 1 commit intodocker:mainfrom
vvoland:c8d-go

Conversation

@vvoland
Copy link
Copy Markdown
Collaborator

@vvoland vvoland commented Apr 15, 2026

1.26 comes from the main branch but the last release (2.2.3) is still on Go 1.25.

For a follow up:

  • adjust the automatic bump to pick the latest stable release as source
  • allow overriding go version in build workflow?

@vvoland vvoland requested a review from a team as a code owner April 15, 2026 09:05
@thaJeztah
Copy link
Copy Markdown
Member

Ah, right, but that also requires the default branch we use for CI to be switched to release/v2.2, because the main branch now requires go1.26 (because ... kubernetes)

Comment thread docker-bake.hcl Outdated
@@ -518,7 +518,7 @@ target "_pkg-containerd" {
PKG_NAME = PKG_NAME != null && PKG_NAME != "" ? PKG_NAME : "containerd.io"
PKG_REPO = PKG_REPO != null && PKG_REPO != "" ? PKG_REPO : "https://github.com/containerd/containerd.git"
PKG_REF = PKG_REF != null && PKG_REF != "" ? PKG_REF : "main"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
PKG_REF = PKG_REF != null && PKG_REF != "" ? PKG_REF : "main"
PKG_REF = PKG_REF != null && PKG_REF != "" ? PKG_REF : "release/2.2"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

or should we have a matrix in CI? (main, release/2.2)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I think it's ok to keep it simple for now and use the 2.2 branch

1.26 comes from the main branch but the last release (2.2.3) is still on
Go 1.25.

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
Copy link
Copy Markdown
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@thaJeztah thaJeztah merged commit 17ed3e6 into docker:main Apr 15, 2026
539 of 540 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.

2 participants