-
Notifications
You must be signed in to change notification settings - Fork 335
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_ddosprotectionplan #493
Conversation
@ikarldasan Thank for your contribution! Please help to change the module name to azure_rm_ddosprotectionplan*, which is consistent with other modules, thank you! |
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.
@ikarldasan There is no content in " tests/integration/targets/azure_rm_ddos_protection_plan/aliases"! Please share the result of you execute "tests/integration/targets/azure_rm_ddos_protection_plan/tasks/main.yml"!
cloud/azure
shippable/azure/group10
destructive
- absent | ||
- present | ||
extends_documentation_fragment: | ||
- azure.azcollection.azure |
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.
- azure.azcollection.azure | |
- azure.azcollection.azure | |
- azure.azcollection.azure_tags |
default: present | ||
choices: | ||
- absent | ||
- present |
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.
Please add log_path and log_mode parameter to this module! Thank you very much!
log_path:
description:
- parent argument.
type: str
log_mode:
description:
- parent argument.
type: str
@Fred-sun , have addressed the changes suggested. ansible-test integration azure_rm_ddosprotectionplan --allow-destructive -e -vvv
Can you please help understand what I may be missing. |
@Fred-sun here are the test results, run locally. Running the integration test as playbooks using
|
* class are worngly named. fixed. * fixing sanity errors.
@ikarldasan Please add following line to ignore2.11.txt, 2.9.txt, 2.10.txt and 2.12.txt. Thank you very much!
|
2281f46
to
8dfc8ed
Compare
Co-authored-by: Fred-sun <[email protected]>
Co-authored-by: Fred-sun <[email protected]>
Co-authored-by: Fred-sun <[email protected]>
…in.yml Co-authored-by: Fred-sun <[email protected]>
…in.yml Co-authored-by: Fred-sun <[email protected]>
@Fred-sun have addressed the changes suggested. |
@ikarldasan Please help to delete all ignores on ignore2.11.txt, ignore2.9.txt, ignore2.10.txt and ignore2.12.txt that you add. It has been fixed by #540! |
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.
@ikarldasan One last update requirement, Thank you very much!
Co-authored-by: Fred-sun <[email protected]>
Co-authored-by: Fred-sun <[email protected]>
@Fred-sun Have updated the changes suggested. Thank you |
@Fred-sun Do we need any more enhancements? Can we merge this? Thank you |
@ikarldasan I will move forward with the merger as soon as possible. Thank you very much! |
* initial commit * add ddos protection plan operations * enhance create flow * enhance operations * enhance get and delete operations * address sanity checks * add integration tests * add log parameters and aliases * rename module for consistency * update pr-pipeline for consistency * 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. * updating ignore text * Update plugins/modules/azure_rm_ddosprotectionplan.py Co-authored-by: Fred-sun <[email protected]> * Update plugins/modules/azure_rm_ddosprotectionplan.py Co-authored-by: Fred-sun <[email protected]> * Update plugins/modules/azure_rm_ddosprotectionplan.py Co-authored-by: Fred-sun <[email protected]> * Update tests/integration/targets/azure_rm_ddosprotectionplan/tasks/main.yml Co-authored-by: Fred-sun <[email protected]> * Update tests/integration/targets/azure_rm_ddosprotectionplan/tasks/main.yml Co-authored-by: Fred-sun <[email protected]> * Update main.yml * delete ignores * remove ignores * Update plugins/modules/azure_rm_ddosprotectionplan.py Co-authored-by: Fred-sun <[email protected]> * Update plugins/modules/azure_rm_ddosprotectionplan_info.py Co-authored-by: Fred-sun <[email protected]> Co-authored-by: haiyuan_zhang <[email protected]> Co-authored-by: Fred-sun <[email protected]>
SUMMARY
This module introduces a wide variety of configurations for creating a DDoS protection plan on Azure.
Create DDoS protection plan on Azure.
Update DDoS protection plan on Azure.
Delete DDoS protection plan on Azure.
Get DDoS protection plan on Azure.
List DDoS protection plan in Azure Resource Group.
List DDoS protection plan in Azure Subscription.
Fixes #492
ISSUE TYPE
COMPONENT NAME
azure_rm_ddos_protection_plan
ADDITIONAL INFORMATION
This module allows user to create/update or delete the DDoS Protection Plan.
It also provides all the parameters given by: https://docs.microsoft.com/en-us/azure/ddos-protection/ddos-protection-overview
More information