Skip to content

updating vault properties#3872

Merged
Chakradhar886 merged 2 commits intomainfrom
v-cchakradha/fix_kv_issue
Apr 22, 2026
Merged

updating vault properties#3872
Chakradhar886 merged 2 commits intomainfrom
v-cchakradha/fix_kv_issue

Conversation

@Chakradhar886
Copy link
Copy Markdown
Member

@Chakradhar886 Chakradhar886 commented Apr 14, 2026

Description

This PR explicitly sets:

enable_rbac_authorization=False

while creating the Azure Key Vault.

Why this change is needed:

Recent failures were observed after upgrading the Azure Management Key Vault SDK from v13 to v14.
Although public documentation suggests that RBAC authorization defaults remain unchanged when the value is not specified, actual behavior changed in v14.
As confirmed during discussion and a local PoC:

In SDK v13, Key Vault creation worked without explicitly setting enable_rbac_authorization
In SDK v14, when this value is not explicitly set, Key Vault access fails during runtime
Explicitly setting enable_rbac_authorization=False restores the earlier, working behavior

Jayesh validated this by running a local PoC comparing SDK v13 vs v14 behavior, and confirmed that explicitly disabling RBAC authorization is the correct and minimal fix.

Checklist

  • I have read the contribution guidelines.
  • I have coordinated with the docs team (mldocs@microsoft.com) if this PR deletes files or changes any file names or file extensions.
  • Pull request includes test coverage for the included changes.
  • This notebook or file is added to the CODEOWNERS file, pointing to the author or the author's team.

@Chakradhar886 Chakradhar886 requested a review from a team as a code owner April 14, 2026 06:24
Copy link
Copy Markdown
Contributor

@sdgilley sdgilley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

docs approval

@lavakumarrepala lavakumarrepala enabled auto-merge (squash) April 22, 2026 03:38
@Chakradhar886
Copy link
Copy Markdown
Member Author

@Chakradhar886 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.

@microsoft-github-policy-service agree [company="{your company}"]

Options:

  • (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@microsoft-github-policy-service agree
  • (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

auto-merge was automatically disabled April 22, 2026 06:17

Pull request was closed

@Chakradhar886 Chakradhar886 reopened this Apr 22, 2026
@Chakradhar886
Copy link
Copy Markdown
Member Author

@microsoft-github-policy-service agree company="Microsoft"

@Chakradhar886 Chakradhar886 merged commit bb4b724 into main Apr 22, 2026
25 checks passed
@Chakradhar886 Chakradhar886 deleted the v-cchakradha/fix_kv_issue branch April 22, 2026 07:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants