Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Migrate Keyvault to Azure RBAC #3602

Merged
merged 26 commits into from
Nov 27, 2023
Merged

Conversation

PTaladay
Copy link
Contributor

Description

Access Policies are legacy and were having issues in PR pipeline. Switched to using Azure RBAC.1

Related issues

Testing

Ran in both PR and CI pipelines to validate changes.

FHIR Team Checklist

  • Update the title of the PR to be succinct and less than 65 characters
  • Add a milestone to the PR for the sprint that it is merged (i.e. add S47)
  • Tag the PR with the type of update: Bug, Build, Dependencies, Enhancement, New-Feature or Documentation
  • Tag the PR with Open source, Azure API for FHIR (CosmosDB or common code) or Azure Healthcare APIs (SQL or common code) to specify where this change is intended to be released.
  • Tag the PR with Schema Version backward compatible or Schema Version backward incompatible or Schema Version unchanged if this adds or updates Sql script which is/is not backward compatible with the code.
  • CI is green before merge Build Status
  • Review squash-merge requirements

Semver Change (docs)

Patch|Skip|Feature|Breaking (reason)

…to Key Vault Secrets Officer. Removing ms graph install since the AzurePowershell task installs Azure Az library.
@PTaladay PTaladay added the Build label Nov 27, 2023
@PTaladay PTaladay added this to the S129 milestone Nov 27, 2023
@PTaladay PTaladay requested a review from a team as a code owner November 27, 2023 16:43
@PTaladay PTaladay merged commit 8d991a1 into main Nov 27, 2023
@PTaladay PTaladay deleted the personal/patalada/PRKeyvaultFailureFix branch November 27, 2023 19:19
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants