-
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_virtualmachine: Add 'accept_terms' #44701
Conversation
Hi @spmp, Thank you for the pullrequest, just so you are aware we have a dedicated Working Group for azure. |
The test
|
The test
|
The test
|
Lights are green @jborean93, @zikalino, @mattclay, @nitzmahone |
@spmp Thanks for the contribution, Could you help confirm the PR ready for review or not? If yes, I will push to review. Thanks! |
Yes, ready as 8)
|
@jborean93, @zikalino, @mattclay, @nitzmahone Could you help review this PR? Thanks! |
…deploying paid marketplace images azure_rm_storageaccount: Add 'StandardSSD_LRS', and choices to correct unrelated sanity error
12bb06c
to
6418698
Compare
azure_rm_virtualmachine: Add 'accept_terms' for accepting terms when deploying paid marketplace images
SUMMARY
Paid Azure Marketplace images requiring accepting the terms of the
image
andplan
before being deployed. Usually this takes place in the Azure web frontend or viaaz
cli prior to provisioning.As
azure_rm_virtualmachine
is for programatically deploying VM images it should have the capacity to accept these terms at the time of deployment. Theaccept_terms
(True
) does just this.ISSUE TYPE
COMPONENT NAME
azure_rm_virtualmachine
ANSIBLE VERSION
ADDITIONAL INFORMATION
Tests are included, but commented out as special CI permissions are required.