[ReleasePR track2_azure-mgmt-labservices] [Hub Generated] Review request for Microsoft.LabServices to add version stable/2023-06-07

sdk/labservices/azure-mgmt-labservices/_meta.json

@@ -1,11 +1,11 @@
 {
   "autorest": "3.7.2",
   "use": [
-    "@autorest/python@5.12.0",
+    "@autorest/python@5.13.0",
     "@autorest/modelerfour@4.19.3"
   ],
-  "commit": "0f149eb7239f7a262601bb5476e7772f06d1b28d",
+  "commit": "33de1e86182c9e74865ebcb2c9f6245774e09ab2",
   "repository_url": "https://github.com/Azure/azure-rest-api-specs",
-  "autorest_command": "autorest specification/labservices/resource-manager/readme.md --multiapi --python --python-mode=update --python-sdks-folder=/home/vsts/work/1/azure-sdk-for-python/sdk --python3-only --track2 --use=@autorest/python@5.12.0 --use=@autorest/modelerfour@4.19.3 --version=3.7.2",
+  "autorest_command": "autorest specification/labservices/resource-manager/readme.md --multiapi --python --python-sdks-folder=/mnt/vss/_work/1/s/azure-sdk-for-python/sdk --python3-only --use=@autorest/python@5.13.0 --use=@autorest/modelerfour@4.19.3 --version=3.7.2",
   "readme": "specification/labservices/resource-manager/readme.md"
 }

sdk/labservices/azure-mgmt-labservices/azure/mgmt/labservices/__init__.py

@@ -6,11 +6,11 @@
 # Changes may cause incorrect behavior and will be lost if the code is regenerated.
 # --------------------------------------------------------------------------
 
-from ._lab_services_client import LabServicesClient
+from ._managed_labs_client import ManagedLabsClient
 from ._version import VERSION
 
 __version__ = VERSION
-__all__ = ['LabServicesClient']
+__all__ = ['ManagedLabsClient']
 
 # `._patch.py` is used for handwritten extensions to the generated code
 # Example: https://github.com/Azure/azure-sdk-for-python/blob/main/doc/dev/customize_code/how-to-patch-sdk-code.md

sdk/labservices/azure-mgmt-labservices/azure/mgmt/labservices/_configuration.py

@@ -19,8 +19,8 @@
     from azure.core.credentials import TokenCredential
 
 
-class LabServicesClientConfiguration(Configuration):
-    """Configuration for LabServicesClient.
+class ManagedLabsClientConfiguration(Configuration):  # pylint: disable=too-many-instance-attributes
+    """Configuration for ManagedLabsClient.
 
     Note that all parameters used to create this instance are saved as instance
     attributes.
@@ -29,6 +29,9 @@ class LabServicesClientConfiguration(Configuration):
     :type credential: ~azure.core.credentials.TokenCredential
     :param subscription_id: The ID of the target subscription.
     :type subscription_id: str
+    :keyword api_version: Api Version. Default value is "2021-11-15-preview". Note that overriding
+     this default value may result in unsupported behavior.
+    :paramtype api_version: str
     """
 
     def __init__(
@@ -37,15 +40,17 @@ def __init__(
         subscription_id: str,
         **kwargs: Any
     ) -> None:
-        super(LabServicesClientConfiguration, self).__init__(**kwargs)
+        super(ManagedLabsClientConfiguration, self).__init__(**kwargs)
+        api_version = kwargs.pop('api_version', "2021-11-15-preview")  # type: str
+
         if credential is None:
             raise ValueError("Parameter 'credential' must not be None.")
         if subscription_id is None:
             raise ValueError("Parameter 'subscription_id' must not be None.")
 
         self.credential = credential
         self.subscription_id = subscription_id
-        self.api_version = "2021-11-15-preview"
+        self.api_version = api_version
         self.credential_scopes = kwargs.pop('credential_scopes', ['https://management.azure.com/.default'])
         kwargs.setdefault('sdk_moniker', 'mgmt-labservices/{}'.format(VERSION))
         self._configure(**kwargs)

sdk/labservices/azure-mgmt-labservices/azure/mgmt/labservices/_lab_services_client.py → sdk/labservices/azure-mgmt-labservices/azure/mgmt/labservices/_managed_labs_client.py

@@ -7,21 +7,22 @@
 # --------------------------------------------------------------------------
 
 from copy import deepcopy
-from typing import Any, Optional, TYPE_CHECKING
+from typing import Any, TYPE_CHECKING
+
+from msrest import Deserializer, Serializer
 
 from azure.core.rest import HttpRequest, HttpResponse
 from azure.mgmt.core import ARMPipelineClient
-from msrest import Deserializer, Serializer
 
 from . import models
-from ._configuration import LabServicesClientConfiguration
+from ._configuration import ManagedLabsClientConfiguration
 from .operations import ImagesOperations, LabPlansOperations, LabsOperations, OperationResultsOperations, Operations, SchedulesOperations, SkusOperations, UsagesOperations, UsersOperations, VirtualMachinesOperations
 
 if TYPE_CHECKING:
     # pylint: disable=unused-import,ungrouped-imports
     from azure.core.credentials import TokenCredential
 
-class LabServicesClient:
+class ManagedLabsClient:    # pylint: disable=too-many-instance-attributes
     """REST API for managing Azure Lab Services images.
 
     :ivar images: ImagesOperations operations
@@ -48,8 +49,11 @@ class LabServicesClient:
     :type credential: ~azure.core.credentials.TokenCredential
     :param subscription_id: The ID of the target subscription.
     :type subscription_id: str
-    :param base_url: Service URL. Default value is 'https://management.azure.com'.
+    :param base_url: Service URL. Default value is "https://management.azure.com".
     :type base_url: str
+    :keyword api_version: Api Version. Default value is "2021-11-15-preview". Note that overriding
+     this default value may result in unsupported behavior.
+    :paramtype api_version: str
     :keyword int polling_interval: Default waiting time between two polls for LRO operations if no
      Retry-After header is present.
     """
@@ -61,7 +65,7 @@ def __init__(
         base_url: str = "https://management.azure.com",
         **kwargs: Any
     ) -> None:
-        self._config = LabServicesClientConfiguration(credential=credential, subscription_id=subscription_id, **kwargs)
+        self._config = ManagedLabsClientConfiguration(credential=credential, subscription_id=subscription_id, **kwargs)
         self._client = ARMPipelineClient(base_url=base_url, config=self._config, **kwargs)
 
         client_models = {k: v for k, v in models.__dict__.items() if isinstance(v, type)}
@@ -82,7 +86,7 @@ def __init__(
 
     def _send_request(
         self,
-        request,  # type: HttpRequest
+        request: HttpRequest,
         **kwargs: Any
     ) -> HttpResponse:
         """Runs the network request through the client's chained policies.
@@ -111,7 +115,7 @@ def close(self):
         self._client.close()
 
     def __enter__(self):
-        # type: () -> LabServicesClient
+        # type: () -> ManagedLabsClient
         self._client.__enter__()
         return self
 

sdk/labservices/azure-mgmt-labservices/azure/mgmt/labservices/_metadata.json

@@ -2,16 +2,16 @@
     "chosen_version": "2021-11-15-preview",
     "total_api_version_list": ["2021-11-15-preview"],
     "client": {
-        "name": "LabServicesClient",
-        "filename": "_lab_services_client",
+        "name": "ManagedLabsClient",
+        "filename": "_managed_labs_client",
         "description": "REST API for managing Azure Lab Services images.",
         "host_value": "\"https://management.azure.com\"",
         "parameterized_host_template": null,
         "azure_arm": true,
         "has_lro_operations": true,
         "client_side_validation": false,
-        "sync_imports": "{\"typing\": {\"azurecore\": {\"azure.core.credentials\": [\"TokenCredential\"]}}, \"regular\": {\"azurecore\": {\"azure.profiles\": [\"KnownProfiles\", \"ProfileDefinition\"], \"azure.profiles.multiapiclient\": [\"MultiApiClientMixin\"], \"msrest\": [\"Deserializer\", \"Serializer\"], \"azure.mgmt.core\": [\"ARMPipelineClient\"]}, \"local\": {\"._configuration\": [\"LabServicesClientConfiguration\"]}}, \"conditional\": {\"stdlib\": {\"typing\": [\"Any\", \"Optional\"]}}}",
-        "async_imports": "{\"typing\": {\"azurecore\": {\"azure.core.credentials_async\": [\"AsyncTokenCredential\"], \"azure.core.credentials\": [\"TokenCredential\"]}}, \"regular\": {\"azurecore\": {\"azure.profiles\": [\"KnownProfiles\", \"ProfileDefinition\"], \"azure.profiles.multiapiclient\": [\"MultiApiClientMixin\"], \"msrest\": [\"Deserializer\", \"Serializer\"], \"azure.mgmt.core\": [\"AsyncARMPipelineClient\"]}, \"local\": {\"._configuration\": [\"LabServicesClientConfiguration\"]}}, \"conditional\": {\"stdlib\": {\"typing\": [\"Any\", \"Optional\"]}}}"
+        "sync_imports": "{\"typing\": {\"azurecore\": {\"azure.core.credentials\": [\"TokenCredential\"]}}, \"regular\": {\"azurecore\": {\"azure.profiles\": [\"KnownProfiles\", \"ProfileDefinition\"], \"azure.profiles.multiapiclient\": [\"MultiApiClientMixin\"], \"azure.mgmt.core\": [\"ARMPipelineClient\"]}, \"local\": {\"._configuration\": [\"ManagedLabsClientConfiguration\"]}, \"thirdparty\": {\"msrest\": [\"Deserializer\", \"Serializer\"]}}, \"conditional\": {\"stdlib\": {\"typing\": [\"Any\", \"Optional\"]}}}",
+        "async_imports": "{\"typing\": {\"azurecore\": {\"azure.core.credentials_async\": [\"AsyncTokenCredential\"], \"azure.core.credentials\": [\"TokenCredential\"]}}, \"regular\": {\"azurecore\": {\"azure.profiles\": [\"KnownProfiles\", \"ProfileDefinition\"], \"azure.profiles.multiapiclient\": [\"MultiApiClientMixin\"], \"azure.mgmt.core\": [\"AsyncARMPipelineClient\"]}, \"local\": {\"._configuration\": [\"ManagedLabsClientConfiguration\"]}, \"thirdparty\": {\"msrest\": [\"Deserializer\", \"Serializer\"]}}, \"conditional\": {\"stdlib\": {\"typing\": [\"Any\", \"Optional\"]}}}"
     },
     "global_parameters": {
         "sync": {

sdk/labservices/azure-mgmt-labservices/azure/mgmt/labservices/_version.py

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

sdk/labservices/azure-mgmt-labservices/azure/mgmt/labservices/aio/__init__.py

@@ -6,8 +6,8 @@
 # Changes may cause incorrect behavior and will be lost if the code is regenerated.
 # --------------------------------------------------------------------------
 
-from ._lab_services_client import LabServicesClient
-__all__ = ['LabServicesClient']
+from ._managed_labs_client import ManagedLabsClient
+__all__ = ['ManagedLabsClient']
 
 # `._patch.py` is used for handwritten extensions to the generated code
 # Example: https://github.com/Azure/azure-sdk-for-python/blob/main/doc/dev/customize_code/how-to-patch-sdk-code.md

sdk/labservices/azure-mgmt-labservices/azure/mgmt/labservices/aio/_configuration.py

@@ -19,8 +19,8 @@
     from azure.core.credentials_async import AsyncTokenCredential
 
 
-class LabServicesClientConfiguration(Configuration):
-    """Configuration for LabServicesClient.
+class ManagedLabsClientConfiguration(Configuration):  # pylint: disable=too-many-instance-attributes
+    """Configuration for ManagedLabsClient.
 
     Note that all parameters used to create this instance are saved as instance
     attributes.
@@ -29,6 +29,9 @@ class LabServicesClientConfiguration(Configuration):
     :type credential: ~azure.core.credentials_async.AsyncTokenCredential
     :param subscription_id: The ID of the target subscription.
     :type subscription_id: str
+    :keyword api_version: Api Version. Default value is "2021-11-15-preview". Note that overriding
+     this default value may result in unsupported behavior.
+    :paramtype api_version: str
     """
 
     def __init__(
@@ -37,15 +40,17 @@ def __init__(
         subscription_id: str,
         **kwargs: Any
     ) -> None:
-        super(LabServicesClientConfiguration, self).__init__(**kwargs)
+        super(ManagedLabsClientConfiguration, self).__init__(**kwargs)
+        api_version = kwargs.pop('api_version', "2021-11-15-preview")  # type: str
+
         if credential is None:
             raise ValueError("Parameter 'credential' must not be None.")
         if subscription_id is None:
             raise ValueError("Parameter 'subscription_id' must not be None.")
 
         self.credential = credential
         self.subscription_id = subscription_id
-        self.api_version = "2021-11-15-preview"
+        self.api_version = api_version
         self.credential_scopes = kwargs.pop('credential_scopes', ['https://management.azure.com/.default'])
         kwargs.setdefault('sdk_moniker', 'mgmt-labservices/{}'.format(VERSION))
         self._configure(**kwargs)

sdk/labservices/azure-mgmt-labservices/azure/mgmt/labservices/aio/_lab_services_client.py → sdk/labservices/azure-mgmt-labservices/azure/mgmt/labservices/aio/_managed_labs_client.py

@@ -7,21 +7,22 @@
 # --------------------------------------------------------------------------
 
 from copy import deepcopy
-from typing import Any, Awaitable, Optional, TYPE_CHECKING
+from typing import Any, Awaitable, TYPE_CHECKING
+
+from msrest import Deserializer, Serializer
 
 from azure.core.rest import AsyncHttpResponse, HttpRequest
 from azure.mgmt.core import AsyncARMPipelineClient
-from msrest import Deserializer, Serializer
 
 from .. import models
-from ._configuration import LabServicesClientConfiguration
+from ._configuration import ManagedLabsClientConfiguration
 from .operations import ImagesOperations, LabPlansOperations, LabsOperations, OperationResultsOperations, Operations, SchedulesOperations, SkusOperations, UsagesOperations, UsersOperations, VirtualMachinesOperations
 
 if TYPE_CHECKING:
     # pylint: disable=unused-import,ungrouped-imports
     from azure.core.credentials_async import AsyncTokenCredential
 
-class LabServicesClient:
+class ManagedLabsClient:    # pylint: disable=too-many-instance-attributes
     """REST API for managing Azure Lab Services images.
 
     :ivar images: ImagesOperations operations
@@ -48,8 +49,11 @@ class LabServicesClient:
     :type credential: ~azure.core.credentials_async.AsyncTokenCredential
     :param subscription_id: The ID of the target subscription.
     :type subscription_id: str
-    :param base_url: Service URL. Default value is 'https://management.azure.com'.
+    :param base_url: Service URL. Default value is "https://management.azure.com".
     :type base_url: str
+    :keyword api_version: Api Version. Default value is "2021-11-15-preview". Note that overriding
+     this default value may result in unsupported behavior.
+    :paramtype api_version: str
     :keyword int polling_interval: Default waiting time between two polls for LRO operations if no
      Retry-After header is present.
     """
@@ -61,7 +65,7 @@ def __init__(
         base_url: str = "https://management.azure.com",
         **kwargs: Any
     ) -> None:
-        self._config = LabServicesClientConfiguration(credential=credential, subscription_id=subscription_id, **kwargs)
+        self._config = ManagedLabsClientConfiguration(credential=credential, subscription_id=subscription_id, **kwargs)
         self._client = AsyncARMPipelineClient(base_url=base_url, config=self._config, **kwargs)
 
         client_models = {k: v for k, v in models.__dict__.items() if isinstance(v, type)}
@@ -109,7 +113,7 @@ def _send_request(
     async def close(self) -> None:
         await self._client.close()
 
-    async def __aenter__(self) -> "LabServicesClient":
+    async def __aenter__(self) -> "ManagedLabsClient":
         await self._client.__aenter__()
         return self