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
Add ansible cli options --ask-vault-password and --vault-pass-file #63782
Conversation
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.
Rather than adding additional calls to add_argument()
, just add additional parameters to the existing add_argument()
calls.
Also, please create a changelog fragment and add integration tests to test/integration/targets/vault
. See this fragment as an example.
@@ -365,5 +365,9 @@ def add_vault_options(parser): | |||
base_group = parser.add_mutually_exclusive_group() | |||
base_group.add_argument('--ask-vault-pass', default=C.DEFAULT_ASK_VAULT_PASS, dest='ask_vault_pass', action='store_true', |
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.
base_group.add_argument('--ask-vault-pass', default=C.DEFAULT_ASK_VAULT_PASS, dest='ask_vault_pass', action='store_true', | |
base_group.add_argument('--ask-vault-pass', '--ask-vault-password', default=C.DEFAULT_ASK_VAULT_PASS, dest='ask_vault_pass', action='store_true', |
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.
Please add integration tests.
@ivog74 Are you still working on this? Thanks. |
@ivog74 this PR contains the following merge commits: Please rebase your branch to remove these commits. |
…-file' to the existing calls to add_argument
…rd and --vault-password-file by default
…vault-password and --vault-pass-file
SUMMARY
The ansible-playbook command provides 2 options to specify a vault password:
--ask-vault-pass
--vault-password-file
One option use 'pass', the other uses 'password'. I propose to use either pass or password for both option. For now I added the following options:
-- ask-vault-password
--vault-pass-file
ISSUE TYPE
COMPONENT NAME
ansible cli
ADDITIONAL INFORMATION
before:
after:
or