Skip to content

[v0.21 backport] vendor: docker, docker/cli v28.0.0-rc.2 - #2996

Merged
crazy-max merged 4 commits into
docker:v0.21from
crazy-max:0.21_backport_docker-28
Feb 17, 2025
Merged

crazy-max merged 4 commits into
docker:v0.21from
crazy-max:0.21_backport_docker-28

Conversation

@crazy-max

@crazy-max crazy-max commented Feb 14, 2025

Copy link
Copy Markdown
Member

backport of

We don't usually ship unstable releases, that's why Docker 28 RC bump on master got reverted before making our first RC:

But as discussed internally, Compose needs new network-related features from Docker 28 (cc @ndeloof @glours @thaJeztah). And because the Docker API have signature changes, Compose would not be able to vendor Docker 28.0.0-rc.1 with Buildx 0.21.0 as we are using the latest Docker 27.5.1 stable in current RC: https://github.com/docker/buildx/blob/v0.21.0-rc2/go.mod#L22

Compose could just vendor Buildx master but if patches are released in 0.21 branch, Compose might need to rely on it and would therefore be in broken state with vendoring. I don't think we have other choice than vendoring Docker 28 RC for this Buildx release.

@thompson-shaun

Copy link
Copy Markdown
Contributor

update to v28 rc2 now?

@crazy-max

crazy-max commented Feb 17, 2025

Copy link
Copy Markdown
Member Author

update to v28 rc2 now?

There is not rc2 tag yet but discussed with @vvoland, and I will backport #2998 once it has a 28.0.0-rc.2 tag.

full diff: moby/moby@v28.0.0-rc.1...v28.0.0-rc.2

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
full diff: docker/cli@v28.0.0-rc.1...v28.0.0-rc.2

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@crazy-max crazy-max changed the title [v0.21 backport] vendor: docker, docker/cli v28.0.0-rc.1 [v0.21 backport] vendor: docker, docker/cli v28.0.0-rc.2 Feb 17, 2025
@crazy-max

Copy link
Copy Markdown
Member Author

Added backport of #2998

@crazy-max
crazy-max marked this pull request as ready for review February 17, 2025 16:33

@thaJeztah thaJeztah left a comment

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.

LGTM

@crazy-max
crazy-max merged commit 5e5568f into docker:v0.21 Feb 17, 2025
@crazy-max
crazy-max deleted the 0.21_backport_docker-28 branch February 17, 2025 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants