Skip to content

Resolve principalId inside diagnostic module to avoid cross-scope reference error#762

Merged
tjementum merged 1 commit into
mainfrom
resolve-sql-managedidentity-principal-id
Jul 4, 2025
Merged

Resolve principalId inside diagnostic module to avoid cross-scope reference error#762
tjementum merged 1 commit into
mainfrom
resolve-sql-managedidentity-principal-id

Conversation

@bluebasher
Copy link
Copy Markdown
Contributor

Summary & Motivation

Resolve principalId inside the diagnostics module to eliminate the cross-scope reference() error during deployment.
This fixes this cloud infrastructure error.

  • Remove the principalId parameter from the diagnostics module interface.
  • Retrieve the principalId internally within the module, ensuring all references stay within the same scope.
  • Update calling templates to reflect the revised module signature.

Checklist

  • I have added tests, or done manual regression tests
  • I have updated the documentation, if necessary

@bluebasher bluebasher self-assigned this Jul 3, 2025
@bluebasher bluebasher requested a review from a team as a code owner July 3, 2025 17:14
@bluebasher bluebasher added the Bug Something isn't working label Jul 3, 2025
@tjementum tjementum merged commit a85f0ea into main Jul 4, 2025
6 checks passed
@tjementum tjementum deleted the resolve-sql-managedidentity-principal-id branch July 4, 2025 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants