From 45bcd7c4961f9bd960094fc5f0674c09e9fce4cb Mon Sep 17 00:00:00 2001 From: Grace Wehner Date: Fri, 12 Apr 2024 09:21:09 -0700 Subject: [PATCH] Build: fix deploy condition to use not(succeeded()) instead of failed() to account for skipped condition (#827) --- .pipelines/azure-pipeline-build.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.pipelines/azure-pipeline-build.yml b/.pipelines/azure-pipeline-build.yml index 19669174a..6f0f543c7 100644 --- a/.pipelines/azure-pipeline-build.yml +++ b/.pipelines/azure-pipeline-build.yml @@ -1071,7 +1071,7 @@ stages: jobs: - deployment: Deploy_Chart_ARC displayName: "Deploy: Arc dev cluster" - condition: and(succeeded(), and(and(eq(variables.IS_PR, false), eq(variables.IS_MAIN_BRANCH, true)), or(failed('Deploy_Chart_ARC'), eq(variables['System.StageAttempt'], 1)))) + condition: and(succeeded(), and(and(eq(variables.IS_PR, false), eq(variables.IS_MAIN_BRANCH, true)), or(not(succeeded('Deploy_Chart_ARC')), eq(variables['System.StageAttempt'], 1)))) environment: Prometheus-Collector pool: name: Azure-Pipelines-CI-Test-EO @@ -1184,7 +1184,7 @@ stages: environment: Prometheus-Collector pool: name: Azure-Pipelines-CI-Test-EO - condition: and(and(eq(variables.IS_PR, false), eq(variables.IS_MAIN_BRANCH, true)), or(failed('Deploy_AKS_Chart'), eq(variables['System.StageAttempt'], 1))) + condition: and(and(eq(variables.IS_PR, false), eq(variables.IS_MAIN_BRANCH, true)), or(not(succeeded('Deploy_AKS_Chart')), eq(variables['System.StageAttempt'], 1))) variables: HELM_CHART_NAME: $[ stageDependencies.Build.Image_Tags_and_Ev2_Artifacts.outputs['setup.HELM_CHART_NAME'] ] HELM_SEMVER: $[ stageDependencies.Build.Image_Tags_and_Ev2_Artifacts.outputs['setup.SEMVER'] ]