Skip to content

Commit 3997f57

Browse files
authored
Merge pull request Azure#6327 from bmoore-msft/cicd
pipeline script updates
2 parents 9b4d6a6 + f7f9e58 commit 3997f57

5 files changed

+745
-6
lines changed

.gitignore

+2
Original file line numberDiff line numberDiff line change
@@ -35,3 +35,5 @@ test/template-validation-tests/mochawesome-report/*
3535

3636
test/ci-scripts/.config.json
3737
test/AzTemplateToolKit.zip
38+
test/pipeline/QuickStarts-Az-CI-Public-Export.json
39+
test/pipeline/QuickStarts-Az-CI-Public-Import.json

test/ci-gen-setup/Create-GEN-Artifacts.ps1

+1-1
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ Creat a KeyVault and add:
4545
# Create the Vault
4646
$vault = Get-AzureRMKeyVault -VaultName $KeyVaultName -verbose -ErrorAction SilentlyContinue
4747
if($vault -eq $null) {
48-
$vault = New-AzureRMKeyVault -VaultName $KeyVaultName -ResourceGroupName $ResourceGroupName -Location $Location -EnabledForTemplateDeployment -EnabledForDiskEncryption
48+
$vault = New-AzureRMKeyVault -VaultName $KeyVaultName -ResourceGroupName $ResourceGroupName -Location $Location -EnabledForTemplateDeployment -EnabledForDiskEncryption -Verbose
4949
}
5050

5151
# 1) Create a sample password

test/ci-scripts/Create-ResourceGroupAssignment.ps1

+2-4
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,7 @@
55
param(
66
[string][Parameter(mandatory=$true)] $ResourceGroupName,
77
[string][Parameter(mandatory=$true)] $Location,
8-
[string][Parameter(mandatory=$true)] $appId,
9-
[string][Parameter(mandatory=$true)] $objectId #TODO this is a workaround until we can figure out why Get-AzADSP failes and role assignment partially fails
8+
[string][Parameter(mandatory=$true)] $appId
109
)
1110

1211
#Create the group only if it doesn't already exist
@@ -15,5 +14,4 @@ if ((Get-AzResourceGroup -Name $ResourceGroupName -Location $Location -Verbose -
1514
}
1615

1716
#Note that the service principal assigning the role must have AAD perms to query AD for the objectId
18-
#New-AzRoleAssignment -ObjectId $(Get-AzADServicePrincipal -ApplicationId $appId).Id -RoleDefinitionName Contributor -ResourceGroupName $ResourceGroupName -Verbose
19-
New-AzRoleAssignment -ObjectId $objectId -RoleDefinitionName Contributor -ResourceGroupName $ResourceGroupName -Verbose
17+
New-AzRoleAssignment -ObjectId $(Get-AzADServicePrincipal -ApplicationId $appId).Id -RoleDefinitionName Contributor -ResourceGroupName $ResourceGroupName -Verbose

test/ci-scripts/Get-SampleFolder.ps1

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ If the PR does not contain changes to a sample folder, it will currently fail bu
55
pass the build in order to trigger a manual review
66
#>
77

8-
Get-ChildItem env: # debugging
8+
# Get-ChildItem env: # debugging
99

1010
$GitHubRepository = $ENV:BUILD_REPOSITORY_NAME
1111
$GitHubPRNumber = $ENV:SYSTEM_PULLREQUEST_PULLREQUESTNUMBER

0 commit comments

Comments
 (0)