-
Notifications
You must be signed in to change notification settings - Fork 39k
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
Run Docker containers as top-level containers in Debian. #8586
Conversation
I wish there was a way to single out GKE from Debian, as Debian has systemd enabled in the latest version. Looking around ... |
6a47489
to
959b434
Compare
Can one of the admins verify that this patch is reasonable to test? (reply "ok to test", or if you trust the user, reply "add to whitelist") If this message is too spammy, please complain @ixdy. |
959b434
to
0e242cd
Compare
ok to test |
PTAL @rjnagal this is ready to go I believe. We can improve the OS family detection later. I will also try to get the necessary changes into Docker 1.7. |
ok, let's go with this for now. LGTM |
@vmarmol sorry about you have to rebase it again. |
0e242cd
to
656be14
Compare
@dchen1107 no worries! Rebased and re-pushed. |
I kicked out shippable for you again. |
Run Docker containers as top-level containers in Debian.
This should fix the issue for GCE/GKE, I'll work on libcontainer/Docker changes to fix overall.
Part of #8215
/cc @rjnagal @dchen1107
Ran e2e and the run came back green!