-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Mention containers.conf settings for podman machine commands #24242
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rhatdan 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 |
@@ -52,6 +52,8 @@ Fedora CoreOS upgrades come out every 14 days and are detected and installed aut | |||
For more information on updates and advanced configuration, | |||
see the Fedora CoreOS documentation about [auto-updates](https://docs.fedoraproject.org/en-US/fedora-coreos/auto-updates/) and [update strategies](https://coreos.github.io/zincati/usage/updates-strategy/). | |||
|
|||
Default podman machines settings can set via the [machine] section in the containers.conf(5) file. |
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.
Default podman machines settings can set via the [machine] section in the containers.conf(5) file. | |
Default podman machines settings can be set via the [machine] section in the containers.conf(5) file. |
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
Cockpit tests failed for commit 3a4c446. @martinpitt, @jelly, @mvollmer please check. |
LGTM |
cockpit f41 test failure is the same "occasional criu failure" than in #24238 (comment) . It ran before the improved debugging cockpit-project/cockpit-podman#1878 , so this is useless, and can be ignored or retried here. I.e. all good from our side. |
/lgtm |
40dae5e
into
containers:main
Does this PR introduce a user-facing change?