-
Notifications
You must be signed in to change notification settings - Fork 5.5k
Stable version 2025-09-01 with prevalidation and autoscale changes #37218
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Stable version 2025-09-01 with prevalidation and autoscale changes #37218
Conversation
Next Steps to MergeNext steps that must be taken to merge this PR:
Comment generated by summarize-checks workflow run. |
Thank you for your contribution @prachinandi! We will review the pull request and get back to you soon. |
API Change CheckAPIView identified API level changes in this PR and created the following API reviews
|
specification/elasticsan/resource-manager/Microsoft.ElasticSan/ElasticSan/readme.md
Show resolved
Hide resolved
...oft.ElasticSan/ElasticSan/stable/2025-09-01/examples/VolumeGroups_Create_MaximumSet_Gen.json
Outdated
Show resolved
Hide resolved
specification/elasticsan/resource-manager/Microsoft.ElasticSan/ElasticSan/back-compatible.tsp
Outdated
Show resolved
Hide resolved
* The 2025-09-01 stable API version. | ||
*/ | ||
@armCommonTypesVersion(Azure.ResourceManager.CommonTypes.Versions.v3) | ||
v2025_09_01: "2025-09-01", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems you need @previewVersion
if you want to put the new version before old version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have a second thought, I'm not sure if we could do version in this order. @markcowl could we?
0600a60
to
821ee8a
Compare
/azp run |
Commenter does not have sufficient privileges for PR 37218 in repo Azure/azure-rest-api-specs |
1 similar comment
Commenter does not have sufficient privileges for PR 37218 in repo Azure/azure-rest-api-specs |
Bunch of automated merging requirements failing. Please fix them and resubmit. |
@prachinandi please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
Contributor License AgreementContribution License AgreementThis Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
|
…://github.com/prachinandi/azure-rest-api-specs into dev/prachinandi/elasticsan_prevalidation_apis
@psah434 can you please review and approve this PR now? I have fixed the pre merge checks. The breaking change error is expected to happen I think, because we have made breaking changes in our latest version. Thanks! |
Choose a PR Template
Switch to "Preview" on this description then select one of the choices below.
Click here to open a PR for a Data Plane API.
Click here to open a PR for a Control Plane (ARM) API.
Click here to open a PR for only SDK configuration.