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.

Sign up for GitHub

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

Jump to bottom

Feat/existing ddos #16

Merged
merged 7 commits into from
Jun 15, 2023
Merged

Feat/existing ddos #16

merged 7 commits into from
Jun 15, 2023

Conversation

13archit
Copy link
Member

what

  • Added support for existing ddos protection plan.
  • Removed flow log resource.

why

  • To link vnet to existing ddos plan(if any)
  • To remove unwanted dependency of vnet module on network security group when flow log for network security group is enabled.

@13archit 13archit requested a review from d4kverma June 15, 2023 11:37
@13archit 13archit self-assigned this Jun 15, 2023
@13archit 13archit added the enhancement New feature or request label Jun 15, 2023
@d4kverma d4kverma merged commit 9f3a38a into master Jun 15, 2023
@delete-merged-branch delete-merged-branch bot deleted the feat/existing_ddos branch June 15, 2023 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@d4kverma d4kverma Awaiting requested review from d4kverma

Assignees

@13archit 13archit

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@13archit @d4kverma