From 1ff7ed26def6bd9aa16dacc15bbd532f7e4e6074 Mon Sep 17 00:00:00 2001 From: zanejohnson-azure Date: Wed, 13 May 2026 15:26:51 -0700 Subject: [PATCH] fix(ci): use variable reference for AllNodesClientId in e2e test stage The ci-logs-dev-aks-all-nodes E2E test stage was passing the literal string 'AllNodesClientId' as azureClientId instead of the variable reference $(AllNodesClientId). This caused the test pod's workload identity to be configured with a non-GUID client ID, resulting in ManagedIdentityCredential failing with 'Identity not found' when querying Log Analytics. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- .pipelines/azure_pipeline_mergedbranches.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.pipelines/azure_pipeline_mergedbranches.yaml b/.pipelines/azure_pipeline_mergedbranches.yaml index 2e2fa1d3f..0f2d4049f 100644 --- a/.pipelines/azure_pipeline_mergedbranches.yaml +++ b/.pipelines/azure_pipeline_mergedbranches.yaml @@ -1062,7 +1062,7 @@ extends: azureSubscription: 'ContainerInsights_Build_Subscription_CI' environmentName: 'CI-Agent-Dev-Aks-All-Nodes' dependsOnDeployJob: 'Deploy_AmaLogs_ci_logs_dev_aks_all_nodes' - azureClientId: 'AllNodesClientId' + azureClientId: $(AllNodesClientId) azureTenantId: $(CI_BUILD_AZURE_TENANT_ID) teamsWebhookUri: $(TeamsWebhookUri) additionalTestParams: 'PerNodeLogCoverage=true'