[Network] az network vnet peering: Refine command interface of subnet peering (Azure#29817)

* chore: refine command interface

* fix: resolve linter issue

* test: add case and recording

Author: necusjz

linter_exclusions.yml

@@ -2754,6 +2754,14 @@ network vnet peering create:
     allow_gateway_transit:
       rule_exclusions:
       - option_length_too_long
+network vnet peering update:
+  parameters:
+    allow_forwarded_traffic:
+      rule_exclusions:
+      - option_length_too_long
+    allow_gateway_transit:
+      rule_exclusions:
+      - option_length_too_long
 network vnet subnet create:
   parameters:
     service_endpoint_policy:

src/azure-cli/azure/cli/command_modules/network/aaz/latest/network/vnet/peering/_create.py

@@ -91,6 +91,7 @@ def _build_arguments_schema(cls, *args, **kwargs):
         _args_schema.peer_complete_vnets = AAZBoolArg(
             options=["--peer-complete-vnets"],
             help="Whether complete virtual network address space is peered.",
+            default=True,
         )
         _args_schema.remote_subnet_names = AAZListArg(
             options=["--remote-subnet-names"],

src/azure-cli/azure/cli/command_modules/network/aaz/latest/network/vnet/peering/_update.py

@@ -15,7 +15,7 @@
     "network vnet peering update",
 )
 class Update(AAZCommand):
-    """Update a peering.
+    """Update a peering in the specified virtual network.
 
     :example: Change forwarded traffic configuration of a virtual network peering.
         az network vnet peering update -g MyResourceGroup -n MyVnet1ToMyVnet2 --vnet-name MyVnet1 --set allowForwardedTraffic=true
@@ -71,6 +71,66 @@ def _build_arguments_schema(cls, *args, **kwargs):
             required=True,
             id_part="child_name_1",
         )
+        _args_schema.sync_remote = AAZStrArg(
+            options=["--sync-remote"],
+            help="Parameter indicates the intention to sync the peering with the current address space on the remote vNet after it's updated.",
+            enum={"true": "true"},
+        )
+        _args_schema.allow_forwarded_traffic = AAZBoolArg(
+            options=["--allow-forwarded-traffic"],
+            help="Allows forwarded traffic from the local VNet to the remote VNet.",
+            nullable=True,
+        )
+        _args_schema.allow_gateway_transit = AAZBoolArg(
+            options=["--allow-gateway-transit"],
+            help="Allows gateway link to be used in the remote VNet.",
+            nullable=True,
+        )
+        _args_schema.allow_vnet_access = AAZBoolArg(
+            options=["--allow-vnet-access"],
+            help="Allows access from the local VNet to the remote VNet.",
+            nullable=True,
+        )
+        _args_schema.enable_only_ipv6 = AAZBoolArg(
+            options=["--enable-only-ipv6"],
+            help="Whether only Ipv6 address space is peered for subnet peering.",
+            nullable=True,
+        )
+        _args_schema.local_subnet_names = AAZListArg(
+            options=["--local-subnet-names"],
+            help="List of local subnet names that are subnet peered with remote virtual network.",
+            nullable=True,
+        )
+        _args_schema.peer_complete_vnets = AAZBoolArg(
+            options=["--peer-complete-vnets"],
+            help="Whether complete virtual network address space is peered.",
+            nullable=True,
+        )
+        _args_schema.remote_subnet_names = AAZListArg(
+            options=["--remote-subnet-names"],
+            help="List of remote subnet names from remote virtual network that are subnet peered.",
+            nullable=True,
+        )
+        _args_schema.remote_vnet = AAZStrArg(
+            options=["--remote-vnet"],
+            help="Name or ID of the remote VNet.",
+            nullable=True,
+        )
+        _args_schema.use_remote_gateways = AAZBoolArg(
+            options=["--use-remote-gateways"],
+            help="Allows VNet to use the remote VNet's gateway. Remote VNet gateway must have --allow-gateway-transit enabled for remote peering. Only 1 peering can have this flag enabled. Cannot be set if the VNet already has a gateway.",
+            nullable=True,
+        )
+
+        local_subnet_names = cls._args_schema.local_subnet_names
+        local_subnet_names.Element = AAZStrArg(
+            nullable=True,
+        )
+
+        remote_subnet_names = cls._args_schema.remote_subnet_names
+        remote_subnet_names.Element = AAZStrArg(
+            nullable=True,
+        )
 
         # define Arg Group "Properties"
 
@@ -287,6 +347,9 @@ def url_parameters(self):
         @property
         def query_parameters(self):
             parameters = {
+                **self.serialize_query_param(
+                    "syncRemoteAddressSpace", self.ctx.args.sync_remote,
+                ),
                 **self.serialize_query_param(
                     "api-version", "2023-11-01",
                     required=True,
@@ -351,7 +414,27 @@ def _update_instance(self, instance):
 
             properties = _builder.get(".properties")
             if properties is not None:
+                properties.set_prop("allowForwardedTraffic", AAZBoolType, ".allow_forwarded_traffic")
+                properties.set_prop("allowGatewayTransit", AAZBoolType, ".allow_gateway_transit")
+                properties.set_prop("allowVirtualNetworkAccess", AAZBoolType, ".allow_vnet_access")
+                properties.set_prop("enableOnlyIPv6Peering", AAZBoolType, ".enable_only_ipv6")
+                properties.set_prop("localSubnetNames", AAZListType, ".local_subnet_names")
+                properties.set_prop("peerCompleteVnets", AAZBoolType, ".peer_complete_vnets")
+                properties.set_prop("remoteSubnetNames", AAZListType, ".remote_subnet_names")
                 properties.set_prop("remoteVirtualNetwork", AAZObjectType)
+                properties.set_prop("useRemoteGateways", AAZBoolType, ".use_remote_gateways")
+
+            local_subnet_names = _builder.get(".properties.localSubnetNames")
+            if local_subnet_names is not None:
+                local_subnet_names.set_elements(AAZStrType, ".")
+
+            remote_subnet_names = _builder.get(".properties.remoteSubnetNames")
+            if remote_subnet_names is not None:
+                remote_subnet_names.set_elements(AAZStrType, ".")
+
+            remote_virtual_network = _builder.get(".properties.remoteVirtualNetwork")
+            if remote_virtual_network is not None:
+                remote_virtual_network.set_prop("id", AAZStrType, ".remote_vnet")
 
             return _instance_value