-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
podman: Wrap the start command with cgroup manager too #8001
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: afbjorklund The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/ok-to-test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do you mind putting in the PR description which version of podman u tried this with on which os? I remember some version of podman didn't have that flag (maybe it was on mac)
Note that this means that podman-remote will be broken, until flipping the server to cgroupfs. /etc/containers/libpod.conf # CGroup Manager - valid values are "systemd" and "cgroupfs"
cgroup_manager = "systemd" Nothing that can be done on the podman client side to change that, so needs server config. |
kvm2 Driver Times for Minikube (PR 8001): [62.765769028 63.40405177600001 63.992705879] Averages Time Per Log
docker Driver Times for Minikube (PR 8001): [25.549244355000006 27.588306021 27.742802625] Averages Time Per Log
|
There is no documentation on how to set up the remote VM for Mac or Win, unfortunately... The upstream install docs all assume that such a Linux host is already available over SSH: https://podman.io/getting-started/installation
https://github.com/containers/libpod/blob/master/docs/source/markdown/podman-remote.conf.5.md |
/retest-this-please |
Got some weird errors testing this locally, after merging all the other required fixes. And the error messages still suck, so they will need to be "translated" for context
That is: there is no "podman daemon" here, and it's not an image. Deleting and starting again seems to fail to remove the podman volume.
|
When running with podman, to match the run cmd
Since it doesn't work for either "run" or "start" Error: unknown flag: --cgroup-manager
kvm2 Driver Times for Minikube (PR 8001): [63.337755976 64.18964069100001 62.98473456199999] Averages Time Per Log
docker Driver Times for Minikube (PR 8001): [25.433446959999998 25.23349494 27.938520155] Averages Time Per Log
|
When running with podman, to match the run cmd
Closes #7996