-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Create pod CGroups with the systemd driver #1266
Conversation
First new PR since papr was fixed, let's see if it works here |
Alright, so the bot is working, but PRs are still not being tested without Homu. |
bot, retest this please |
2 similar comments
bot, retest this please |
bot, retest this please |
1adca70
to
afc0893
Compare
I can't replicate this travis failure at all locally. It looks like it should be succeeding - the devmapper and btrfs packages it complained about are present. |
Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
721247c
to
53f5b04
Compare
Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
869cc40
to
804e830
Compare
bot, retest this please |
1 similar comment
bot, retest this please |
@baude We fixed travis and now Papr is flaking. Just can't win. |
I always win |
LGTM |
LGTM |
LGTM |
📌 Commit 804e830 has been approved by |
⚡ Test exempted: merge already tested. |
Signed-off-by: Matthew Heon <matthew.heon@gmail.com> Closes: #1266 Approved by: baude
Signed-off-by: Matthew Heon <matthew.heon@gmail.com> Closes: #1266 Approved by: baude
Signed-off-by: Matthew Heon <matthew.heon@gmail.com> Closes: #1266 Approved by: baude
Like the name says. Expand pod support so we now create per-pod CGroups when managed by systemd.
Also change the default parent to machine.slice (more correct than system.slice - it's where containers are supposed to go)