Skip to content

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