-
Notifications
You must be signed in to change notification settings - Fork 517
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
Replaced DDoS Standard with DDoS Network Protection #396
Conversation
Hey lachaves, thanks for creating this PR. We noticed that this PR is ammending some things related to policies (definitions or assignments) 🔍 Thanks for this however, please keep in mind we replicate all policies from our upstream repo https://github.com/Azure/Enterprise-Scale (via some GitHub Actions) into these Bicep modules. So it may be that this PR may be closed and we ask you to re-submit the changes to the respective code in our other repo. However, we will review and advise further anyway but this is just a friendly premptive message and reminder of how policies are used and updated in this repo. Thanks from your friendly msft-bot. |
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 @lachaves. Can you rollback/not make changes to these files please as we will pull these in from upstream ESLZ repo:
infra-as-code/bicep/modules/policy/assignments/lib/china/policy_assignments/policy_assignment_es_enable_ddos_vnet.tmpl.json
infra-as-code/bicep/modules/policy/assignments/lib/policy_assignments/policy_assignment_es_enable_ddos_vnet.tmpl.json
Rolled Back the changes to policy assigment DDoS json files for China
Rolled back changes for DDoS in Polci Assignment json file
@jtracey93 I believe it should now be ready for merging as I rolled back the changes in both files. Please check and let me know if still missing something. |
/azp run validateazcloud |
Azure Pipelines successfully started running 1 pipeline(s). |
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.
LGTM
Overview/Summary
As per ADO User Story 24459, performing a replacement of "DDoS Standard" with "DDoS Network Protection" in all the ALZ-Bicep repo.
This PR fixes/adds/changes/removes
Breaking Changes
NA
Testing Evidence
NA
As part of this Pull Request I have
.bicep
file/s I am adding/editing are using the latest API version possiblemain
branch