Skip to content

Commit 343b8a2

Browse files
fix(assign-arc-app-roles): guard against empty MI principal ID and add success output
1 parent 445750f commit 343b8a2

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

scripts/bash/assign_arc_app_roles.sh

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,11 @@ while IFS= read -r machine; do
3737
--subscription "$SUBSCRIPTION" \
3838
--query "identity.principalId" -o tsv)
3939

40+
if [ -z "$miPrincipalId" ]; then
41+
echo " Warning: $machine has no managed identity principal ID, skipping."
42+
continue
43+
fi
44+
4045
echo "Assigning $appRoleValue to $machine (MI: $miPrincipalId)..."
4146
if ! output=$(az rest --method POST \
4247
--uri "https://graph.microsoft.com/v1.0/servicePrincipals/${spObjectId}/appRoleAssignedTo" \
@@ -48,6 +53,8 @@ while IFS= read -r machine; do
4853
echo "Error: $output"
4954
exit 1
5055
fi
56+
else
57+
echo " Successfully assigned."
5158
fi
5259
done <<< "$arcMachines"
5360

0 commit comments

Comments
 (0)