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

Network Watcher Flow Logs & AVD DCR Bug Fix #1158

Draft
wants to merge 51 commits into
base: main
Choose a base branch
from
Draft

Network Watcher Flow Logs & AVD DCR Bug Fix #1158

wants to merge 51 commits into from

Conversation

jamasten
Copy link
Contributor

@jamasten jamasten commented Mar 7, 2025

Description

  • Enabled network watcher flow logs for either VNETs or NSGs in MLZ and all add-ons integrated with tier3
  • Fixed a DCR bug which ended up being a configuration change that wasn't communicated by PG

Issue reference

The issue this PR will close: #1157, #1137

Checklist

Please make sure you've completed the relevant tasks for this PR out of the following list:

  • All acceptance criteria in the backlog item are met
  • The documentation is updated to cover any new or changed features
  • Manual tests have passed
  • Relevant issues are linked to this PR

jamasten added 30 commits March 4, 2025 21:03
@jamasten
Added & sorted params for flow logs, Removed defender auto provisioni…
8409202 
…ng param, Added resources for flow logs
@jamasten
Removed dependency, Fixed deployment scope
2e96e90
@jamasten
Moved api, Added flow logs, & Renamed section
e9e9017
@jamasten
Added params for flow logs
5d18f62
@jamasten
Added params for flow logs
4bc662d
@jamasten
Added elements for flow logs
cc47f4b
@jamasten
Added params for flow logs
7dbc484
@jamasten
Added param for flow logs
3c64880
@jamasten
Moved api, Added inputs for flow logs
4c0dad7
@jamasten
Added params for flow logs
e86b52c
@jamasten
Fixed formatting
7050018
@jamasten
Added default value for param
4f41fbc
@jamasten
Added missing params
6c6d178
@jamasten
Compiled bicep changes
89d6d6b
@jamasten
Added missing params
bdfdc41
@jamasten
Added params for flow logs
308f032
@jamasten
Compiled bicep changes
04bc309
@jamasten
Added inputs for flow logs & compliance
cb9b413
@jamasten
Added subnet NSG to AVD session host NICs for NIST compliance
4cebfce
@jamasten
Fixed text typos
85c48cf
@jamasten
Fixed input value for param
55aa8e6
@jamasten
Fixed grammar in comments
dcd4717
@jamasten
Refactored flow logs
25cf0d9
@jamasten
Sorted params
df605b1
@jamasten
Updated flow logs section & params, Fixed tags resources
d1c4128
@jamasten
Refactored flow logs
5c1e9c3
@jamasten
Removed location from URLs
f2444dd
@jamasten
Removed location from URL
1000065
@jamasten
Refactored flow logs, Added missing params
70bb8e2
@jamasten
Added tags resource
0acdc55
@jamasten jamasten added user interface bicep Related to Bicep code labels Mar 7, 2025
@jamasten jamasten self-assigned this Mar 7, 2025
@jamasten jamasten linked an issue Mar 7, 2025 that may be closed by this pull request
MLZ is not CMMC Level 3 compliant out of the box #1137
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jamasten jamasten

Labels
bicep Related to Bicep code user interface
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

AVD Add-On: DCR fails to deploy in US Gov Virginia MLZ is not CMMC Level 3 compliant out of the box
1 participant
@jamasten