Skip to content
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

Merged
merged 4 commits into from
Nov 24, 2022

Conversation

lachaves
Copy link
Contributor

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

  1. Closes Issue Replace DDoS Standard with DDoS Network Protection #395

Breaking Changes

NA

Testing Evidence

NA

As part of this Pull Request I have

@ghost
Copy link

ghost commented Nov 22, 2022

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.

@ghost ghost added Area: Policy Needs: Triage 🔍 Needs triaging by the team labels Nov 22, 2022
Copy link
Collaborator

@jtracey93 jtracey93 left a 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:

  1. infra-as-code/bicep/modules/policy/assignments/lib/china/policy_assignments/policy_assignment_es_enable_ddos_vnet.tmpl.json
  2. 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
@ghost ghost removed the Needs: Author Feedback label Nov 24, 2022
Rolled back changes for DDoS in Polci Assignment json file
@lachaves
Copy link
Contributor Author

@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.

@lachaves lachaves requested a review from jtracey93 November 24, 2022 16:38
@jtracey93
Copy link
Collaborator

/azp run validateazcloud

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Collaborator

@jtracey93 jtracey93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jtracey93 jtracey93 merged commit 53c6b0f into Azure:main Nov 24, 2022
@jtracey93 jtracey93 linked an issue Nov 24, 2022 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs: Triage 🔍 Needs triaging by the team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace DDoS Standard with DDoS Network Protection
2 participants