Skip to content

Commit c1f3be2

Browse files
author
Sheyla Trudo
committed
fixup! Artifacts
1 parent f8ccbfc commit c1f3be2

File tree

2 files changed

+15
-13
lines changed

2 files changed

+15
-13
lines changed

.pipelines/templates/artifact-storage.steps.yaml

Lines changed: 15 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
steps:
33

44
- task: AzureCLI@2
5+
name: build
56
displayName: "[Output] SC Environment App Details"
67
inputs:
78
azureSubscription: $(ACN_TEST_SERVICE_CONNECTION)
@@ -14,10 +15,10 @@ steps:
1415
1516
# App OID
1617
SP_APP_OID=$(az ad sp show --id "$servicePrincipalId" | jq -rc '.id')
17-
echo >&2 "##vso[task.setvariable variable=SP_APP_OID]$SP_APP_OID"
18+
echo >&2 "##vso[task.setvariable variable=ACNCI_BUILD_SP_APP_OID;isoutput=true;]$SP_APP_OID"
1819
# Get Subscription ID.
1920
SUBSCRIPTION_ID=$(az account show | jq -rc '.id')
20-
echo >&2 "##vso[task.setvariable variable=SUBSCRIPTION_ID]$SUBSCRIPTION_ID"
21+
echo >&2 "##vso[task.setvariable variable=ACNCI_BUILD_SUBSCRIPTION_ID;isoutput=true;issecret=true]$SUBSCRIPTION_ID"
2122
2223
2324
## Resource Groups ##
@@ -147,11 +148,12 @@ steps:
147148
--resource-group "$ACNCI_BUILD_RESOURCEGROUP" | \
148149
jq -rc '.')
149150
MI_DATA_LENGTH=$(echo "$MI_DATA" | jq length)
151+
echo >&2 "##vso[task.setvariable variable=MI_DATA]$MI_DATA"
152+
echo >&2 "##vso[task.setvariable variable=MI_DATA_LENGTH]$MI_DATA_LENGTH"
150153
151154
if [[ -n $MI_DATA ]] && \
152155
[[ $MI_DATA != null ]] && \
153156
(( $MI_DATA_LENGTH > 0 )); then
154-
echo >&2 "##vso[task.setvariable variable=MI_DATA;isoutput=true]$MI_DATA"
155157
echo >&2 "##vso[task.setvariable variable=CREATE_NEW_MI;]false"
156158
else
157159
echo >&2 "##vso[task.setvariable variable=CREATE_NEW_MI;]true"
@@ -207,6 +209,9 @@ steps:
207209
echo >&2 "##vso[task.setvariable variable=ACNCI_MANAGEDIDENTITY_APPID;isoutput=true]$MI_ID"
208210
MI_NAME=$(echo "$MI_DATA" | jq -r '.name')
209211
echo >&2 "##vso[task.setvariable variable=ACNCI_MANAGEDIDENTITY_NAME;isoutput=true]$MI_NAME"
212+
env:
213+
MI_DATA: $(MI_DATA)
214+
MI_DATA_LENGTH: $(MI_DATA_LENGTH)
210215

211216

212217
## MI Role Assignments ##
@@ -229,7 +234,7 @@ steps:
229234
230235
DEF=$(cat ./azure-container-networking/.pipelines/templates/mi-build-role.json | \
231236
jq -rc \
232-
--arg RESOURCEID "/subscriptions/$SUBSCRIPTION_ID" \
237+
--arg RESOURCEID "/subscriptions/$ACNCI_BUILD_SUBSCRIPTION_ID" \
233238
'.assignableScopes[] = $RESOURCEID')
234239
235240
if (( "$DEFS_FOUND" < 1 )); then
@@ -242,8 +247,8 @@ steps:
242247
echo >&2 "##[error]There was an error creating/updating the role definition."
243248
exit 1
244249
fi
245-
#env:
246-
# ACNCI_BUILD_RESOURCEGROUP_ID: $(resourcegroups.ACNCI_BUILD_RESOURCEGROUP_ID)
250+
env:
251+
ACNCI_BUILD_SUBSCRIPTION_ID: $(build.ACNCI_BUILD_SUBSCRIPTION_ID)
247252

248253
- task: AzureCLI@2
249254
displayName: "[Check] Build User MI Roles"
@@ -283,8 +288,8 @@ steps:
283288
[[ -n $SYSTEM_DEBUG ]] && [[ $SYSTEM_DEBUG =~ $IS_TRUE ]] && set -x || set +x
284289
285290
if [[ -n $MI_ROLE_DATA_LENGTH ]] && \
286-
[[ $MI_DATA != null ]] && \
287-
(( $MI_DATA_LENGTH > 0 )); then
291+
[[ $MI_ROLE_DATA != null ]] && \
292+
(( $MI_ROLE_DATA_LENGTH > 0 )); then
288293
az role assignment create \
289294
--role "$ACNCI_BUILDUSER_ROLE_NAME"
290295
--assignee-object-id "$ACNCI_MANAGEDIDENTITY_OBJECTID" \
@@ -299,11 +304,11 @@ steps:
299304
--scope "$ACNCI_BUILD_RESOURCEGROUP_ID"
300305
fi
301306
env:
307+
MI_ROLE_DATA: $(MI_ROLE_DATA)
308+
MI_ROLE_DATA_LENGTH: $(MI_ROLE_DATA_LENGTH)
302309
ACNCI_BUILD_RESOURCEGROUP_ID: $(resourcegroups.ACNCI_BUILD_RESOURCEGROUP_ID)
303310
ACNCI_MANAGEDIDENTITY_OBJECTID: $(managedidentity.ACNCI_MANAGEDIDENTITY_OBJECTID)
304311
ACNCI_BUILDUSER_ROLE_NAME: $(ACNCI_BUILDUSER_ROLE_NAME)
305-
MI_ROLE_DATA: $(MI_ROLE_DATA)
306-
MI_ROLE_DATA_LENGTH: $(MI_ROLE_DATA_LENGTH)
307312

308313

309314
#- task: AzureCLI@2

.pipelines/templates/get-storage-accounts.steps.yaml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,6 @@ steps:
2828
set -eu
2929
[[ -n $SYSTEM_DEBUG ]] && [[ $SYSTEM_DEBUG =~ $IS_TRUE ]] && set -x || set +x
3030
31-
# Get Subscription ID.
32-
SUBSCRIPTION_ID=$(az account show | jq -rc .)
33-
echo $SUBSCRIPTION_ID
3431
# Query for Test Storage Accounts
3532
R_QUERY="[? type == 'Microsoft.Storage/storageAccounts' && resourceGroup == '$RG_NAME']"
3633
az resource list \

0 commit comments

Comments
 (0)