Add succeeded() condition to docker push PullRequest condition

Co-authored-by: Wes Haggard <[email protected]>
Azure · Oct 14, 2021 · 7bc1654 · 7bc1654
1 parent d324c0f
commit 7bc1654
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/eng/containers/ci.yml b/eng/containers/ci.yml
@@ -80,7 +80,7 @@ jobs:
 
         - task: Docker@2
           displayName: Push ${{ config.name }}:$(imageTag)
-          condition: ne(variables['Build.Reason'], 'PullRequest')
+          condition: and(succeeded(), ne(variables['Build.Reason'], 'PullRequest'))
           inputs:
             containerRegistry: $(containerRegistry)
             repository: ${{ config.dockerRepo }}
@@ -98,7 +98,8 @@ jobs:
 
           - task: Docker@2
             displayName: Push ${{ config.name }}:${{ stableTag }}
-            condition: ne(variables['Build.Reason'], 'PullRequest')
+            condition: and(succeeded(), ne(variables['Build.Reason'], 'PullRequest'))
+
             inputs:
               containerRegistry: $(containerRegistry)
               repository: ${{ config.dockerRepo }}