Fix conditions to include suceeded on send to helix inner (#45432)
authorSantiago Fernandez Madero <safern@microsoft.com>
Wed, 2 Dec 2020 07:17:54 +0000 (23:17 -0800)
committerGitHub <noreply@github.com>
Wed, 2 Dec 2020 07:17:54 +0000 (23:17 -0800)
* Fix conditions to include suceeded on send to helix inner

* Fix yaml syntax

eng/pipelines/common/templates/runtimes/send-to-helix-inner-step.yml

index 2ea5157..cff23ec 100644 (file)
@@ -12,11 +12,11 @@ steps:
   # TODO: Remove and consolidate this when we move to arcade via init-tools.cmd.
   - powershell: $(Build.SourcesDirectory)\eng\common\build.ps1 -ci ${{ parameters.restoreParams }}
     displayName: Restore blob feed tasks (Windows)
-    condition: ${{ and(ne(parameters.condition, false), ne(parameters.restoreParams, '')) }}
+    condition: and(succeeded(), ${{ and(ne(parameters.condition, false), ne(parameters.sendParams, '')) }})
 
   - powershell: $(Build.SourcesDirectory)\eng\common\msbuild.ps1 -ci ${{ parameters.sendParams }}
     displayName: ${{ parameters.displayName }} (Windows)
-    condition: ${{ and(ne(parameters.condition, false), ne(parameters.sendParams, '')) }}
+    condition: and(succeeded(), ${{ and(ne(parameters.condition, false), ne(parameters.sendParams, '')) }})
     env: ${{ parameters.environment }}
     continueOnError: ${{ parameters.shouldContinueOnError }}
 
@@ -24,7 +24,7 @@ steps:
   # TODO: Remove and consolidate this when we move to arcade via init-tools.sh.
   - script: $(Build.SourcesDirectory)/eng/common/build.sh --ci ${{ parameters.restoreParams }}
     displayName: Restore blob feed tasks (Unix)
-    condition: ${{ and(ne(parameters.condition, false), ne(parameters.restoreParams, '')) }}
+    condition: and(succeeded(), ${{ and(ne(parameters.condition, false), ne(parameters.sendParams, '')) }})
     ${{ if eq(parameters.osGroup, 'FreeBSD') }}:
       env:
         # Arcade uses this SDK instead of trying to restore one.
@@ -32,6 +32,6 @@ steps:
 
   - script: $(Build.SourcesDirectory)/eng/common/msbuild.sh --ci ${{ parameters.sendParams }}
     displayName: ${{ parameters.displayName }} (Unix)
-    condition: ${{ and(ne(parameters.condition, false), ne(parameters.sendParams, '')) }}
+    condition: and(succeeded(), ${{ and(ne(parameters.condition, false), ne(parameters.sendParams, '')) }})
     env: ${{ parameters.environment }}
     continueOnError: ${{ parameters.shouldContinueOnError }}