feature: Registering Microsoft.ContainerService/IdentityBindingPreview - #6322
feature: Registering Microsoft.ContainerService/IdentityBindingPreview#6322rachelvweber wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the subscription provider feature-registration configuration to include the Azure preview feature Microsoft.ContainerService/IdentityBindingPreview, and regenerates the rendered dev environment configs so the feature is applied in those deployments.
Changes:
- Added
IdentityBindingPreviewto theMicrosoft.ContainerServicefeature list under the SVC subscription provider defaults (config/config.yaml). - Propagated the change into the rendered dev configs for westus3 and centralus.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| config/config.yaml | Adds IdentityBindingPreview to the default SVC subscription provider feature registrations for Microsoft.ContainerService. |
| config/rendered/dev/pers/westus3.yaml | Rendered output includes the new IdentityBindingPreview feature under Microsoft.ContainerService. |
| config/rendered/dev/perf/westus3.yaml | Rendered output includes the new IdentityBindingPreview feature under Microsoft.ContainerService. |
| config/rendered/dev/dev/westus3.yaml | Rendered output includes the new IdentityBindingPreview feature under Microsoft.ContainerService. |
| config/rendered/dev/cspr/westus3.yaml | Rendered output includes the new IdentityBindingPreview feature under Microsoft.ContainerService. |
| config/rendered/dev/ci01/centralus.yaml | Rendered output includes the new IdentityBindingPreview feature under Microsoft.ContainerService. |
| config/rendered/dev/ci00/centralus.yaml | Rendered output includes the new IdentityBindingPreview feature under Microsoft.ContainerService. |
|
/retest-required |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: rachelvweber, weinong The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
27bdbf4 to
df4de88
Compare
|
@rachelvweber: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
ARO-28745
What
Registers Microsoft.ContainerService/IdentityBindingPreview preview feature
Why
This is needed for the quota management app quotaizer because the app uses a global identity for managing quota groups at the management group level.
Testing
Ran the e2e tests
Special notes for your reviewer
PR Checklist
If E2E tests are included:
demonstrate that the test is able to detect a defect/error and fail with
proper error message and logs which communicates nature of the problem.