[AutoRelease] t2-avs-2024-09-11-70074(can only be merged by SDK owner) (Azure#37287)

* code and test

* update-testcase

* Update package-lock.json

* Update package.json

---------

Co-authored-by: azure-sdk <PythonSdkPipelines>
Co-authored-by: ChenxiJiang333 <[email protected]>
Co-authored-by: ChenxiJiang333 <[email protected]>

Author: azure-sdk

sdk/compute/azure-mgmt-avs/CHANGELOG.md

@@ -1,5 +1,95 @@
 # Release History
 
+## 9.0.0 (2024-09-22)
+
+### Features Added
+
+  - Added operation WorkloadNetworksOperations.list_public_ips
+  - Added operation group IscsiPathsOperations
+  - Model Addon has a new parameter system_data
+  - Model CloudLink has a new parameter provisioning_state
+  - Model CloudLink has a new parameter system_data
+  - Model Cluster has a new parameter system_data
+  - Model Cluster has a new parameter vsan_datastore_name
+  - Model ClusterUpdate has a new parameter sku
+  - Model Datastore has a new parameter elastic_san_volume
+  - Model Datastore has a new parameter system_data
+  - Model Endpoints has a new parameter hcx_cloud_manager_ip
+  - Model Endpoints has a new parameter nsxt_manager_ip
+  - Model Endpoints has a new parameter vcenter_ip
+  - Model ExpressRouteAuthorization has a new parameter system_data
+  - Model GlobalReachConnection has a new parameter system_data
+  - Model HcxEnterpriseSite has a new parameter provisioning_state
+  - Model HcxEnterpriseSite has a new parameter system_data
+  - Model ManagementCluster has a new parameter vsan_datastore_name
+  - Model Operation has a new parameter action_type
+  - Model PlacementPolicy has a new parameter system_data
+  - Model PrivateCloud has a new parameter dns_zone_type
+  - Model PrivateCloud has a new parameter system_data
+  - Model PrivateCloud has a new parameter virtual_network_id
+  - Model PrivateCloudUpdate has a new parameter dns_zone_type
+  - Model PrivateCloudUpdate has a new parameter sku
+  - Model ProxyResource has a new parameter system_data
+  - Model Resource has a new parameter system_data
+  - Model ScriptCmdlet has a new parameter audience
+  - Model ScriptCmdlet has a new parameter provisioning_state
+  - Model ScriptCmdlet has a new parameter system_data
+  - Model ScriptExecution has a new parameter system_data
+  - Model ScriptPackage has a new parameter provisioning_state
+  - Model ScriptPackage has a new parameter system_data
+  - Model Sku has a new parameter capacity
+  - Model Sku has a new parameter family
+  - Model Sku has a new parameter size
+  - Model Sku has a new parameter tier
+  - Model TrackedResource has a new parameter system_data
+  - Model VirtualMachine has a new parameter provisioning_state
+  - Model VirtualMachine has a new parameter system_data
+  - Model WorkloadNetwork has a new parameter provisioning_state
+  - Model WorkloadNetwork has a new parameter system_data
+  - Model WorkloadNetworkDhcp has a new parameter system_data
+  - Model WorkloadNetworkDnsService has a new parameter system_data
+  - Model WorkloadNetworkDnsZone has a new parameter system_data
+  - Model WorkloadNetworkGateway has a new parameter provisioning_state
+  - Model WorkloadNetworkGateway has a new parameter system_data
+  - Model WorkloadNetworkPortMirroring has a new parameter system_data
+  - Model WorkloadNetworkPublicIP has a new parameter system_data
+  - Model WorkloadNetworkSegment has a new parameter system_data
+  - Model WorkloadNetworkVMGroup has a new parameter system_data
+  - Model WorkloadNetworkVirtualMachine has a new parameter provisioning_state
+  - Model WorkloadNetworkVirtualMachine has a new parameter system_data
+
+### Breaking Changes
+
+  - Model Operation no longer has parameter properties
+  - Operation WorkloadNetworksOperations.get no longer has parameter workload_network_name
+  - Parameter location of model PrivateCloud is now required
+  - Parameter location of model TrackedResource is now required
+  - Parameter type of model PrivateCloudIdentity is now required
+  - Parameter value of model AddonList is now required
+  - Parameter value of model CloudLinkList is now required
+  - Parameter value of model ClusterList is now required
+  - Parameter value of model DatastoreList is now required
+  - Parameter value of model ExpressRouteAuthorizationList is now required
+  - Parameter value of model GlobalReachConnectionList is now required
+  - Parameter value of model HcxEnterpriseSiteList is now required
+  - Parameter value of model PlacementPoliciesList is now required
+  - Parameter value of model PrivateCloudList is now required
+  - Parameter value of model ScriptCmdletsList is now required
+  - Parameter value of model ScriptExecutionsList is now required
+  - Parameter value of model ScriptPackagesList is now required
+  - Parameter value of model VirtualMachinesList is now required
+  - Parameter value of model WorkloadNetworkDhcpList is now required
+  - Parameter value of model WorkloadNetworkDnsServicesList is now required
+  - Parameter value of model WorkloadNetworkDnsZonesList is now required
+  - Parameter value of model WorkloadNetworkGatewayList is now required
+  - Parameter value of model WorkloadNetworkList is now required
+  - Parameter value of model WorkloadNetworkPortMirroringList is now required
+  - Parameter value of model WorkloadNetworkPublicIPsList is now required
+  - Parameter value of model WorkloadNetworkSegmentsList is now required
+  - Parameter value of model WorkloadNetworkVMGroupsList is now required
+  - Parameter value of model WorkloadNetworkVirtualMachinesList is now required
+  - Removed operation WorkloadNetworksOperations.list_public_i_ps
+
 ## 9.0.0b1 (2024-06-28)
 
 ### Features Added

sdk/compute/azure-mgmt-avs/_meta.json

@@ -1,11 +1,11 @@
 {
-  "commit": "cc92c5922e1d69bab8f0deb8e1f778f6d9455739",
+  "commit": "3519c80fe510a268f6e59a29ccac8a53fdec15b6",
   "repository_url": "https://github.com/Azure/azure-rest-api-specs",
   "autorest": "3.10.2",
   "use": [
-    "@autorest/python@6.13.19",
+    "@autorest/python@6.19.0",
     "@autorest/[email protected]"
   ],
-  "autorest_command": "autorest specification/vmware/resource-manager/readme.md --generate-sample=True --include-x-ms-examples-original-file=True --python --python-sdks-folder=/home/vsts/work/1/azure-sdk-for-python/sdk --use=@autorest/python@6.13.19 --use=@autorest/[email protected] --version=3.10.2 --version-tolerant=False",
+  "autorest_command": "autorest specification/vmware/resource-manager/readme.md --generate-sample=True --generate-test=True --include-x-ms-examples-original-file=True --python --python-sdks-folder=/home/vsts/work/1/azure-sdk-for-python/sdk --use=@autorest/python@6.19.0 --use=@autorest/[email protected] --version=3.10.2 --version-tolerant=False",
   "readme": "specification/vmware/resource-manager/readme.md"
 }

sdk/compute/azure-mgmt-avs/assets.json

@@ -8,6 +8,7 @@
 
 from copy import deepcopy
 from typing import Any, TYPE_CHECKING
+from typing_extensions import Self
 
 from azure.core.pipeline import policies
 from azure.core.rest import HttpRequest, HttpResponse
@@ -179,7 +180,7 @@ def _send_request(self, request: HttpRequest, *, stream: bool = False, **kwargs:
     def close(self) -> None:
         self._client.close()
 
-    def __enter__(self) -> "AVSClient":
+    def __enter__(self) -> Self:
         self._client.__enter__()
         return self
 

sdk/compute/azure-mgmt-avs/azure/mgmt/avs/_avs_client.py

@@ -144,6 +144,8 @@ def _json_attemp(data):
                 # context otherwise.
                 _LOGGER.critical("Wasn't XML not JSON, failing")
                 raise DeserializationError("XML is invalid") from err
+        elif content_type.startswith("text/"):
+            return data_as_str
         raise DeserializationError("Cannot deserialize content-type: {}".format(content_type))
 
     @classmethod

sdk/compute/azure-mgmt-avs/azure/mgmt/avs/_serialization.py

@@ -6,4 +6,4 @@
 # Changes may cause incorrect behavior and will be lost if the code is regenerated.
 # --------------------------------------------------------------------------
 
-VERSION = "9.0.0b1"
+VERSION = "9.0.0"

sdk/compute/azure-mgmt-avs/azure/mgmt/avs/_vendor.py

@@ -8,6 +8,7 @@
 
 from copy import deepcopy
 from typing import Any, Awaitable, TYPE_CHECKING
+from typing_extensions import Self
 
 from azure.core.pipeline import policies
 from azure.core.rest import AsyncHttpResponse, HttpRequest
@@ -182,7 +183,7 @@ def _send_request(
     async def close(self) -> None:
         await self._client.close()
 
-    async def __aenter__(self) -> "AVSClient":
+    async def __aenter__(self) -> Self:
         await self._client.__aenter__()
         return self
 

sdk/compute/azure-mgmt-avs/azure/mgmt/avs/_version.py

@@ -8,7 +8,7 @@
 # --------------------------------------------------------------------------
 from io import IOBase
 import sys
-from typing import Any, AsyncIterable, Callable, Dict, IO, Optional, Type, TypeVar, Union, cast, overload
+from typing import Any, AsyncIterable, AsyncIterator, Callable, Dict, IO, Optional, Type, TypeVar, Union, cast, overload
 import urllib.parse
 
 from azure.core.async_paging import AsyncItemPaged, AsyncList
@@ -18,20 +18,20 @@
     ResourceExistsError,
     ResourceNotFoundError,
     ResourceNotModifiedError,
+    StreamClosedError,
+    StreamConsumedError,
     map_error,
 )
 from azure.core.pipeline import PipelineResponse
-from azure.core.pipeline.transport import AsyncHttpResponse
 from azure.core.polling import AsyncLROPoller, AsyncNoPolling, AsyncPollingMethod
-from azure.core.rest import HttpRequest
+from azure.core.rest import AsyncHttpResponse, HttpRequest
 from azure.core.tracing.decorator import distributed_trace
 from azure.core.tracing.decorator_async import distributed_trace_async
 from azure.core.utils import case_insensitive_dict
 from azure.mgmt.core.exceptions import ARMErrorFormat
 from azure.mgmt.core.polling.async_arm_polling import AsyncARMPolling
 
 from ... import models as _models
-from ..._vendor import _convert_request
 from ...operations._addons_operations import (
     build_create_or_update_request,
     build_delete_request,
@@ -104,7 +104,6 @@ def prepare_request(next_link=None):
                     headers=_headers,
                     params=_params,
                 )
-                _request = _convert_request(_request)
                 _request.url = self._client.format_url(_request.url)
 
             else:
@@ -120,7 +119,6 @@ def prepare_request(next_link=None):
                 _request = HttpRequest(
                     "GET", urllib.parse.urljoin(next_link, _parsed_next_link.path), params=_next_request_params
                 )
-                _request = _convert_request(_request)
                 _request.url = self._client.format_url(_request.url)
                 _request.method = "GET"
             return _request
@@ -190,7 +188,6 @@ async def get(
             headers=_headers,
             params=_params,
         )
-        _request = _convert_request(_request)
         _request.url = self._client.format_url(_request.url)
 
         _stream = False
@@ -205,7 +202,7 @@ async def get(
             error = self._deserialize.failsafe_deserialize(_models.ErrorResponse, pipeline_response)
             raise HttpResponseError(response=response, model=error, error_format=ARMErrorFormat)
 
-        deserialized = self._deserialize("Addon", pipeline_response)
+        deserialized = self._deserialize("Addon", pipeline_response.http_response)
 
         if cls:
             return cls(pipeline_response, deserialized, {})  # type: ignore
@@ -219,7 +216,7 @@ async def _create_or_update_initial(
         addon_name: str,
         addon: Union[_models.Addon, IO[bytes]],
         **kwargs: Any
-    ) -> _models.Addon:
+    ) -> AsyncIterator[bytes]:
         error_map: MutableMapping[int, Type[HttpResponseError]] = {
             401: ClientAuthenticationError,
             404: ResourceNotFoundError,
@@ -233,7 +230,7 @@ async def _create_or_update_initial(
 
         api_version: str = kwargs.pop("api_version", _params.pop("api-version", self._config.api_version))
         content_type: Optional[str] = kwargs.pop("content_type", _headers.pop("Content-Type", None))
-        cls: ClsType[_models.Addon] = kwargs.pop("cls", None)
+        cls: ClsType[AsyncIterator[bytes]] = kwargs.pop("cls", None)
 
         content_type = content_type or "application/json"
         _json = None
@@ -255,29 +252,30 @@ async def _create_or_update_initial(
             headers=_headers,
             params=_params,
         )
-        _request = _convert_request(_request)
         _request.url = self._client.format_url(_request.url)
 
-        _stream = False
+        _decompress = kwargs.pop("decompress", True)
+        _stream = True
         pipeline_response: PipelineResponse = await self._client._pipeline.run(  # pylint: disable=protected-access
             _request, stream=_stream, **kwargs
         )
 
         response = pipeline_response.http_response
 
         if response.status_code not in [200, 201]:
+            try:
+                await response.read()  # Load the body in memory and close the socket
+            except (StreamConsumedError, StreamClosedError):
+                pass
             map_error(status_code=response.status_code, response=response, error_map=error_map)
             error = self._deserialize.failsafe_deserialize(_models.ErrorResponse, pipeline_response)
             raise HttpResponseError(response=response, model=error, error_format=ARMErrorFormat)
 
         response_headers = {}
-        if response.status_code == 200:
-            deserialized = self._deserialize("Addon", pipeline_response)
-
         if response.status_code == 201:
             response_headers["Retry-After"] = self._deserialize("int", response.headers.get("Retry-After"))
 
-            deserialized = self._deserialize("Addon", pipeline_response)
+        deserialized = response.stream_download(self._client._pipeline, decompress=_decompress)
 
         if cls:
             return cls(pipeline_response, deserialized, response_headers)  # type: ignore
@@ -390,10 +388,11 @@ async def begin_create_or_update(
                 params=_params,
                 **kwargs
             )
+            await raw_result.http_response.read()  # type: ignore
         kwargs.pop("error_map", None)
 
         def get_long_running_output(pipeline_response):
-            deserialized = self._deserialize("Addon", pipeline_response)
+            deserialized = self._deserialize("Addon", pipeline_response.http_response)
             if cls:
                 return cls(pipeline_response, deserialized, {})  # type: ignore
             return deserialized
@@ -418,9 +417,9 @@ def get_long_running_output(pipeline_response):
             self._client, raw_result, get_long_running_output, polling_method  # type: ignore
         )
 
-    async def _delete_initial(  # pylint: disable=inconsistent-return-statements
+    async def _delete_initial(
         self, resource_group_name: str, private_cloud_name: str, addon_name: str, **kwargs: Any
-    ) -> None:
+    ) -> AsyncIterator[bytes]:
         error_map: MutableMapping[int, Type[HttpResponseError]] = {
             401: ClientAuthenticationError,
             404: ResourceNotFoundError,
@@ -433,7 +432,7 @@ async def _delete_initial(  # pylint: disable=inconsistent-return-statements
         _params = case_insensitive_dict(kwargs.pop("params", {}) or {})
 
         api_version: str = kwargs.pop("api_version", _params.pop("api-version", self._config.api_version))
-        cls: ClsType[None] = kwargs.pop("cls", None)
+        cls: ClsType[AsyncIterator[bytes]] = kwargs.pop("cls", None)
 
         _request = build_delete_request(
             resource_group_name=resource_group_name,
@@ -444,17 +443,21 @@ async def _delete_initial(  # pylint: disable=inconsistent-return-statements
             headers=_headers,
             params=_params,
         )
-        _request = _convert_request(_request)
         _request.url = self._client.format_url(_request.url)
 
-        _stream = False
+        _decompress = kwargs.pop("decompress", True)
+        _stream = True
         pipeline_response: PipelineResponse = await self._client._pipeline.run(  # pylint: disable=protected-access
             _request, stream=_stream, **kwargs
         )
 
         response = pipeline_response.http_response
 
         if response.status_code not in [200, 202, 204]:
+            try:
+                await response.read()  # Load the body in memory and close the socket
+            except (StreamConsumedError, StreamClosedError):
+                pass
             map_error(status_code=response.status_code, response=response, error_map=error_map)
             error = self._deserialize.failsafe_deserialize(_models.ErrorResponse, pipeline_response)
             raise HttpResponseError(response=response, model=error, error_format=ARMErrorFormat)
@@ -464,8 +467,12 @@ async def _delete_initial(  # pylint: disable=inconsistent-return-statements
             response_headers["Location"] = self._deserialize("str", response.headers.get("Location"))
             response_headers["Retry-After"] = self._deserialize("int", response.headers.get("Retry-After"))
 
+        deserialized = response.stream_download(self._client._pipeline, decompress=_decompress)
+
         if cls:
-            return cls(pipeline_response, None, response_headers)  # type: ignore
+            return cls(pipeline_response, deserialized, response_headers)  # type: ignore
+
+        return deserialized  # type: ignore
 
     @distributed_trace_async
     async def begin_delete(
@@ -493,7 +500,7 @@ async def begin_delete(
         lro_delay = kwargs.pop("polling_interval", self._config.polling_interval)
         cont_token: Optional[str] = kwargs.pop("continuation_token", None)
         if cont_token is None:
-            raw_result = await self._delete_initial(  # type: ignore
+            raw_result = await self._delete_initial(
                 resource_group_name=resource_group_name,
                 private_cloud_name=private_cloud_name,
                 addon_name=addon_name,
@@ -503,6 +510,7 @@ async def begin_delete(
                 params=_params,
                 **kwargs
             )
+            await raw_result.http_response.read()  # type: ignore
         kwargs.pop("error_map", None)
 
         def get_long_running_output(pipeline_response):  # pylint: disable=inconsistent-return-statements