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 & Fixes #1158

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

Network Watcher Flow Logs & Fixes #1158

wants to merge 84 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 changed the title Network Watcher Flow Logs & AVD DCR Bug Fix Network Watcher Flow Logs & Fixes Mar 10, 2025
jamasten and others added 23 commits March 11, 2025 15:03
@jamasten
Added logs & metrics, Updated param descriptions & default values, Re…
0fd2ad0 
…moved nsg metrics, Sorted params, Fixed tags
@jamasten
Added default value for Windows VM SKU
16f8f37
GitHub Action: Build Bicep to JSON
92c754c
@jamasten
Flattened array
e19a021
GitHub Action: Build Bicep to JSON
90fffab
@jamasten
Added supported clouds for VNET logs
83c7f5d
GitHub Action: Build Bicep to JSON
0dcff6f
@jamasten
Fixed formatting
4630b91
GitHub Action: Build Bicep to JSON
4e8b9e2
@jamasten
Added network watcher RG to policy assignments
8008809
@jamasten
Merge branch '1137-flow-logs' of https://github.com/Azure/missionlz i…
acfea70 
…nto 1137-flow-logs
GitHub Action: Build Bicep to JSON
5f253b9
@jamasten
Added param, Sorted params
d3ed0b6
@jamasten
Compiled bicep changes
3082a78
@jamasten
Merge branch '1137-flow-logs' of https://github.com/Azure/missionlz i…
93eb7fa 
…nto 1137-flow-logs
@jamasten
Added condition for network watcher RG
b3642bd
GitHub Action: Build Bicep to JSON
17983d9
@jamasten
Added output condition for VM NIC resource IDs
717c146
GitHub Action: Build Bicep to JSON
b666fed
@jamasten
Fixed policy params for members to include & exclude
e428aa9
GitHub Action: Build Bicep to JSON
1483fa2
@jamasten
Moved dynamic policy values to bicep, Added condition for MAC
352baa7
GitHub Action: Build Bicep to JSON
2891e80
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