Skip to content

manager/allocator: delete cnmallocator package#3172

Merged
dperny merged 1 commit into
moby:masterfrom
corhere:drop-cnmallocator
Feb 27, 2024
Merged

manager/allocator: delete cnmallocator package#3172
dperny merged 1 commit into
moby:masterfrom
corhere:drop-cnmallocator

Conversation

@corhere

@corhere corhere commented Feb 26, 2024

Copy link
Copy Markdown
Collaborator

It is only needed to integrate Swarmkit into Moby networking and no Swarmkit package imports it. It is also the only package remaining which imports from github.com/docker/docker. Delete it to break the module dependency cycle with Moby. The code will be moved over to the https://github.com/moby/moby repo to live closer to the libnetwork logic that it is so tightly integrated into.

- What I did

- How I did it

- How to test it

- Description for the changelog

It is only needed to integrate Swarmkit into Moby networking and no
Swarmkit package imports it. It is also the only package remaining which
imports from github.com/docker/docker. Delete it to break the module
dependency cycle with Moby. The code will be moved over to the
https://github.com/moby/moby repo to live closer to the libnetwork logic
that it is so tightly integrated into.

Signed-off-by: Cory Snider <csnider@mirantis.com>
@dperny dperny merged commit 911c976 into moby:master Feb 27, 2024
@corhere corhere deleted the drop-cnmallocator branch February 27, 2024 17:35
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