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
Update sl_vm.py #43966
Update sl_vm.py #43966
Conversation
Updated the list of datacenters to include new datacenters. +label: docsite_pr
The test
|
The test
The test
|
@mcltn what would be the timeframe to get this merged? |
@mcltn Hi, could you please review this. Thanks in advance |
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.
Thanks for adding these. We should look into getting this list form the API as well for future datacenters.
* Update sl_vm.py Updated the list of datacenters to include new datacenters. (cherry picked from commit d1688ad)
@bankabanka Thank you for this PR, your first I believe. It does take us a while to get these review and merged, though that's now been done, thanks to @mcltn Backport so we updates the docs in 2.7 will be done via #46109 Thanks again! |
Updated the list of datacenters to include new datacenters.
+label: docsite_pr
SUMMARY
ISSUE TYPE
COMPONENT NAME
lib/ansible/modules/cloud/softlayer/sl_vm.py
ANSIBLE VERSION