Skip to content

switch to github.com/opencontainers/cgroups#49562

Merged
AkihiroSuda merged 1 commit intomoby:masterfrom
thaJeztah:switch_cgroups
Mar 24, 2025
Merged

switch to github.com/opencontainers/cgroups#49562
AkihiroSuda merged 1 commit intomoby:masterfrom
thaJeztah:switch_cgroups

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

@thaJeztah thaJeztah commented Feb 28, 2025

depends on / stacked:

switch to github.com/opencontainers/cgroups

The runc libcontainer/cgroups package was moved to a separate
module; switch our use of the runc module to use the new
location.

- How to verify it

- Human readable description for the release notes

- A picture of a cute animal (not mandatory but encouraged)

@thaJeztah thaJeztah force-pushed the switch_cgroups branch 2 times, most recently from 2ce2e6f to f1fa233 Compare March 3, 2025 20:55
@thaJeztah thaJeztah self-assigned this Mar 3, 2025
@thaJeztah thaJeztah force-pushed the switch_cgroups branch 2 times, most recently from 4192c5d to f0b8b37 Compare March 6, 2025 09:11
Comment thread vendor.mod Outdated
github.com/opencontainers/cgroups v0.0.1
github.com/opencontainers/go-digest v1.0.0
github.com/opencontainers/image-spec v1.1.0
github.com/opencontainers/runc v1.2.5
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This reminds me of an old issue of mine, which can probably be closed already, but definitely after this PR? 🤔

#39166

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Not directly related; that ticket is more related to packaging (and http://github.com/docker/containerd-packaging). This is only a build-dependency on some of the Go libraries provided by runc.

The runc libcontainer/cgroups package was moved to a separate
module; switch our use of the runc module to use the new
location.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@vvoland vvoland added this to the 28.0.3 milestone Mar 24, 2025
@AkihiroSuda AkihiroSuda merged commit accda31 into moby:master Mar 24, 2025
163 checks passed
@thaJeztah thaJeztah deleted the switch_cgroups branch March 24, 2025 11:58
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.

Remove (or reduce) dependency on runc libcontainer dependency

4 participants