Skip to content

Commit c9c36a4

Browse files
authored
Update upgrade-basic-standard-with-powershell.md
The -validateScenarioOnly parameter is a SWITCH, defaulted to FALSE. The documentation page has an example of how to use this, but it's missing the $true at the end. I have proposed this correction (line 120 in the file). If a user just copies and pastes from the documentation (which arguably is what the documentation is FOR), expecting to validate the scenario, they will instead go ahead with migrations which can cause potential issues - not to mention any issues/troubles that may rise internally from an engineer unintentionally upgrading the load balancer.
1 parent 7e84584 commit c9c36a4

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

articles/load-balancer/upgrade-basic-standard-with-powershell.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ Install-Module -Name AzureBasicLoadBalancerUpgrade -Scope CurrentUser -Repositor
117117
Validate that a Basic Load Balancer is supported for upgrade
118118
119119
```powershell
120-
Start-AzBasicLoadBalancerUpgrade -ResourceGroupName <loadBalancerRGName> -BasicLoadBalancerName <basicLBName> -validateScenarioOnly
120+
Start-AzBasicLoadBalancerUpgrade -ResourceGroupName <loadBalancerRGName> -BasicLoadBalancerName <basicLBName> -validateScenarioOnly $true
121121
```
122122

123123
### Example: upgrade by name
@@ -315,4 +315,4 @@ If your failed migration was targeting multiple load balancers at the same time,
315315
## Next steps
316316

317317
- [If skipped, migrate from using NAT Pools to NAT Rules for Virtual Machine Scale Sets](load-balancer-nat-pool-migration.md)
318-
- [Learn about Azure Load Balancer](load-balancer-overview.md)
318+
- [Learn about Azure Load Balancer](load-balancer-overview.md)

0 commit comments

Comments
 (0)