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

Support cgroupns option in containers.run/containers.create #2930

Merged
merged 1 commit into from
Jul 29, 2022

Conversation

david0
Copy link
Contributor

@david0 david0 commented Jan 3, 2022

cgroupns=host is needs be used to run systemd v2 containers, which needs to be used with docker desktop >=4.3.0 (docker/for-mac#6073)

@vician
Copy link

vician commented May 27, 2022

Hi, What needs to be completed to get this merged? It is currently blocking ansible-collections/community.docker#338 Can I help somehow?

@fcayre
Copy link

fcayre commented Jul 28, 2022

I am very interesting in getting this merge too (running containers with saltstack on debian >= buster requires this patch IFAIU).
Is it planned, can I help?
The failing check ("continuous-integration/jenkins/pr-merge") does not seem related to the change.

@david0
Copy link
Contributor Author

david0 commented Jul 28, 2022

We are blocked by this too. Unfortunately the docker-py seems to be dead-ish atm. The commits on master give some hope...

Signed-off-by: David Otto <ottodavid@gmx.net>
@milas milas merged commit d69de54 into docker:main Jul 29, 2022
milas added a commit to milas/docker-py that referenced this pull request Jul 29, 2022
See docker#2930.

Signed-off-by: Milas Bowman <milas.bowman@docker.com>
@milas milas added this to the 6.0.0 milestone Jul 30, 2022
@david0
Copy link
Contributor Author

david0 commented Aug 1, 2022

@milas Thanks a lot! 🎉

milas added a commit to milas/docker-py that referenced this pull request Aug 12, 2022
See docker#2930.

Signed-off-by: Milas Bowman <milas.bowman@docker.com>
milas added a commit that referenced this pull request Aug 12, 2022
See #2930.

Signed-off-by: Milas Bowman <milas.bowman@docker.com>
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