-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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_image: fix creation of image with data disks #49394
Conversation
Hi @jeking3, thank you for submitting this pull-request! |
@jeking3, just so you are aware we have a dedicated Working Group for azure. |
@jeking3 Thanks for your contribution, When you finished and ready for review, I will test the feature and push to review. Thanks! |
This is ready. Without this change, the call fails. |
shipit |
dbf5887
to
d0752f3
Compare
I have updated the integration test to make an image with a data disk. This was missing, and likely the reason the issue went unnoticed. |
FYI I have a more comprehensive update for this module coming soon, which allows for specification of all the OS and data disk parameters (caching, storage account type, lun, etc..) - now that I know how to update and run the integration test, that PR will include test changes too. |
does this need to be backported to stable-2.7 branch? @yuwzho |
yes, I will backport it |
(cherry picked from commit 79c0e30)
SUMMARY
Creation of images with data disks would always fail, as the lun parameter to ImageDataDisk must be named.
ISSUE TYPE
COMPONENT NAME
azure_rm_image
ADDITIONAL INFORMATION