-
Notifications
You must be signed in to change notification settings - Fork 336
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 encryption to azure_rm_galleryimageversion #1311
Add encryption to azure_rm_galleryimageversion #1311
Conversation
Add the parameter `encryption` to `target_regions` of the `azure_rm_galleryimageversion` module, making it possible to create encrypted images. Fixes ansible-collections#1290
9793c2d
to
9637412
Compare
@ephracis Please help fix the error described below, the new parameters also need to be supported by the test case! Thank you very much!
|
Sure thing! I'll start working on it right away. |
Add the usage of disk encryption to all regions in the tests of the module azure_rm_galleryimageversion. This required a fix, moving the deprecated managed_image to storage_profile.source_image instead.
I have now fixed the issues in the module specification, and added integrations tests. |
@@ -0,0 +1,92 @@ | |||
# (c) 2018 Yunge Zhu, <yungez@microsoft.com> |
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.
@ephracis you are the author.
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.
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.
Yes, that's true. It would actually be better to put the file somewhere where it could be shared by both integration tests but this was my first attempt at it.
If you have any suggestions on where to put the file and then share it, let me know and I can fix it as part of this PR.
I'm actually wondering why this is not part of the collection itself, and only part of the integration tests. Seems like a good plugin to have. But perhaps there's a reason you don't want it shipped.
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.
@ephracis Yes, we are considering it as a plugin that can be applied to the entire repo. Thanks!
SUMMARY
Add the parameter
encryption
totarget_regions
of theazure_rm_galleryimageversion
module.Fixes #1290
ISSUE TYPE
COMPONENT NAME
azure_rm_galleryimageversion
ADDITIONAL INFORMATION
This makes it possible to create encrypted images.
Had to bump API version to get access to the new fields.