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

Initial commit for new version and new child resource type extended l… (Azure#17681) * Initial commit for new version and new child resource type extended location New version: 2021-08-31-preview New child resource type: resource sync rule * Fix lint check errors and cross version breaking changes validation issues * Try to fix Cross-Version Breaking Changes * Revert changes to previous stable version * Remove "x-ms-identifiers" definition which will cause golang sdk generation error * Address review comments, revert unnecessary changes * Try fix breaking error * Revert changes which fix lint error but will cause cross version breaking changes * Fix lintdiff validation issue and also add back two updates for this version only * Update descriptions of properties of resource sync rules. * Update description of the resource sync rules API * Update description of resource sync rules selector property * Update description of matchExpressions * Update description of targetResourceGroup * Fix a required LintDiff check Adding "x-ms-identifiers": [] to matchExpressionsList * Add title to readme.typescript.md * Update specification/extendedlocation/resource-manager/readme.typescript.md * Revert cross version breacking changes Co-authored-by: Wei Dong <[email protected]>
AzureSDKAutomation · Feb 17, 2022 · 123f686 · 123f686
1 parent f424ae4
commit 123f686
Show file tree

Hide file tree

Showing 60 changed files with 7,362 additions and 2,818 deletions.
diff --git a/sdk/extendedlocation/azure-mgmt-extendedlocation/_meta.json b/sdk/extendedlocation/azure-mgmt-extendedlocation/_meta.json
@@ -1,11 +1,11 @@
 {
-  "autorest": "3.4.5",
+  "autorest": "3.7.2",
   "use": [
-    "@autorest/python@5.8.4",
-    "@autorest/[email protected].2"
+    "@autorest/python@5.12.0",
+    "@autorest/[email protected].3"
   ],
-  "commit": "e687b0f9c1856f7f20e1c0e1aca53b1e342cde3e",
+  "commit": "bb4552404ae10aae4389289182bd0b91f6691f10",
   "repository_url": "https://github.com/Azure/azure-rest-api-specs",
-  "autorest_command": "autorest specification/extendedlocation/resource-manager/readme.md --multiapi --python --python-mode=update --python-sdks-folder=/home/vsts/work/1/s/azure-sdk-for-python/sdk --track2 --use=@autorest/python@5.8.4 --use=@autorest/[email protected].2 --version=3.4.5",
+  "autorest_command": "autorest specification/extendedlocation/resource-manager/readme.md --multiapi --python --python-mode=update --python-sdks-folder=/home/vsts/work/1/s/azure-sdk-for-python/sdk --python3-only --track2 --use=@autorest/python@5.12.0 --use=@autorest/[email protected].3 --version=3.7.2",
   "readme": "specification/extendedlocation/resource-manager/readme.md"
 }
diff --git a/...xtendedlocation/azure-mgmt-extendedlocation/azure/mgmt/extendedlocation/_configuration.py b/...xtendedlocation/azure-mgmt-extendedlocation/azure/mgmt/extendedlocation/_configuration.py
@@ -12,7 +12,7 @@
 
 from azure.core.configuration import Configuration
 from azure.core.pipeline import policies
-from azure.mgmt.core.policies import ARMHttpLoggingPolicy
+from azure.mgmt.core.policies import ARMChallengeAuthenticationPolicy, ARMHttpLoggingPolicy
 
 from ._version import VERSION
 
@@ -68,4 +68,4 @@ def _configure(
         self.redirect_policy = kwargs.get('redirect_policy') or policies.RedirectPolicy(**kwargs)
         self.authentication_policy = kwargs.get('authentication_policy')
         if self.credential and not self.authentication_policy:
-            self.authentication_policy = policies.BearerTokenCredentialPolicy(self.credential, *self.credential_scopes, **kwargs)
+            self.authentication_policy = ARMChallengeAuthenticationPolicy(self.credential, *self.credential_scopes, **kwargs)
diff --git a/...ndedlocation/azure-mgmt-extendedlocation/azure/mgmt/extendedlocation/_custom_locations.py b/...ndedlocation/azure-mgmt-extendedlocation/azure/mgmt/extendedlocation/_custom_locations.py
@@ -23,7 +23,6 @@
     from typing import Any, Optional
 
     from azure.core.credentials import TokenCredential
-    from azure.core.pipeline.transport import HttpRequest, HttpResponse
 
 class _SDKClient(object):
     def __init__(self, *args, **kwargs):
@@ -70,12 +69,10 @@ def __init__(
         credential,  # type: "TokenCredential"
         subscription_id,  # type: str
         api_version=None, # type: Optional[str]
-        base_url=None,  # type: Optional[str]
+        base_url="https://management.azure.com",  # type: str
         profile=KnownProfiles.default, # type: KnownProfiles
         **kwargs  # type: Any
     ):
-        if not base_url:
-            base_url = 'https://management.azure.com'
         self._config = CustomLocationsConfiguration(credential, subscription_id, **kwargs)
         self._client = ARMPipelineClient(base_url=base_url, config=self._config, **kwargs)
         super(CustomLocations, self).__init__(
@@ -93,13 +90,17 @@ def models(cls, api_version=DEFAULT_API_VERSION):
 
            * 2021-03-15-preview: :mod:`v2021_03_15_preview.models<azure.mgmt.extendedlocation.v2021_03_15_preview.models>`
            * 2021-08-15: :mod:`v2021_08_15.models<azure.mgmt.extendedlocation.v2021_08_15.models>`
+           * 2021-08-31-preview: :mod:`v2021_08_31_preview.models<azure.mgmt.extendedlocation.v2021_08_31_preview.models>`
         """
         if api_version == '2021-03-15-preview':
             from .v2021_03_15_preview import models
             return models
         elif api_version == '2021-08-15':
             from .v2021_08_15 import models
             return models
+        elif api_version == '2021-08-31-preview':
+            from .v2021_08_31_preview import models
+            return models
         raise ValueError("API version {} is not available".format(api_version))
 
     @property
@@ -108,16 +109,32 @@ def custom_locations(self):
 
            * 2021-03-15-preview: :class:`CustomLocationsOperations<azure.mgmt.extendedlocation.v2021_03_15_preview.operations.CustomLocationsOperations>`
            * 2021-08-15: :class:`CustomLocationsOperations<azure.mgmt.extendedlocation.v2021_08_15.operations.CustomLocationsOperations>`
+           * 2021-08-31-preview: :class:`CustomLocationsOperations<azure.mgmt.extendedlocation.v2021_08_31_preview.operations.CustomLocationsOperations>`
         """
         api_version = self._get_api_version('custom_locations')
         if api_version == '2021-03-15-preview':
             from .v2021_03_15_preview.operations import CustomLocationsOperations as OperationClass
         elif api_version == '2021-08-15':
             from .v2021_08_15.operations import CustomLocationsOperations as OperationClass
+        elif api_version == '2021-08-31-preview':
+            from .v2021_08_31_preview.operations import CustomLocationsOperations as OperationClass
         else:
             raise ValueError("API version {} does not have operation group 'custom_locations'".format(api_version))
         return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version)))
 
+    @property
+    def resource_sync_rules(self):
+        """Instance depends on the API version:
+
+           * 2021-08-31-preview: :class:`ResourceSyncRulesOperations<azure.mgmt.extendedlocation.v2021_08_31_preview.operations.ResourceSyncRulesOperations>`
+        """
+        api_version = self._get_api_version('resource_sync_rules')
+        if api_version == '2021-08-31-preview':
+            from .v2021_08_31_preview.operations import ResourceSyncRulesOperations as OperationClass
+        else:
+            raise ValueError("API version {} does not have operation group 'resource_sync_rules'".format(api_version))
+        return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version)))
+
     def close(self):
         self._client.close()
     def __enter__(self):

diff --git a/...dedlocation/azure-mgmt-extendedlocation/azure/mgmt/extendedlocation/aio/_configuration.py b/...dedlocation/azure-mgmt-extendedlocation/azure/mgmt/extendedlocation/aio/_configuration.py
@@ -12,7 +12,7 @@
 
 from azure.core.configuration import Configuration
 from azure.core.pipeline import policies
-from azure.mgmt.core.policies import ARMHttpLoggingPolicy
+from azure.mgmt.core.policies import ARMHttpLoggingPolicy, AsyncARMChallengeAuthenticationPolicy
 
 from .._version import VERSION
 
@@ -64,4 +64,4 @@ def _configure(
         self.redirect_policy = kwargs.get('redirect_policy') or policies.AsyncRedirectPolicy(**kwargs)
         self.authentication_policy = kwargs.get('authentication_policy')
         if self.credential and not self.authentication_policy:
-            self.authentication_policy = policies.AsyncBearerTokenCredentialPolicy(self.credential, *self.credential_scopes, **kwargs)
+            self.authentication_policy = AsyncARMChallengeAuthenticationPolicy(self.credential, *self.credential_scopes, **kwargs)
diff --git a/...location/azure-mgmt-extendedlocation/azure/mgmt/extendedlocation/aio/_custom_locations.py b/...location/azure-mgmt-extendedlocation/azure/mgmt/extendedlocation/aio/_custom_locations.py
@@ -11,7 +11,6 @@
 
 from typing import Any, Optional, TYPE_CHECKING
 
-from azure.core.pipeline.transport import AsyncHttpResponse, HttpRequest
 from azure.mgmt.core import AsyncARMPipelineClient
 from azure.profiles import KnownProfiles, ProfileDefinition
 from azure.profiles.multiapiclient import MultiApiClientMixin
@@ -21,6 +20,7 @@
 
 if TYPE_CHECKING:
     # pylint: disable=unused-import,ungrouped-imports
+    from azure.core.credentials import TokenCredential
     from azure.core.credentials_async import AsyncTokenCredential
 
 class _SDKClient(object):
@@ -68,12 +68,10 @@ def __init__(
         credential: "AsyncTokenCredential",
         subscription_id: str,
         api_version: Optional[str] = None,
-        base_url: Optional[str] = None,
+        base_url: str = "https://management.azure.com",
         profile: KnownProfiles = KnownProfiles.default,
         **kwargs  # type: Any
     ) -> None:
-        if not base_url:
-            base_url = 'https://management.azure.com'
         self._config = CustomLocationsConfiguration(credential, subscription_id, **kwargs)
         self._client = AsyncARMPipelineClient(base_url=base_url, config=self._config, **kwargs)
         super(CustomLocations, self).__init__(
@@ -91,13 +89,17 @@ def models(cls, api_version=DEFAULT_API_VERSION):
 
            * 2021-03-15-preview: :mod:`v2021_03_15_preview.models<azure.mgmt.extendedlocation.v2021_03_15_preview.models>`
            * 2021-08-15: :mod:`v2021_08_15.models<azure.mgmt.extendedlocation.v2021_08_15.models>`
+           * 2021-08-31-preview: :mod:`v2021_08_31_preview.models<azure.mgmt.extendedlocation.v2021_08_31_preview.models>`
         """
         if api_version == '2021-03-15-preview':
             from ..v2021_03_15_preview import models
             return models
         elif api_version == '2021-08-15':
             from ..v2021_08_15 import models
             return models
+        elif api_version == '2021-08-31-preview':
+            from ..v2021_08_31_preview import models
+            return models
         raise ValueError("API version {} is not available".format(api_version))
 
     @property
@@ -106,16 +108,32 @@ def custom_locations(self):
 
            * 2021-03-15-preview: :class:`CustomLocationsOperations<azure.mgmt.extendedlocation.v2021_03_15_preview.aio.operations.CustomLocationsOperations>`
            * 2021-08-15: :class:`CustomLocationsOperations<azure.mgmt.extendedlocation.v2021_08_15.aio.operations.CustomLocationsOperations>`
+           * 2021-08-31-preview: :class:`CustomLocationsOperations<azure.mgmt.extendedlocation.v2021_08_31_preview.aio.operations.CustomLocationsOperations>`
         """
         api_version = self._get_api_version('custom_locations')
         if api_version == '2021-03-15-preview':
             from ..v2021_03_15_preview.aio.operations import CustomLocationsOperations as OperationClass
         elif api_version == '2021-08-15':
             from ..v2021_08_15.aio.operations import CustomLocationsOperations as OperationClass
+        elif api_version == '2021-08-31-preview':
+            from ..v2021_08_31_preview.aio.operations import CustomLocationsOperations as OperationClass
         else:
             raise ValueError("API version {} does not have operation group 'custom_locations'".format(api_version))
         return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version)))
 
+    @property
+    def resource_sync_rules(self):
+        """Instance depends on the API version:
+
+           * 2021-08-31-preview: :class:`ResourceSyncRulesOperations<azure.mgmt.extendedlocation.v2021_08_31_preview.aio.operations.ResourceSyncRulesOperations>`
+        """
+        api_version = self._get_api_version('resource_sync_rules')
+        if api_version == '2021-08-31-preview':
+            from ..v2021_08_31_preview.aio.operations import ResourceSyncRulesOperations as OperationClass
+        else:
+            raise ValueError("API version {} does not have operation group 'resource_sync_rules'".format(api_version))
+        return OperationClass(self._client, self._config, Serializer(self._models_dict(api_version)), Deserializer(self._models_dict(api_version)))
+
     async def close(self):
         await self._client.close()
     async def __aenter__(self):

diff --git a/...n/azure-mgmt-extendedlocation/azure/mgmt/extendedlocation/v2021_03_15_preview/__init__.py b/...n/azure-mgmt-extendedlocation/azure/mgmt/extendedlocation/v2021_03_15_preview/__init__.py
@@ -12,8 +12,7 @@
 __version__ = VERSION
 __all__ = ['CustomLocations']
 
-try:
-    from ._patch import patch_sdk  # type: ignore
-    patch_sdk()
-except ImportError:
-    pass
+# `._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()
diff --git a/...e-mgmt-extendedlocation/azure/mgmt/extendedlocation/v2021_03_15_preview/_configuration.py b/...e-mgmt-extendedlocation/azure/mgmt/extendedlocation/v2021_03_15_preview/_configuration.py
@@ -6,18 +6,16 @@
 # Changes may cause incorrect behavior and will be lost if the code is regenerated.
 # --------------------------------------------------------------------------
 
-from typing import TYPE_CHECKING
+from typing import Any, TYPE_CHECKING
 
 from azure.core.configuration import Configuration
 from azure.core.pipeline import policies
-from azure.mgmt.core.policies import ARMHttpLoggingPolicy
+from azure.mgmt.core.policies import ARMChallengeAuthenticationPolicy, ARMHttpLoggingPolicy
 
 from ._version import VERSION
 
 if TYPE_CHECKING:
     # pylint: disable=unused-import,ungrouped-imports
-    from typing import Any
-
     from azure.core.credentials import TokenCredential
 
 
@@ -35,16 +33,15 @@ class CustomLocationsConfiguration(Configuration):
 
     def __init__(
         self,
-        credential,  # type: "TokenCredential"
-        subscription_id,  # type: str
-        **kwargs  # type: Any
-    ):
-        # type: (...) -> None
+        credential: "TokenCredential",
+        subscription_id: str,
+        **kwargs: Any
+    ) -> None:
+        super(CustomLocationsConfiguration, self).__init__(**kwargs)
         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.")
-        super(CustomLocationsConfiguration, self).__init__(**kwargs)
 
         self.credential = credential
         self.subscription_id = subscription_id
@@ -68,4 +65,4 @@ def _configure(
         self.redirect_policy = kwargs.get('redirect_policy') or policies.RedirectPolicy(**kwargs)
         self.authentication_policy = kwargs.get('authentication_policy')
         if self.credential and not self.authentication_policy:
-            self.authentication_policy = policies.BearerTokenCredentialPolicy(self.credential, *self.credential_scopes, **kwargs)
+            self.authentication_policy = ARMChallengeAuthenticationPolicy(self.credential, *self.credential_scopes, **kwargs)
diff --git a/...gmt-extendedlocation/azure/mgmt/extendedlocation/v2021_03_15_preview/_custom_locations.py b/...gmt-extendedlocation/azure/mgmt/extendedlocation/v2021_03_15_preview/_custom_locations.py
@@ -6,74 +6,79 @@
 # Changes may cause incorrect behavior and will be lost if the code is regenerated.
 # --------------------------------------------------------------------------
 
-from typing import TYPE_CHECKING
+from copy import deepcopy
+from typing import Any, Optional, TYPE_CHECKING
 
+from azure.core.rest import HttpRequest, HttpResponse
 from azure.mgmt.core import ARMPipelineClient
 from msrest import Deserializer, Serializer
 
-if TYPE_CHECKING:
-    # pylint: disable=unused-import,ungrouped-imports
-    from typing import Any, Optional
-
-    from azure.core.credentials import TokenCredential
-    from azure.core.pipeline.transport import HttpRequest, HttpResponse
-
+from . import models
 from ._configuration import CustomLocationsConfiguration
 from .operations import CustomLocationsOperations
-from . import models
 
+if TYPE_CHECKING:
+    # pylint: disable=unused-import,ungrouped-imports
+    from azure.core.credentials import TokenCredential
 
-class CustomLocations(object):
+class CustomLocations:
     """The customLocations Rest API spec.
 
     :ivar custom_locations: CustomLocationsOperations operations
-    :vartype custom_locations: azure.mgmt.extendedlocation.v2021_03_15_preview.operations.CustomLocationsOperations
+    :vartype custom_locations:
+     azure.mgmt.extendedlocation.v2021_03_15_preview.operations.CustomLocationsOperations
     :param credential: Credential needed for the client to connect to Azure.
     :type credential: ~azure.core.credentials.TokenCredential
     :param subscription_id: The ID of the target subscription.
     :type subscription_id: str
-    :param str base_url: Service URL
-    :keyword int polling_interval: Default waiting time between two polls for LRO operations if no Retry-After header is present.
+    :param base_url: Service URL. Default value is 'https://management.azure.com'.
+    :type base_url: str
+    :keyword int polling_interval: Default waiting time between two polls for LRO operations if no
+     Retry-After header is present.
     """
 
     def __init__(
         self,
-        credential,  # type: "TokenCredential"
-        subscription_id,  # type: str
-        base_url=None,  # type: Optional[str]
-        **kwargs  # type: Any
-    ):
-        # type: (...) -> None
-        if not base_url:
-            base_url = 'https://management.azure.com'
-        self._config = CustomLocationsConfiguration(credential, subscription_id, **kwargs)
+        credential: "TokenCredential",
+        subscription_id: str,
+        base_url: str = "https://management.azure.com",
+        **kwargs: Any
+    ) -> None:
+        self._config = CustomLocationsConfiguration(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)}
         self._serialize = Serializer(client_models)
-        self._serialize.client_side_validation = False
         self._deserialize = Deserializer(client_models)
+        self._serialize.client_side_validation = False
+        self.custom_locations = CustomLocationsOperations(self._client, self._config, self._serialize, self._deserialize)
 
-        self.custom_locations = CustomLocationsOperations(
-            self._client, self._config, self._serialize, self._deserialize)
 
-    def _send_request(self, http_request, **kwargs):
-        # type: (HttpRequest, Any) -> HttpResponse
+    def _send_request(
+        self,
+        request,  # type: HttpRequest
+        **kwargs: Any
+    ) -> HttpResponse:
         """Runs the network request through the client's chained policies.
 
-        :param http_request: The network request you want to make. Required.
-        :type http_request: ~azure.core.pipeline.transport.HttpRequest
-        :keyword bool stream: Whether the response payload will be streamed. Defaults to True.
+        >>> from azure.core.rest import HttpRequest
+        >>> request = HttpRequest("GET", "https://www.example.org/")
+        <HttpRequest [GET], url: 'https://www.example.org/'>
+        >>> response = client._send_request(request)
+        <HttpResponse: 200 OK>
+
+        For more information on this code flow, see https://aka.ms/azsdk/python/protocol/quickstart
+
+        :param request: The network request you want to make. Required.
+        :type request: ~azure.core.rest.HttpRequest
+        :keyword bool stream: Whether the response payload will be streamed. Defaults to False.
         :return: The response of your network call. Does not do error handling on your response.
-        :rtype: ~azure.core.pipeline.transport.HttpResponse
+        :rtype: ~azure.core.rest.HttpResponse
         """
-        path_format_arguments = {
-            'subscriptionId': self._serialize.url("self._config.subscription_id", self._config.subscription_id, 'str', min_length=1),
-        }
-        http_request.url = self._client.format_url(http_request.url, **path_format_arguments)
-        stream = kwargs.pop("stream", True)
-        pipeline_response = self._client._pipeline.run(http_request, stream=stream, **kwargs)
-        return pipeline_response.http_response
+
+        request_copy = deepcopy(request)
+        request_copy.url = self._client.format_url(request_copy.url)
+        return self._client.send_request(request_copy, **kwargs)
 
     def close(self):
         # type: () -> None