Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(generate): add options to ignore unsupported pod options #116

Merged
merged 1 commit into from
Oct 6, 2024

Conversation

k9withabone
Copy link
Member

Added --ignore-infra-conmon-pidfile and --ignore-pod-id-file options to podlet generate pod.

The --infra-conmon-pidfile and --pod-id-file options are deliberately unimplemented by podlet podman pod create as they are set by Quadlet when generating the systemd service unit file, and cannot be set multiple times. Because podlet generate pod uses podlet podman pod create internally, this caused an error when attempting to generate a .pod Quadlet file from a pod created using either of those options, such as those created with the deprecated podman generate systemd command.

Closes: #86

Added `--ignore-infra-conmon-pidfile` and `--ignore-pod-id-file` options
to `podlet generate pod`.

The `--infra-conmon-pidfile` and `--pod-id-file` options are
deliberately unimplemented by `podlet podman pod create` as they are set
by Quadlet when generating the systemd service unit file, and cannot be
set multiple times. Because `podlet generate pod` uses
`podlet podman pod create` internally, this caused an error when
attempting to generate a `.pod` Quadlet file from a pod created using
either of those options, such as those created with the deprecated
`podman generate systemd` command.

Closes: #86
Signed-off-by: Paul Nettleton <k9@k9withabone.dev>
@k9withabone k9withabone added the enhancement New feature or request label Oct 2, 2024
@k9withabone k9withabone added this to the v0.3.1 milestone Oct 2, 2024
@k9withabone
Copy link
Member Author

@rapus95 @strzinek Would you like the test this out to see if it meets your expectations? Do the help and error messages make sense?

@strzinek
Copy link

strzinek commented Oct 2, 2024

@rapus95 @strzinek Would you like the test this out to see if it meets your expectations? Do the help and error messages make sense?

Thanks for the mention, but I don't have a suitable environment for the test right now.

@k9withabone k9withabone merged commit 3f2bd47 into main Oct 6, 2024
13 checks passed
@k9withabone k9withabone deleted the generate-pod-ignore-options branch October 6, 2024 15:24
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add --ignore-infra-conmon-pidfile and --ignore-pod-id-file options to podlet generate pod
2 participants