microsoft / microsoft/AzureTRE
Simplify NSGs applied to workspace VNets by removing peer from virtual network tag
@marrobi is already working on this.
Since Jan 30, 2023.
- Dominant language
- Python
- Stars
- 235
- Forks
- 192
- Avg merge
- 1d 23h
- Merged PRs (30d)
- 13
Description
At present the workspace to core vnet peer and core to workspace net peer has this setting set to Allow:

This makes the core VNet part of the VirtualNetwork tag used by network security groups. This means within NSGs we need to keep track of workspace address spaces, and if the VirtualNetwork is used inadvertently it also allows access to the core network.
As we have explicit allow and deny rules on the NSGs changing this setting to Block all traffic to the remote virtual network should have no adverse affect. However this should be verified.
Once disabled NSG configurations can revert to using the VirtualNetwork tag which leads to easier to read definitions and simplifies Terraform.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.