[Key Vault] Add API version 7.2 for certificates (Azure#18722)

Author: mccoyp

sdk/keyvault/azure-keyvault-certificates/CHANGELOG.md

@@ -1,6 +1,10 @@
 # Release History
 
 ## 4.3.0b1 (Unreleased)
+### Changed
+- Key Vault API version 7.2 is now the default
+- Updated msrest requirement to >=0.6.21
+
 ### Added
 - Added method `parse_key_vault_certificate_id` that parses out a full ID returned by Key Vault, so users can easily
 access the certificate's `name`, `vault_url`, and `version`.

sdk/keyvault/azure-keyvault-certificates/azure/keyvault/certificates/_generated/_configuration.py

@@ -8,13 +8,16 @@
 # Changes may cause incorrect behavior and will be lost if the code is
 # regenerated.
 # --------------------------------------------------------------------------
-from typing import Any
+from typing import TYPE_CHECKING
 
 from azure.core.configuration import Configuration
 from azure.core.pipeline import policies
 
 from ._version import VERSION
 
+if TYPE_CHECKING:
+    # pylint: disable=unused-import,ungrouped-imports
+    from typing import Any
 
 class KeyVaultClientConfiguration(Configuration):
     """Configuration for KeyVaultClient.

sdk/keyvault/azure-keyvault-certificates/azure/keyvault/certificates/_generated/_key_vault_client.py

@@ -9,13 +9,22 @@
 # regenerated.
 # --------------------------------------------------------------------------
 
-from azure.core import PipelineClient
-from msrest import Serializer, Deserializer
+from typing import TYPE_CHECKING
 
+from azure.core import PipelineClient
 from azure.profiles import KnownProfiles, ProfileDefinition
 from azure.profiles.multiapiclient import MultiApiClientMixin
+from msrest import Deserializer, Serializer
+
 from ._configuration import KeyVaultClientConfiguration
 from ._operations_mixin import KeyVaultClientOperationsMixin
+
+if TYPE_CHECKING:
+    # pylint: disable=unused-import,ungrouped-imports
+    from typing import Any, Optional
+
+    from azure.core.pipeline.transport import HttpRequest, HttpResponse
+
 class _SDKClient(object):
     def __init__(self, *args, **kwargs):
         """This is a fake class to support current implemetation of MultiApiClientMixin."
@@ -33,8 +42,8 @@ class KeyVaultClient(KeyVaultClientOperationsMixin, MultiApiClientMixin, _SDKCli
     The profile sets a mapping between an operation group and its API version.
     The api-version parameter sets the default API version if the operation
     group is not described in the profile.
-    :param str api_version: API version to use if no profile is provided, or if
-     missing in profile.
+    :param api_version: API version to use if no profile is provided, or if missing in profile.
+    :type api_version: str
     :param profile: A profile definition, from KnownProfiles to dict.
     :type profile: azure.profiles.KnownProfiles
     """
@@ -50,14 +59,14 @@ class KeyVaultClient(KeyVaultClientOperationsMixin, MultiApiClientMixin, _SDKCli
 
     def __init__(
         self,
-        api_version=None,
-        profile=KnownProfiles.default,
+        api_version=None, # type: Optional[str]
+        profile=KnownProfiles.default, # type: KnownProfiles
         **kwargs  # type: Any
     ):
-        if api_version == '2016-10-01' or api_version == '7.0' or api_version == '7.1':
+        if api_version == '2016-10-01' or api_version == '7.0' or api_version == '7.1' or api_version == '7.2':
             base_url = '{vaultBaseUrl}'
         else:
-            raise NotImplementedError("APIVersion {} is not available".format(api_version))
+            raise ValueError("API version {} is not available".format(api_version))
         self._config = KeyVaultClientConfiguration(**kwargs)
         self._client = PipelineClient(base_url=base_url, config=self._config, **kwargs)
         super(KeyVaultClient, self).__init__(
@@ -76,6 +85,7 @@ def models(cls, api_version=DEFAULT_API_VERSION):
            * 2016-10-01: :mod:`v2016_10_01.models<azure.keyvault.v2016_10_01.models>`
            * 7.0: :mod:`v7_0.models<azure.keyvault.v7_0.models>`
            * 7.1: :mod:`v7_1.models<azure.keyvault.v7_1.models>`
+           * 7.2: :mod:`v7_2.models<azure.keyvault.v7_2.models>`
         """
         if api_version == '2016-10-01':
             from .v2016_10_01 import models
@@ -86,7 +96,10 @@ def models(cls, api_version=DEFAULT_API_VERSION):
         elif api_version == '7.1':
             from .v7_1 import models
             return models
-        raise NotImplementedError("APIVersion {} is not available".format(api_version))
+        elif api_version == '7.2':
+            from .v7_2 import models
+            return models
+        raise ValueError("API version {} is not available".format(api_version))
 
     def close(self):
         self._client.close()

sdk/keyvault/azure-keyvault-certificates/azure/keyvault/certificates/_generated/_operations_mixin.py

@@ -6,5 +6,5 @@
 # Changes may cause incorrect behavior and will be lost if the code is regenerated.
 # --------------------------------------------------------------------------
 
-from ._key_vault_client_async import KeyVaultClient
+from ._key_vault_client import KeyVaultClient
 __all__ = ['KeyVaultClient']

sdk/keyvault/azure-keyvault-certificates/azure/keyvault/certificates/_generated/aio/__init__.py

@@ -15,7 +15,6 @@
 
 from .._version import VERSION
 
-
 class KeyVaultClientConfiguration(Configuration):
     """Configuration for KeyVaultClient.
 

…s/_generated/aio/_configuration_async.py …ficates/_generated/aio/_configuration.pysdk/keyvault/azure-keyvault-certificates/azure/keyvault/certificates/_generated/aio/_configuration_async.py renamed to sdk/keyvault/azure-keyvault-certificates/azure/keyvault/certificates/_generated/aio/_configuration.py sdk/keyvault/azure-keyvault-certificates/azure/keyvault/certificates/_generated/aio/_configuration_async.py renamed to sdk/keyvault/azure-keyvault-certificates/azure/keyvault/certificates/_generated/aio/_configuration.py

@@ -9,13 +9,17 @@
 # regenerated.
 # --------------------------------------------------------------------------
 
-from azure.core import AsyncPipelineClient
-from msrest import Serializer, Deserializer
+from typing import Any, Optional
 
+from azure.core import AsyncPipelineClient
+from azure.core.pipeline.transport import AsyncHttpResponse, HttpRequest
 from azure.profiles import KnownProfiles, ProfileDefinition
 from azure.profiles.multiapiclient import MultiApiClientMixin
-from ._configuration_async import KeyVaultClientConfiguration
-from ._operations_mixin_async import KeyVaultClientOperationsMixin
+from msrest import Deserializer, Serializer
+
+from ._configuration import KeyVaultClientConfiguration
+from ._operations_mixin import KeyVaultClientOperationsMixin
+
 class _SDKClient(object):
     def __init__(self, *args, **kwargs):
         """This is a fake class to support current implemetation of MultiApiClientMixin."
@@ -33,8 +37,8 @@ class KeyVaultClient(KeyVaultClientOperationsMixin, MultiApiClientMixin, _SDKCli
     The profile sets a mapping between an operation group and its API version.
     The api-version parameter sets the default API version if the operation
     group is not described in the profile.
-    :param str api_version: API version to use if no profile is provided, or if
-     missing in profile.
+    :param api_version: API version to use if no profile is provided, or if missing in profile.
+    :type api_version: str
     :param profile: A profile definition, from KnownProfiles to dict.
     :type profile: azure.profiles.KnownProfiles
     """
@@ -50,14 +54,14 @@ class KeyVaultClient(KeyVaultClientOperationsMixin, MultiApiClientMixin, _SDKCli
 
     def __init__(
         self,
-        api_version=None,
-        profile=KnownProfiles.default,
+        api_version: Optional[str] = None,
+        profile: KnownProfiles = KnownProfiles.default,
         **kwargs  # type: Any
     ) -> None:
-        if api_version == '2016-10-01' or api_version == '7.0' or api_version == '7.1':
+        if api_version == '2016-10-01' or api_version == '7.0' or api_version == '7.1' or api_version == '7.2':
             base_url = '{vaultBaseUrl}'
         else:
-            raise NotImplementedError("APIVersion {} is not available".format(api_version))
+            raise ValueError("API version {} is not available".format(api_version))
         self._config = KeyVaultClientConfiguration(**kwargs)
         self._client = AsyncPipelineClient(base_url=base_url, config=self._config, **kwargs)
         super(KeyVaultClient, self).__init__(
@@ -76,6 +80,7 @@ def models(cls, api_version=DEFAULT_API_VERSION):
            * 2016-10-01: :mod:`v2016_10_01.models<azure.keyvault.v2016_10_01.models>`
            * 7.0: :mod:`v7_0.models<azure.keyvault.v7_0.models>`
            * 7.1: :mod:`v7_1.models<azure.keyvault.v7_1.models>`
+           * 7.2: :mod:`v7_2.models<azure.keyvault.v7_2.models>`
         """
         if api_version == '2016-10-01':
             from ..v2016_10_01 import models
@@ -86,7 +91,10 @@ def models(cls, api_version=DEFAULT_API_VERSION):
         elif api_version == '7.1':
             from ..v7_1 import models
             return models
-        raise NotImplementedError("APIVersion {} is not available".format(api_version))
+        elif api_version == '7.2':
+            from ..v7_2 import models
+            return models
+        raise ValueError("API version {} is not available".format(api_version))
 
     async def close(self):
         await self._client.close()