CodeGen from PR 19508 in Azure/azure-rest-api-specs

[Hub Generated] Review request for Microsoft.ContainerInstance to add version stable/2021-10-01 (Azure#19508)

* Fix workspacekey and identity on Get

* Update Get for dnsNameLabelReusePolicy, fix example

* Add secret identifier to properties

* Add missing ID in usage

* Fix autoGeneratedDomainNameLabelScope and dnsNameLabelReusePolicy mixup

* Remove secret id for env.val

* Add missing start return code

* Improve response message

* Integrate Rubin's changes to fix linting error

* Add Abdel's changes for missing operation

* Remove duplicate RG name parameter

* Run prettier

* Add missing description

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

@@ -1,11 +1,11 @@
 {
   "autorest": "3.7.2",
   "use": [
-    "@autorest/python@5.12.0",
+    "@autorest/python@5.16.0",
     "@autorest/modelerfour@4.19.3"
   ],
-  "commit": "5dad3b83d0926768ede15ed41895d5f38defd512",
+  "commit": "61c02924d569e1860c1011805728b16081cc1fb7",
   "repository_url": "https://github.com/Azure/azure-rest-api-specs",
-  "autorest_command": "autorest specification/containerinstance/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/containerinstance/resource-manager/readme.md --python --python-sdks-folder=/mnt/vss/_work/1/s/azure-sdk-for-python/sdk --python3-only --use=@autorest/python@5.16.0 --use=@autorest/modelerfour@4.19.3 --version=3.7.2",
   "readme": "specification/containerinstance/resource-manager/readme.md"
 }

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

@@ -10,9 +10,14 @@
 from ._version import VERSION
 
 __version__ = VERSION
+
+try:
+    from ._patch import __all__ as _patch_all
+    from ._patch import *  # type: ignore # pylint: disable=unused-wildcard-import
+except ImportError:
+    _patch_all = []
+from ._patch import patch_sdk as _patch_sdk
 __all__ = ['ContainerInstanceManagementClient']
+__all__.extend([p for p in _patch_all if p not in __all__])
 
-# `._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
-from ._patch import patch_sdk
-patch_sdk()
+_patch_sdk()

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

@@ -19,16 +19,20 @@
     from azure.core.credentials import TokenCredential
 
 
-class ContainerInstanceManagementClientConfiguration(Configuration):
+class ContainerInstanceManagementClientConfiguration(Configuration):  # pylint: disable=too-many-instance-attributes
     """Configuration for ContainerInstanceManagementClient.
 
     Note that all parameters used to create this instance are saved as instance
     attributes.
 
     :param credential: Credential needed for the client to connect to Azure.
     :type credential: ~azure.core.credentials.TokenCredential
-    :param subscription_id: Subscription credentials which uniquely identify Microsoft Azure subscription. The subscription ID forms part of the URI for every service call.
+    :param subscription_id: Subscription credentials which uniquely identify Microsoft Azure
+     subscription. The subscription ID forms part of the URI for every service call.
     :type subscription_id: str
+    :keyword api_version: Api Version. Default value is "2021-10-01". Note that overriding this
+     default value may result in unsupported behavior.
+    :paramtype api_version: str
     """
 
     def __init__(
@@ -38,14 +42,16 @@ def __init__(
         **kwargs: Any
     ) -> None:
         super(ContainerInstanceManagementClientConfiguration, self).__init__(**kwargs)
+        api_version = kwargs.pop('api_version', "2021-10-01")  # 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-10-01"
+        self.api_version = api_version
         self.credential_scopes = kwargs.pop('credential_scopes', ['https://management.azure.com/.default'])
         kwargs.setdefault('sdk_moniker', 'mgmt-containerinstance/{}'.format(VERSION))
         self._configure(**kwargs)

sdk/containerinstance/azure-mgmt-containerinstance/azure/mgmt/containerinstance/_container_instance_management_client.py

@@ -7,15 +7,16 @@
 # --------------------------------------------------------------------------
 
 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 ContainerInstanceManagementClientConfiguration
-from .operations import ContainerGroupsOperations, ContainersOperations, LocationOperations, Operations
+from .operations import ContainerGroupsOperations, ContainersOperations, LocationOperations, Operations, SubnetServiceAssociationLinkOperations
 
 if TYPE_CHECKING:
     # pylint: disable=unused-import,ungrouped-imports
@@ -32,13 +33,19 @@ class ContainerInstanceManagementClient:
     :vartype location: azure.mgmt.containerinstance.operations.LocationOperations
     :ivar containers: ContainersOperations operations
     :vartype containers: azure.mgmt.containerinstance.operations.ContainersOperations
+    :ivar subnet_service_association_link: SubnetServiceAssociationLinkOperations operations
+    :vartype subnet_service_association_link:
+     azure.mgmt.containerinstance.operations.SubnetServiceAssociationLinkOperations
     :param credential: Credential needed for the client to connect to Azure.
     :type credential: ~azure.core.credentials.TokenCredential
     :param subscription_id: Subscription credentials which uniquely identify Microsoft Azure
      subscription. The subscription ID forms part of the URI for every service call.
     :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-10-01". 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.
     """
@@ -57,15 +64,26 @@ def __init__(
         self._serialize = Serializer(client_models)
         self._deserialize = Deserializer(client_models)
         self._serialize.client_side_validation = False
-        self.container_groups = ContainerGroupsOperations(self._client, self._config, self._serialize, self._deserialize)
-        self.operations = Operations(self._client, self._config, self._serialize, self._deserialize)
-        self.location = LocationOperations(self._client, self._config, self._serialize, self._deserialize)
-        self.containers = ContainersOperations(self._client, self._config, self._serialize, self._deserialize)
+        self.container_groups = ContainerGroupsOperations(
+            self._client, self._config, self._serialize, self._deserialize
+        )
+        self.operations = Operations(
+            self._client, self._config, self._serialize, self._deserialize
+        )
+        self.location = LocationOperations(
+            self._client, self._config, self._serialize, self._deserialize
+        )
+        self.containers = ContainersOperations(
+            self._client, self._config, self._serialize, self._deserialize
+        )
+        self.subnet_service_association_link = SubnetServiceAssociationLinkOperations(
+            self._client, self._config, self._serialize, self._deserialize
+        )
 
 
     def _send_request(
         self,
-        request,  # type: HttpRequest
+        request: HttpRequest,
         **kwargs: Any
     ) -> HttpResponse:
         """Runs the network request through the client's chained policies.

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

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

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

@@ -7,9 +7,14 @@
 # --------------------------------------------------------------------------
 
 from ._container_instance_management_client import ContainerInstanceManagementClient
+
+try:
+    from ._patch import __all__ as _patch_all
+    from ._patch import *  # type: ignore # pylint: disable=unused-wildcard-import
+except ImportError:
+    _patch_all = []
+from ._patch import patch_sdk as _patch_sdk
 __all__ = ['ContainerInstanceManagementClient']
+__all__.extend([p for p in _patch_all if p not in __all__])
 
-# `._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
-from ._patch import patch_sdk
-patch_sdk()
+_patch_sdk()

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

@@ -19,16 +19,20 @@
     from azure.core.credentials_async import AsyncTokenCredential
 
 
-class ContainerInstanceManagementClientConfiguration(Configuration):
+class ContainerInstanceManagementClientConfiguration(Configuration):  # pylint: disable=too-many-instance-attributes
     """Configuration for ContainerInstanceManagementClient.
 
     Note that all parameters used to create this instance are saved as instance
     attributes.
 
     :param credential: Credential needed for the client to connect to Azure.
     :type credential: ~azure.core.credentials_async.AsyncTokenCredential
-    :param subscription_id: Subscription credentials which uniquely identify Microsoft Azure subscription. The subscription ID forms part of the URI for every service call.
+    :param subscription_id: Subscription credentials which uniquely identify Microsoft Azure
+     subscription. The subscription ID forms part of the URI for every service call.
     :type subscription_id: str
+    :keyword api_version: Api Version. Default value is "2021-10-01". Note that overriding this
+     default value may result in unsupported behavior.
+    :paramtype api_version: str
     """
 
     def __init__(
@@ -38,14 +42,16 @@ def __init__(
         **kwargs: Any
     ) -> None:
         super(ContainerInstanceManagementClientConfiguration, self).__init__(**kwargs)
+        api_version = kwargs.pop('api_version', "2021-10-01")  # 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-10-01"
+        self.api_version = api_version
         self.credential_scopes = kwargs.pop('credential_scopes', ['https://management.azure.com/.default'])
         kwargs.setdefault('sdk_moniker', 'mgmt-containerinstance/{}'.format(VERSION))
         self._configure(**kwargs)