Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Use -Mail for test resources local user lookup to support TME #9529

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

benbp
Copy link
Member

@benbp benbp commented Dec 12, 2024

For TME accounts, the -UserPrincipalName does not match the email (e.g. [email protected]) and is instead like foobar_microsoft.com#EXT#@<tenant>), so the local user id/oid lookup is broken for TME testing. Using -Mail works as the value is the same across tenants.

So for local TME testing, one can do:

Set-AzContext -Subscription '<tme subscription>' -Tenant '<tme tenant>'

./eng/common/TestResources/New-TestResources.ps1 -ServiceDirectory <dir> -SubscriptionId '<tme subscription>'

At a certain point we will switch the default off playground and the -Subscription override won't be necessary, but we'll have to add more UX handling to help people make sure they are on the right az context.

CC @catalinaperalta

@benbp benbp requested a review from a team as a code owner December 12, 2024 20:48
@benbp benbp added the Central-EngSys This issue is owned by the Engineering System team. label Dec 12, 2024
@benbp benbp self-assigned this Dec 12, 2024
@benbp benbp requested review from weshaggard and scbedd December 12, 2024 20:48
@azure-sdk
Copy link
Collaborator

The following pipelines have been queued for testing:
java - template
java - template - tests
js - template
net - template
net - template - tests
python - template
python - template - tests
You can sign off on the approval gate to test the release stage of each pipeline.
See eng/common workflow

@azure-sdk
Copy link
Collaborator

The following pipelines have been queued for testing:
java - template
java - template - tests
js - template
net - template
net - template - tests
python - template
python - template - tests
You can sign off on the approval gate to test the release stage of each pipeline.
See eng/common workflow

@benbp benbp force-pushed the benbp/tme-local-test branch from 724b9bb to 0cda589 Compare December 13, 2024 00:22
@azure-sdk
Copy link
Collaborator

The following pipelines have been queued for testing:
java - template
java - template - tests
js - template
net - template
net - template - tests
python - template
python - template - tests
You can sign off on the approval gate to test the release stage of each pipeline.
See eng/common workflow

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Central-EngSys This issue is owned by the Engineering System team.
Projects
Status: 🔬 Dev in PR
Development

Successfully merging this pull request may close these issues.

3 participants