Skip to content

Sync eng/common directory with azure-sdk-tools for PR 7580 #22292

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.

Sign up for GitHub

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

Merged
azure-sdk merged 6 commits into from
Jan 24, 2024
Merged

Sync eng/common directory with azure-sdk-tools for PR 7580 #22292

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
be3b81b
Support creating resources with user auth
JoshLove-msft Jan 24, 2024
24d3349
Log warning if TestApplicationId is set
JoshLove-msft Jan 24, 2024
204dcb0
missing space
JoshLove-msft Jan 24, 2024
138d60f
regenerate md file
JoshLove-msft Jan 24, 2024
e45cb65
Rename
JoshLove-msft Jan 24, 2024
fa9a206
Update link
JoshLove-msft Jan 24, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Support creating resources with user auth
  • Loading branch information
@JoshLove-msft @azure-sdk
JoshLove-msft authored and azure-sdk committed Jan 24, 2024
commit be3b81b11f6602d241a025dee554e78bd28ca965
19 changes: 18 additions & 1 deletion eng/common/TestResources/New-TestResources.ps1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -92,6 +92,9 @@ param (
[Parameter()]
[switch] $SuppressVsoCommands = ($null -eq $env:SYSTEM_TEAMPROJECTID),

[Parameter()]
[switch] $UseUserCredentials,

# Captures any arguments not declared here (no parameter errors)
# This enables backwards compatibility with old script versions in
# hotfix branches if and when the dynamic subscription configuration
Expand Down Expand Up @@ -611,8 +614,14 @@ try {
}
}


if ($UseUserCredentials) {
$TestApplicationOid = (Get-AzADUser -UserPrincipalName (Get-AzContext).Account).Id
$TestApplicationId = $testApplicationOid
Log "User-based app id '$TestApplicationId' will be used."
}
# If no test application ID was specified during an interactive session, create a new service principal.
if (!$CI -and !$TestApplicationId) {
elseif (!$CI -and !$TestApplicationId) {
# Cache the created service principal in this session for frequent reuse.
$servicePrincipal = if ($AzureTestPrincipal -and (Get-AzADServicePrincipal -ApplicationId $AzureTestPrincipal.AppId) -and $AzureTestSubscription -eq $SubscriptionId) {
Log "TestApplicationId was not specified; loading cached service principal '$($AzureTestPrincipal.AppId)'"
Expand Down Expand Up @@ -1001,6 +1010,14 @@ The environment file will be named for the test resources template that it was
generated for. For ARM templates, it will be test-resources.json.env. For
Bicep templates, test-resources.bicep.env.

.PARAMETER UseUserCredentials
Create the resource group and deploy the template using the signed in user's credentials.
No service principal will be created or used.

The environment file will be named for the test resources template that it was
generated for. For ARM templates, it will be test-resources.json.env. For
Bicep templates, test-resources.bicep.env.

.PARAMETER SuppressVsoCommands
By default, the -CI parameter will print out secrets to logs with Azure Pipelines log
commands that cause them to be redacted. For CI environments that don't support this (like
Expand Down