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

Conditionally call MSI auth when interacting with keyvault #356

Merged

Conversation

opfpqgoon
Copy link
Contributor

SUMMARY

MSI auth shouldn't be used unless the module specifically calls for it. Currently MSI will be tried anytime the modules are called from an Azure VM, even if the MSI doesn't (and shouldn't) have access to the keyvault.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

azure_rm_keyvaultsecret
azure_rm_keyvaultsecret_info
azure_rm_keyvaultkey
azure_rm_keyvaultkey_info

ADDITIONAL INFORMATION

@opfpqgoon opfpqgoon changed the title Conditionally call MSI auth Conditionally call MSI auth when interacting with keyvault Dec 8, 2020
@Fred-sun
Copy link
Collaborator

Fred-sun commented Feb 5, 2021

@opfpqgoon

ERROR: plugins/modules/azure_rm_keyvaultkey.py:185:9: E115: expected an indented block (comment)
ERROR: plugins/modules/azure_rm_keyvaultkey.py:185:9: E265: block comment should start with '# '
ERROR: plugins/modules/azure_rm_keyvaultkey_info.py:320:9: E115: expected an indented block (comment)
ERROR: plugins/modules/azure_rm_keyvaultkey_info.py:320:9: E265: block comment should start with '# '
ERROR: plugins/modules/azure_rm_keyvaultsecret.py:184:9: E115: expected an indented block (comment)
ERROR: plugins/modules/azure_rm_keyvaultsecret.py:184:9: E265: block comment should start with '# '
ERROR: plugins/modules/azure_rm_keyvaultsecret_info.py:274:9: E115: expected an indented block (comment)
ERROR: plugins/modules/azure_rm_keyvaultsecret_info.py:274:9: E265: block comment should start with '# '

@Fred-sun
Copy link
Collaborator

Fred-sun commented Feb 7, 2021

@opfpqgoon Can I make some updates to some of the Sanity errors above? Thank you very much!

@opfpqgoon
Copy link
Contributor Author

opfpqgoon commented Feb 7, 2021 via email

@Fred-sun Fred-sun added the work in In trying to solve, or in working with contributors label Feb 8, 2021
@Fred-sun Fred-sun added ready_for_review The PR has been modified and can be reviewed and merged and removed work in In trying to solve, or in working with contributors labels Feb 8, 2021
@Fred-sun
Copy link
Collaborator

Fred-sun commented Feb 8, 2021

@haiyuazhang

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

None yet

3 participants