Skip to content

Conversation

@NirajC-Microsoft
Copy link
Contributor

Purpose

This pull request updates the logic for setting the createdBy tag in the infra/main.bicep file. The main improvement is to make the assignment of the createdBy parameter more robust by handling cases where userPrincipalName may not be present in the deployer object.

Tag assignment improvement:

  • Updated the createdBy parameter to check if userPrincipalName exists in the deployer() object. If it does, it uses the username portion before the @; otherwise, it falls back to using the objectId of the deployer. This ensures the tag is always set with meaningful information, even if the user principal name is missing.
  • ...

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

fix: CI Pipeline Validate Deployment - MACAE
@Roopan-Microsoft Roopan-Microsoft merged commit 85ec07d into macae-v2 Sep 23, 2025
8 checks passed
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.

4 participants