Skip to content

Improve Microsoft Graph module installation in AAD cleanup job#5322

Merged
mikaelweave merged 2 commits intomainfrom
personal/mikaelw/cleanup-aad-graph-module-fix
Jan 10, 2026
Merged

Improve Microsoft Graph module installation in AAD cleanup job#5322
mikaelweave merged 2 commits intomainfrom
personal/mikaelw/cleanup-aad-graph-module-fix

Conversation

@mikaelweave
Copy link
Contributor

@mikaelweave mikaelweave commented Jan 9, 2026

Description

Improves the Microsoft Graph module installation in the AAD cleanup build job:

  • Refactors module installation to use a version variable for easier maintenance
  • Adds conditional installation to skip modules that are already installed
  • Adds explicit module imports with error handling
  • Adds -NoWelcome flag to Connect-MgGraph to suppress the welcome message
  • Adds -Scope CurrentUser and -AllowClobber flags for more reliable installation

AB#180395

This fixes the below pipeline issue that started earlier today.

CleanShot 2026-01-09 at 15 47 32@2x

Related issues

N/A - Build improvement

Testing

This change was tested in PR pipeline run for this branch.

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.
  • When changing or adding behavior, if your code modifies the system design or changes design assumptions, please create and include an ADR.
  • CI is green before merge Build Status
  • Review squash-merge requirements

Semver Change (docs)

Patch

@mikaelweave mikaelweave requested a review from a team as a code owner January 9, 2026 23:28
@mikaelweave mikaelweave added this to the FY26\Q2\2Wk\2Wk13 milestone Jan 9, 2026
@mikaelweave mikaelweave added Bug Bug bug bug. Open source This change is only relevant to the OSS code or release. No-PaaS-breaking-change labels Jan 9, 2026
@mikaelweave mikaelweave enabled auto-merge (squash) January 9, 2026 23:50
@mikaelweave mikaelweave merged commit 9444ee9 into main Jan 10, 2026
63 of 68 checks passed
@mikaelweave mikaelweave deleted the personal/mikaelw/cleanup-aad-graph-module-fix branch January 10, 2026 00:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Bug bug bug. No-PaaS-breaking-change Open source This change is only relevant to the OSS code or release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants