Repository navigation
New Resource Provider - Microsoft.AIInfraOptimization - #47287
sakshamgargMS wants to merge 4 commits into
Conversation
…view) Ports Azure/azure-rest-api-specs-pr#29872 (merge commit e28667b52ed73ea6d86715cdcc1a61480da34189) to main. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: c2cd748b-6652-4ee5-8a03-32be4f02aa23
Next Steps to MergeNext steps that must be taken to merge this PR:
Comment generated by summarize-checks workflow run. |
Contributor readiness: required accessWarning Access issues can block approvals, Azure DevOps pipelines, or labeling.
Internal contributors: set up / renew access. Recheck: Contributors with verified access (2)Public Azure membership and repository write access verified.
Non-blocking check; fix access issues to unblock affected actions. Last updated: 2026-10-19T13:01:57.682Z |
Package Name Review RequiredPlane: Management Plane
How to approve:
Merge is blocked until all languages are approved. Approver list: .github/protected-labels.yml |
API Change CheckAPIView identified API level changes in this PR and created the following API reviews
Comment generated by After APIView workflow run. |
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: c2cd748b-6652-4ee5-8a03-32be4f02aa23
ARM API Review Summary\n\nResult: Changes requested\n\nScope and revision: Full review of PR #47287 at pinned head
|
There was a problem hiding this comment.
ARM API review completed for pinned head 1cebe29062cab838573b850bc8078ec7d1c7829b (full scope). The independent ARM API Review Critic verified the two inline blocking findings: both are new, technically justified TypeSpec suppressions that lack the required Approved-TypeSpecSuppression approval context. No additional blocking API, security, LRO, paging, examples, or operations-contract issues were identified. Please obtain the required approval or remove/remodel the suppressions.
🔍 ARM API review by ARM API Review: Automated Workflow
|
The provider level action is already gone through ARM modeling office hours, and same changes are already present in RPSaaSMaster branch of pr repo. Hence approved the suppressions. |
|
/azsdk check-access |
|
Arthur Ma (@ArthurMa1978) m-nash JoshLove-msft could you please review the management-plane package names for the new
All follow the standard ARM naming pattern. ARM sign-off and TypeSpec suppression approval are complete; package-name approval ( |
✅ Package Name ApprovedAll required package name approvals received. This PR is clear to merge from a package name perspective. |
|
All package-name approvals are now present. Python package name azure-mgmt-aiinfraoptimization has been reserved through the official pipeline (https://dev.azure.com/azure-sdk/internal/_build/results?buildId=6972222), Python SDK validation passed on retry, and the refreshed SDK Validation Status and Next Steps to Merge report success. Venu Sivanadham (@venu-sivanadham) could you please provide the approving PR review required for merge? ARM sign-off and TypeSpec suppression approvals are already recorded. GitHub still reports CHANGES_REQUESTED from the earlier automated review about those suppressions; could a maintainer also resolve that stale review through the normal review process? We have not bypassed any review gates. |
The two findings requested approval for TypeSpec suppressions. Venu confirmed the ARM modeling review and approved the suppressions on 2026-10-16; typespec-suppressions-approved is present and validation passes. Venu formally approved this PR on 2026-10-19. Dismissing the superseded automated review; no API or code findings remain from that review.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 2a026d05-114e-4177-be18-fcb2a68e98dc
ARM (Control Plane) API Specification Update Pull Request
Tip
Overwhelmed by all this guidance? See the
Getting helpsection 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.
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:
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.
Additional information
This PR publishes the spec merged in the private repo via Azure/azure-rest-api-specs-pr#29872 (merge commit
e28667b52ed73ea6d86715cdcc1a61480da34189onRPSaaSMaster). All files underspecification/aiinfraoptimization/are byte-identical to that merge commit. The only difference is the CODEOWNERS entry, which additionally lists sakshamgargMS.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 ApiViewcomment 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 to get approval.
Getting help
Purpose of this PRandDue diligence checklist.write accessper aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositoriesNext Steps to Mergecomment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.and https://aka.ms/ci-fix.
queuedstate, please add a comment with contents/azp run.This should result in a new comment denoting a
PR validation pipelinehas started and the checks should be updated after few minutes.