Skip to content
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

azure_rm_virtualmachine: suppress no_log warning on ssh_password_enabled parameter #622

Merged

Conversation

nbr23
Copy link
Contributor

@nbr23 nbr23 commented Aug 28, 2021

SUMMARY

azure_rm_virtualmachine is issuing a warning about ssh_password_enabled not being set to no_log=true. This patch explicitly sets no_log to false for this benign parameter.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME
  • azure_rm_virtualmachine
ADDITIONAL INFORMATION

The warning being displayed by the module prior to this patch:

[WARNING]: Module did not set no_log for ssh_password_enabled

ssh_password_enabled is only a boolean and does not contain secret information or credentials. Ansible's heuristics are assuming wrong.

@Fred-sun Fred-sun added enhancement New feature or request medium_priority Medium priority ready_for_review The PR has been modified and can be reviewed and merged labels Aug 30, 2021
@xuzhang3
Copy link
Collaborator

LGTM

@xuzhang3 xuzhang3 merged commit 0153e1e into ansible-collections:dev Aug 31, 2021
@nbr23 nbr23 deleted the virtualmachine_ssh_password_enabled branch August 31, 2021 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request medium_priority Medium priority ready_for_review The PR has been modified and can be reviewed and merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants