-
Notifications
You must be signed in to change notification settings - Fork 334
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
New module: azure_rm_expressroute #484
Conversation
@praveenghuge Thank for your contribute! I will review and push for merge! |
@Fred-sun can you please review |
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.
@Fred-sun can you please review
@praveenghuge Thanks for your contribution! I working in this!
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.
@praveenghuge Would you please share the result of you execute the playbook "tests/integration/targets/azure_rm_expressroute/tasks/main.yml"? It will helpful to push merge this PR! Thank you very much!
playbook
output
|
playbook
output
|
playbook
output
|
playbook
output
|
playbook
output
|
playbook
output
|
playbook
output
|
playbook
output
|
Hi @Fred-sun can you please review the PR. addressed the changes |
I'm working on it! |
Hi @Fred-sun can you please review the PR. addressed the changes |
@Fred-sun anything pending from our side. Thanks you so much 👍 |
@praveenghuge It is not pending, the new version will be released temporarily, and the PR will be merged after the new version is released. Thank you very much! |
@Fred-sun updated the latest version |
@praveenghuge Greate! Waiting merged! |
LGTM |
* first commit * fix file * new changes * working create/update/delete/get * change username * minor changes * Fix sanity issues * Improve integration test * remove return * address changes * changes addressed * address changes * address changes * small change * fixing update account_enabled bug in azure_rm_aduser.py (ansible-collections#524) * fixing ad related auth issue when using service pricinpal. (ansible-collections#525) * change class name of azure_rm_aduser (ansible-collections#526) * class are worngly named. fixed. * fixing sanity errors. * address cahnges * new changes * new changes * addrss changes * address changes * address changes * address changes * merge upstream changes * address changes * update version Co-authored-by: root <[email protected]> Co-authored-by: root <[email protected]> Co-authored-by: Praveen Ghuge <[email protected]> Co-authored-by: Praveen Ghuge <[email protected]> Co-authored-by: Karl Dasan <[email protected]> Co-authored-by: haiyuan_zhang <[email protected]>
SUMMARY
This module introduces a wide variety of configurations for creating an express route on Azure.
Fixes #475
ISSUE TYPE
COMPONENT NAME
Additional Support:
for express route peering support we need to create resource in ansible https://docs.microsoft.com/en-us/azure/expressroute/expressroute-introduction
created issue #483
ADDITIONAL INFORMATION
This module allows user to create/update or delete the express route. It also provides all the parameters given by https://docs.microsoft.com/en-us/azure/expressroute/expressroute-introduction
More information: