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

[WIP] podman: add new cgroup mode conmon-delegated #2

Open
wants to merge 7 commits into
base: conmon-delegate
Choose a base branch
from

Conversation

goochjj
Copy link

@goochjj goochjj commented Jun 18, 2020

My testing w.r.t. containers#6666

giuseppe and others added 3 commits June 18, 2020 16:44
add a new mode cgroup mode conmon-delegated.

When running under systemd there is no need to create yet another
cgroup for the container.

With conmon-delegated the current cgroup will be split in two sub
cgroups:

- supervisor
- container

The supervisor cgroup will hold conmon and the podman process, while
the container cgroup is used by the OCI runtime (using the cgroupfs
backend).

Closes: containers#6400

Depends on: containers/crun#409

Signed-off-by: Giuseppe Scrivano <[email protected]>
Not sure why we need another function to do cgroups in utils - we have pkg/cgroups
oci_conmon_linux already deals with moving conmon...
A lot of the conmonDelegated exceptions seem to deal with
manufacturing a cgroupParent when needed, instead of just setting
the cgroupParent we already have
@goochjj goochjj force-pushed the conmon-delegate-cgv1 branch from 1147665 to f67ceec Compare June 18, 2020 18:44
@giuseppe giuseppe force-pushed the conmon-delegate branch 6 times, most recently from ac92674 to 6ee5f74 Compare June 25, 2020 15:16
giuseppe pushed a commit that referenced this pull request Jun 30, 2020
Friendly amendment for pr 6751
giuseppe pushed a commit that referenced this pull request Jan 24, 2022
giuseppe pushed a commit that referenced this pull request Jan 24, 2022
Fix #2 for compat commit handling of --changes
giuseppe pushed a commit that referenced this pull request Aug 7, 2023
* Use vfkit command line assembly
* Inject ignition file into guest using http over vsock
* Ready notification through use of vsock

[NO NEW TESTS NEEDED]

Signed-off-by: Brent Baude <[email protected]>
giuseppe pushed a commit that referenced this pull request Aug 7, 2023
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