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

[Hub Generated] Publish private branch 'vaibmishra/onboardStorageResource' (Azure#23901)

* onboarding storage resource

* addressing failures from automated checks

* onboarding as a new resource provider instead of resoruce type

* addressing the checks

* addressign check fails x2

* addressing check fails x3

* resource onboarding in the same rp

* checks fail x1

* prettier fix

* swagger model validation check

* fix the pattern change

* support put, list, patch, delete api calls

* address checks x1

* put and delete api with 2xx code

* addressing check and restructure abmsi

* fixing checks again

* checks againnnn

* fixing PAYG and adding 201 for put

* try to fix lintdiff check

* check fails

* missed this file

* fixing the error response

* fixing the error response for bmi

Author: SDKAuto

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

@@ -1,11 +1,11 @@
 {
-  "commit": "7d6c14d986a67dca3451d7d92d8f6b9416d61fbf",
+  "commit": "99dad1e94c408d931172e35eef941e76837a6d76",
   "repository_url": "https://github.com/Azure/azure-rest-api-specs",
   "autorest": "3.9.2",
   "use": [
-    "@autorest/python@6.2.1",
+    "@autorest/python@6.4.8",
     "@autorest/modelerfour@4.24.3"
   ],
-  "autorest_command": "autorest specification/baremetalinfrastructure/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.2.1 --use=@autorest/modelerfour@4.24.3 --version=3.9.2 --version-tolerant=False",
+  "autorest_command": "autorest specification/baremetalinfrastructure/resource-manager/readme.md --generate-sample=True --include-x-ms-examples-original-file=True --python --python-sdks-folder=/mnt/vss/_work/1/s/azure-sdk-for-python/sdk --use=@autorest/python@6.4.8 --use=@autorest/modelerfour@4.24.3 --version=3.9.2 --version-tolerant=False",
   "readme": "specification/baremetalinfrastructure/resource-manager/readme.md"
 }

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

@@ -13,7 +13,7 @@
 
 try:
     from ._patch import __all__ as _patch_all
-    from ._patch import *  # type: ignore # pylint: disable=unused-wildcard-import
+    from ._patch import *  # pylint: disable=unused-wildcard-import
 except ImportError:
     _patch_all = []
 from ._patch import patch_sdk as _patch_sdk

sdk/baremetalinfrastructure/azure-mgmt-baremetalinfrastructure/azure/mgmt/baremetalinfrastructure/_bare_metal_infrastructure_client.py

@@ -12,10 +12,10 @@
 from azure.core.rest import HttpRequest, HttpResponse
 from azure.mgmt.core import ARMPipelineClient
 
-from . import models
+from . import models as _models
 from ._configuration import BareMetalInfrastructureClientConfiguration
 from ._serialization import Deserializer, Serializer
-from .operations import AzureBareMetalInstancesOperations, Operations
+from .operations import AzureBareMetalInstancesOperations, AzureBareMetalStorageInstancesOperations, Operations
 
 if TYPE_CHECKING:
     # pylint: disable=unused-import,ungrouped-imports
@@ -30,13 +30,16 @@ class BareMetalInfrastructureClient:  # pylint: disable=client-accepts-api-versi
      azure.mgmt.baremetalinfrastructure.operations.AzureBareMetalInstancesOperations
     :ivar operations: Operations operations
     :vartype operations: azure.mgmt.baremetalinfrastructure.operations.Operations
+    :ivar azure_bare_metal_storage_instances: AzureBareMetalStorageInstancesOperations operations
+    :vartype azure_bare_metal_storage_instances:
+     azure.mgmt.baremetalinfrastructure.operations.AzureBareMetalStorageInstancesOperations
     :param credential: Credential needed for the client to connect to Azure. Required.
     :type credential: ~azure.core.credentials.TokenCredential
     :param subscription_id: The ID of the target subscription. Required.
     :type subscription_id: str
     :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-08-09". Note that overriding this
+    :keyword api_version: Api Version. Default value is "2023-04-06". Note that overriding this
      default value may result in unsupported behavior.
     :paramtype api_version: str
     """
@@ -51,16 +54,19 @@ def __init__(
         self._config = BareMetalInfrastructureClientConfiguration(
             credential=credential, subscription_id=subscription_id, **kwargs
         )
-        self._client = ARMPipelineClient(base_url=base_url, config=self._config, **kwargs)
+        self._client: ARMPipelineClient = ARMPipelineClient(base_url=base_url, config=self._config, **kwargs)
 
-        client_models = {k: v for k, v in models.__dict__.items() if isinstance(v, type)}
+        client_models = {k: v for k, v in _models.__dict__.items() if isinstance(v, type)}
         self._serialize = Serializer(client_models)
         self._deserialize = Deserializer(client_models)
         self._serialize.client_side_validation = False
         self.azure_bare_metal_instances = AzureBareMetalInstancesOperations(
             self._client, self._config, self._serialize, self._deserialize
         )
         self.operations = Operations(self._client, self._config, self._serialize, self._deserialize)
+        self.azure_bare_metal_storage_instances = AzureBareMetalStorageInstancesOperations(
+            self._client, self._config, self._serialize, self._deserialize
+        )
 
     def _send_request(self, request: HttpRequest, **kwargs: Any) -> HttpResponse:
         """Runs the network request through the client's chained policies.
@@ -84,15 +90,12 @@ def _send_request(self, request: HttpRequest, **kwargs: Any) -> HttpResponse:
         request_copy.url = self._client.format_url(request_copy.url)
         return self._client.send_request(request_copy, **kwargs)
 
-    def close(self):
-        # type: () -> None
+    def close(self) -> None:
         self._client.close()
 
-    def __enter__(self):
-        # type: () -> BareMetalInfrastructureClient
+    def __enter__(self) -> "BareMetalInfrastructureClient":
         self._client.__enter__()
         return self
 
-    def __exit__(self, *exc_details):
-        # type: (Any) -> None
+    def __exit__(self, *exc_details: Any) -> None:
         self._client.__exit__(*exc_details)

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

@@ -6,7 +6,6 @@
 # Changes may cause incorrect behavior and will be lost if the code is regenerated.
 # --------------------------------------------------------------------------
 
-import sys
 from typing import Any, TYPE_CHECKING
 
 from azure.core.configuration import Configuration
@@ -15,11 +14,6 @@
 
 from ._version import VERSION
 
-if sys.version_info >= (3, 8):
-    from typing import Literal  # pylint: disable=no-name-in-module, ungrouped-imports
-else:
-    from typing_extensions import Literal  # type: ignore  # pylint: disable=ungrouped-imports
-
 if TYPE_CHECKING:
     # pylint: disable=unused-import,ungrouped-imports
     from azure.core.credentials import TokenCredential
@@ -35,14 +29,14 @@ class BareMetalInfrastructureClientConfiguration(Configuration):  # pylint: disa
     :type credential: ~azure.core.credentials.TokenCredential
     :param subscription_id: The ID of the target subscription. Required.
     :type subscription_id: str
-    :keyword api_version: Api Version. Default value is "2021-08-09". Note that overriding this
+    :keyword api_version: Api Version. Default value is "2023-04-06". Note that overriding this
      default value may result in unsupported behavior.
     :paramtype api_version: str
     """
 
     def __init__(self, credential: "TokenCredential", subscription_id: str, **kwargs: Any) -> None:
         super(BareMetalInfrastructureClientConfiguration, self).__init__(**kwargs)
-        api_version = kwargs.pop("api_version", "2021-08-09")  # type: Literal["2021-08-09"]
+        api_version: str = kwargs.pop("api_version", "2023-04-06")
 
         if credential is None:
             raise ValueError("Parameter 'credential' must not be None.")
@@ -56,10 +50,7 @@ def __init__(self, credential: "TokenCredential", subscription_id: str, **kwargs
         kwargs.setdefault("sdk_moniker", "mgmt-baremetalinfrastructure/{}".format(VERSION))
         self._configure(**kwargs)
 
-    def _configure(
-        self, **kwargs  # type: Any
-    ):
-        # type: (...) -> None
+    def _configure(self, **kwargs: Any) -> None:
         self.user_agent_policy = kwargs.get("user_agent_policy") or policies.UserAgentPolicy(**kwargs)
         self.headers_policy = kwargs.get("headers_policy") or policies.HeadersPolicy(**kwargs)
         self.proxy_policy = kwargs.get("proxy_policy") or policies.ProxyPolicy(**kwargs)