Skip to content
This repository has been archived by the owner on Jan 8, 2023. It is now read-only.

Allow to additionally configure arbitrary command-line options #107

Merged
merged 2 commits into from
Apr 8, 2022
Merged

Allow to additionally configure arbitrary command-line options #107

merged 2 commits into from
Apr 8, 2022

Conversation

lucendio
Copy link
Contributor

It would be a significant maintenance burden to keep the set of configurable
attributes up-to-date. So, while providing an easy way to set essential
arguments with dedicated attributes, with the proposed change any available
option implemented by Podman can be utilized.

This change is inspired by Ansible's unarchive module1, its attribute
'extra_opts' forwards additional options to the underlying CLI,
regardless of implementation of the CLI.

Footnotes

  1. https://docs.ansible.com/ansible/latest/collections/ansible/builtin/unarchive_module.html#parameter-extra_opts

@ssbarnea ssbarnea added the enhancement This issue/PR relates to a feature request. label Apr 1, 2022
It would be a significant maintenance burden to keep the set of configurable
attributes up-to-date. So, while providing an easy way to set essential
arguments with dedicated attributes, with the proposed change any available
option implemented by Podman can be utilized.

This change is inspired by Ansible's unarchive module [1], its attribute
'extra_opts' forwards additional options to the underlying CLI,
regardless of implementation of the CLI.

[1] https://docs.ansible.com/ansible/latest/collections/ansible/builtin/unarchive_module.html#parameter-extra_opts
@lucendio
Copy link
Contributor Author

lucendio commented Apr 5, 2022

@ssbarnea let me know it there is something I missed to do, or if I can assist to move this forward.

@ssbarnea ssbarnea merged commit 272516d into ansible-community:main Apr 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement This issue/PR relates to a feature request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants