Azure / azure-rest-api-specs

The source for REST API specifications for Microsoft Azure.
MIT License
2.68k stars 5.11k forks source link

add ManagedIdentitySettings to session pool 2024-10-02-preview #30771

Closed najian closed 4 weeks ago

najian commented 1 month ago

ARM (Control Plane) API Specification Update Pull Request

[!TIP] Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram

Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

spec_pr_review_workflow_diagram

Purpose of this PR

What's the purpose of this PR? Check the specific option that applies. This is mandatory!

Due diligence checklist

To merge this PR, you must go through the following checklist and confirm you understood and followed the instructions by checking all the boxes:

Additional information

Viewing API changes For convenient view of the API changes made by this PR, refer to the URLs provided in the table in the `Generated ApiView` comment added to this PR. You can use ApiView to show API versions diff.
Suppressing failures If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the [suppressions guide](https://aka.ms/azsdk/pr-suppressions) to get approval.

Getting help

openapi-pipeline-app[bot] commented 1 month ago

Next Steps to Merge

✅ All automated merging requirements have been met! To get your PR merged, see aka.ms/azsdk/specreview/merge.
openapi-pipeline-app[bot] commented 1 month ago

Generated ApiView

Language Package Name ApiView Link
JavaScript @azure/arm-appcontainers https://apiview.dev/Assemblies/Review/ba41cd9978b542efb0688a072b2319a1?revisionId=76f0db4044bf4ebcb7cd3893a9b07797
Java azure-resourcemanager-appcontainers https://apiview.dev/Assemblies/Review/7da9ac1e2db64477ada2ba8bbb71169b?revisionId=80b4af670b2143c1839407d54105848a
Swagger Microsoft.App https://apiview.dev/Assemblies/Review/53999dbad23b45cba7812a00a713cab8?revisionId=6dc2902475b743c580af69048949dc7e
ms-bogdan-rotaru commented 1 month ago

/azp run

azure-pipelines[bot] commented 1 month ago
Azure Pipelines successfully started running 4 pipeline(s).
ms-bogdan-rotaru commented 1 month ago

Please fix the errors reported by Swagger PrettierCheck.

mikeharder commented 1 month ago

/azp run

azure-pipelines[bot] commented 1 month ago
Azure Pipelines successfully started running 4 pipeline(s).
rkmanda commented 1 month ago

Pl fix the errors here - seems like there is a parsing error for the open api spec json https://github.com/Azure/azure-rest-api-specs/pull/30771/checks?check_run_id=30846335360

For the patch... rule that is breaking please add a suppression..