Review Comments

Author: ritwikbasu

ChangeLog.md

@@ -68,9 +68,6 @@
 * Fixed validation logic for Ipv6 IP Prefix to check for correct IPv6 prefix length
 * Added parameter set to get by subnet resource id to **Get-AzVirtualNetworkSubnetConfig** 
 * Updated description of **Location** parameter for **AzNetworkServiceTag**
-* New Generations and SKUs for VirtualNetworkGateways
-  - Introduce new Generations for VirtualNetworkGateways.
-  - Introduce new high throughput SKUs for VirtualNetworkGateways.
 
 #### Az.OperationalInsights
 * Updated documentation for **New-AzOperationalInsightsLinuxSyslogDataSource**:

src/Network/Network/ChangeLog.md

@@ -40,6 +40,9 @@
         - New-VpnConnection
         - Update-VpnConnection
 * Fixed documents for some PowerShell examples to use Az cmdlets instead of AzureRM cmdlets
+* New Generations and SKUs for VirtualNetworkGateways
+  - Introduce new Generations for VirtualNetworkGateways.
+  - Introduce new high throughput SKUs for VirtualNetworkGateways.
 
 ## Version 1.13.0
 * Updated New-AzPrivateLinkServiceIpConfig

src/Network/Network/VirtualNetworkGateway/NewAzureVirtualNetworkGatewayCommand.cs

@@ -267,11 +267,10 @@ public class NewAzureVirtualNetworkGatewayCommand : VirtualNetworkGatewayBaseCmd
         [Parameter(
             Mandatory = false,
             HelpMessage = "The generation for this VirtualNetwork VPN gateway. Must be None if GatewayType is not VPN.")]
-        [ValidateSet(
+        [PSArgumentCompleter(
             MNM.VpnGatewayGeneration.None,
             MNM.VpnGatewayGeneration.Generation1,
             MNM.VpnGatewayGeneration.Generation2)]
-        [PSArgumentCompleter]
         [ValidateNotNullOrEmpty]
         public string VpnGatewayGeneration { get; set; }