Commit 7ee6dd7
feat(finops-hub): PowerShell parity for networkMode + changelog
Adds -NetworkMode parameter to Deploy-FinOpsHub and Deploy-Hub so the
new tri-state network model (public|vnet|private) introduced for the
Sept 2025 implicit-outbound retirement is reachable from PowerShell,
not just the portal UI. Addresses the CHANGES_REQUESTED review on #2163.
Deploy-FinOpsHub (cmdlet):
- Add -NetworkMode <public|vnet|private> with ValidateSet
- Replace stale .PARAMETER EnablePublicAccess help block with accurate
.PARAMETER NetworkMode + .PARAMETER DisablePublicAccess (deprecated)
- Version-guard the networkMode template parameter for v15.0+
- When -NetworkMode is supplied on v15+, skip enablePublicAccess so the
two parameters never go on the wire together (bicep effectiveNetworkMode
already lets networkMode win; sending both clutters deployment history)
Deploy-Hub.ps1 (dev script):
- Add -NetworkMode <public|vnet|private> with ValidateSet
- Keep -Private switch as deprecated back-compat alias
- -NetworkMode wins when both are supplied
infrastructure.bicep:
- Add comment noting that in vnet mode all subnets get
defaultOutboundAccess:false with no NAT attached -- intentionally
egress-less scaffolding for future private-mode upgrades
changelog.md:
- Document the new networkMode parameter under FinOps hubs v15 Added
- Add upgrade-impact note under Changed flagging that existing
enablePublicAccess=false deployments will gain a NAT Gateway and
Public IP on next deploy (NAT hours + PIP + data processing charges)
- Bump ms.date to 06/03/2026 per docs-mslearn rule
Verified end-to-end via az deployment group what-if against the built
template for all three modes:
public: no VNet, no NSG, no NAT (correct)
vnet: VNet + NSG only, no NAT (correct)
private: VNet + NSG + NAT Gateway + Public IP + PE + DNS zones (correct)
Refs #2161 #2163
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>1 parent aff09d7 commit 7ee6dd7
4 files changed
Lines changed: 55 additions & 11 deletions
File tree
- docs-mslearn/toolkit
- src
- powershell/Public
- scripts
- templates/finops-hub/modules/Microsoft.FinOpsHubs/Core
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3 | 3 | | |
4 | 4 | | |
5 | 5 | | |
6 | | - | |
| 6 | + | |
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
| |||
47 | 47 | | |
48 | 48 | | |
49 | 49 | | |
| 50 | + | |
| 51 | + | |
50 | 52 | | |
51 | 53 | | |
| 54 | + | |
52 | 55 | | |
53 | 56 | | |
54 | 57 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
84 | 84 | | |
85 | 85 | | |
86 | 86 | | |
87 | | - | |
88 | | - | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
89 | 92 | | |
90 | 93 | | |
91 | 94 | | |
| |||
187 | 190 | | |
188 | 191 | | |
189 | 192 | | |
| 193 | + | |
| 194 | + | |
| 195 | + | |
| 196 | + | |
| 197 | + | |
190 | 198 | | |
191 | 199 | | |
192 | 200 | | |
| |||
281 | 289 | | |
282 | 290 | | |
283 | 291 | | |
284 | | - | |
| 292 | + | |
| 293 | + | |
| 294 | + | |
| 295 | + | |
| 296 | + | |
| 297 | + | |
| 298 | + | |
285 | 299 | | |
286 | 300 | | |
287 | 301 | | |
| |||
299 | 313 | | |
300 | 314 | | |
301 | 315 | | |
| 316 | + | |
| 317 | + | |
| 318 | + | |
| 319 | + | |
| 320 | + | |
302 | 321 | | |
303 | 322 | | |
304 | 323 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
88 | 88 | | |
89 | 89 | | |
90 | 90 | | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
91 | 94 | | |
92 | | - | |
| 95 | + | |
93 | 96 | | |
94 | 97 | | |
95 | | - | |
| 98 | + | |
96 | 99 | | |
97 | 100 | | |
98 | 101 | | |
| |||
116 | 119 | | |
117 | 120 | | |
118 | 121 | | |
| 122 | + | |
| 123 | + | |
119 | 124 | | |
120 | 125 | | |
121 | 126 | | |
| |||
283 | 288 | | |
284 | 289 | | |
285 | 290 | | |
286 | | - | |
287 | | - | |
288 | | - | |
| 291 | + | |
| 292 | + | |
| 293 | + | |
| 294 | + | |
289 | 295 | | |
290 | | - | |
| 296 | + | |
291 | 297 | | |
292 | 298 | | |
293 | 299 | | |
294 | 300 | | |
295 | | - | |
| 301 | + | |
| 302 | + | |
| 303 | + | |
| 304 | + | |
| 305 | + | |
| 306 | + | |
| 307 | + | |
| 308 | + | |
| 309 | + | |
| 310 | + | |
| 311 | + | |
| 312 | + | |
| 313 | + | |
| 314 | + | |
296 | 315 | | |
297 | 316 | | |
298 | 317 | | |
| |||
Lines changed: 3 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
28 | 31 | | |
29 | 32 | | |
30 | 33 | | |
| |||
0 commit comments