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

Add auth_mode to azure_rm_storageblob #1315

Merged

Conversation

ephracis
Copy link
Contributor

@ephracis ephracis commented Nov 1, 2023

Add the parameter auth_mode to the
azure_rm_storageblob module, making it possible to create blobs without using access keys.

SUMMARY

Add the parameter auth_mode to the azure_rm_storageblob module.

Fixes #1283, #1255

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

azure_rm_storageblob

ADDITIONAL INFORMATION

This allows customer which has disabled access keys to use the module.

Add the parameter `auth_mode` to the
`azure_rm_storageblob` module, making it possible to create blobs
without using access keys.
@ephracis ephracis force-pushed the 1283-storage-blob-with-login-auth branch from 41b231d to ac33b1b Compare November 2, 2023 08:24
@ephracis ephracis mentioned this pull request Nov 3, 2023
Test using auth_mode='login' for storage blobs.
This has been tested on a machine with MSI.
@ephracis
Copy link
Contributor Author

ephracis commented Nov 3, 2023

Fixed sanity tests and adjusted integration tests.

Not sure how this will work in your pipeline but I have tried it on a server where we are using ANSIBLE_AZURE_AUTH_SOURCE: msi against an Azure account where access keys are forbidden and it works fine, which it doesn't without this change.

Hope that's enough for now. :)

@Fred-sun
Copy link
Collaborator

Fred-sun commented Nov 6, 2023

@ephracis Small change request, line 678 contain unexpected spaces around keyword / parameter equals, Please deleted! Thank you very much!

@Fred-sun Fred-sun added enhancement New feature or request ready_for_review The PR has been modified and can be reviewed and merged medium_priority Medium priority new_feature New feature requirments labels Nov 6, 2023
@xuzhang3
Copy link
Collaborator

xuzhang3 commented Nov 8, 2023

@ephracis LGTM

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 new_feature New feature requirments 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.

Add ability to upload blob without using access keys
3 participants