Skip to content

Conversation

dwmsft
Copy link
Member

@dwmsft dwmsft commented Sep 29, 2025

SDK configuration pull request

Purpose of this PR

  • Make changes to the SDK configuration only when there are no modifications to the API specification, eliminating the need for an ARM or Stewardship Board API review.

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:

Getting help

  • First, carefully read through this PR description, from top to bottom. Fill out the Purpose of this PR and Due diligence checklist.
  • If you don't have permissions to remove or add labels to the PR, request write access per aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories
  • To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
  • For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure and https://aka.ms/ci-fix.
  • If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
    This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
  • If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

@dwmsft dwmsft requested a review from naveedaz as a code owner September 29, 2025 16:46
Copy link

github-actions bot commented Sep 29, 2025

Next Steps to Merge

Next steps that must be taken to merge this PR:
  • ❌ This PR is NotReadyForARMReview because it has the CI-NewRPNamespaceWithoutRPaaS label.
  • ❌ This PR has CI-NewRPNamespaceWithoutRPaaS label. This means it is introducing a new RP (Resource Provider) namespace that has not been onboarded with RPaaS. Merging this PR to the main branch is blocked as RPaaS is required for new RPs.
    To resolve, use RPaaS to onboard the new RP. To apply for exception, see aka.ms/RPaaSException.


Comment generated by summarize-checks workflow run.

@github-actions github-actions bot added BreakingChange-JavaScript-Sdk BreakingChangeReviewRequired <valid label in PR review process>add this label when breaking change review is required labels Sep 29, 2025
@dwmsft dwmsft added the ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review label Sep 29, 2025
@github-actions github-actions bot removed the ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review label Sep 29, 2025
@dwmsft
Copy link
Member Author

dwmsft commented Sep 29, 2025

This PR is to get #37232 checked into main

@JeffreyRichter JeffreyRichter added the BreakingChange-Approved-Benign Changes are not breaking at the REST API level and have at most minor impact to generated SDKs. label Sep 29, 2025
@ravimeda ravimeda added the ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review label Sep 30, 2025
@github-actions github-actions bot removed the ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review label Sep 30, 2025
@dwmsft dwmsft added PublishToCustomers Acknowledgement the changes will be published to Azure customers. and removed NotReadyForARMReview labels Sep 30, 2025
@ravimeda ravimeda added the RPaaSException https://github.com/Azure/azure-sdk-tools/issues/5311#issue-1567009479 label Sep 30, 2025
@github-actions github-actions bot removed the RPaaSException https://github.com/Azure/azure-sdk-tools/issues/5311#issue-1567009479 label Sep 30, 2025
@ravimeda
Copy link

@suhasaraos please any suggestions on how to unblock the merge. @dwmsft got an exception from Evan.

@raosuhas
Copy link

@suhasaraos please any suggestions on how to unblock the merge. @dwmsft got an exception from Evan.

you can just remove the CI-NewRPNamespaceWithoutRPaaS tag. I removed for you

@raosuhas raosuhas added the RPaaSException https://github.com/Azure/azure-sdk-tools/issues/5311#issue-1567009479 label Sep 30, 2025
@github-actions github-actions bot removed the RPaaSException https://github.com/Azure/azure-sdk-tools/issues/5311#issue-1567009479 label Sep 30, 2025
@raosuhas raosuhas added RPaaSException https://github.com/Azure/azure-sdk-tools/issues/5311#issue-1567009479 and removed CI-NewRPNamespaceWithoutRPaaS labels Sep 30, 2025
@github-actions github-actions bot added CI-NewRPNamespaceWithoutRPaaS and removed RPaaSException https://github.com/Azure/azure-sdk-tools/issues/5311#issue-1567009479 labels Sep 30, 2025
@raosuhas
Copy link

@suhasaraos please any suggestions on how to unblock the merge. @dwmsft got an exception from Evan.

you can just remove the CI-NewRPNamespaceWithoutRPaaS tag. I removed for you

@mikeharder seems like the CI-NewRPNamespaceWithoutRPaaS does not respect the RPaaSException tag , could you please check ?

@mikeharder mikeharder merged commit 2caaf9e into main Oct 1, 2025
198 of 209 checks passed
@mikeharder mikeharder deleted the release-web-2025-05-01 branch October 1, 2025 00:09
@mikeharder
Copy link
Member

merged on behalf of @raosuhas

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants