-
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
fixed boolean value for zone_redundant in azure_rm_sqldatabase.py #41974
Conversation
I can confirm that the proposed solution fixes the problem. |
@vpopescu thanks for the contribution. could you pls modify one of existing test case to cover this paramter? thanks. |
kindly ping |
@vpopescu Thanks for you contribute to open this PR. Could you help update the PR status? ready for review or need new change? Thanks! |
@zikalino Please help to review this PR. Thanks! |
2 similar comments
@zikalino Please help to review this PR. Thanks! |
@zikalino Please help to review this PR. Thanks! |
SUMMARY
Per https://docs.microsoft.com/en-us/azure/templates/microsoft.sql/servers/databases, the zone_redundant field is a boolean, but it's passed in as a string, causing a deserialization error while trying to create a sql database in azure.
Fixes #41742
ISSUE TYPE
COMPONENT NAME
azure_rm_sqldatabase.py
ANSIBLE VERSION
ADDITIONAL INFORMATION
Please review the change carefully, I'm not sure why maintainer cannot replicate the issue.