Skip to content

Security: use a new separated managed identity in the SDK validation check and the SDK breaking change label check #39448

Merged
raych1 merged 2 commits intoAzure:mainfrom
raych1:updated-app-registration
Jan 6, 2026
Merged

Security: use a new separated managed identity in the SDK validation check and the SDK breaking change label check #39448
raych1 merged 2 commits intoAzure:mainfrom
raych1:updated-app-registration

Conversation

@raych1
Copy link
Member

@raych1 raych1 commented Jan 5, 2026

Resolved #38974

I've already granted read permission to this app registration for azure-sdk/internal project.

@raych1 raych1 self-assigned this Jan 5, 2026
@github-actions
Copy link

github-actions bot commented Jan 5, 2026

Next Steps to Merge

Next steps that must be taken to merge this PR:
  • ❌ The required check named Swagger Avocado has failed. Refer to the check in the PR's 'Checks' tab for details on how to fix it and consult the aka.ms/ci-fix guide


Comment generated by summarize-checks workflow run.

@raych1 raych1 moved this from 🤔 Triage to 🔬 Dev in PR in Azure SDK EngSys 📆🎇 Jan 5, 2026
uses: azure/login@v2
with:
client-id: "936c56f0-298b-467f-b702-3ad5bf4b15c1"
client-id: "34bd541d-1803-437e-b663-da8114650466"
Copy link
Member

Choose a reason for hiding this comment

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

Let's be sure this identity is a MI instead of App registration.

Copy link
Member Author

Choose a reason for hiding this comment

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

Updated it to use managed identity.

@raych1
Copy link
Member Author

raych1 commented Jan 6, 2026

Validated with this run in forked repo.

@raych1 raych1 merged commit b212d0d into Azure:main Jan 6, 2026
44 of 46 checks passed
@raych1 raych1 changed the title Updated to use a new app registration Security: use a new separated managed identity in the SDK validation check and the SDK breaking change label check Jan 6, 2026
@kurtzeborn kurtzeborn moved this from 🔬 Dev in PR to 🎊 Closed in Azure SDK EngSys 📆🎇 Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

[SDK Validation] Use separated managed identity to access ADO pipeline

3 participants