Steps/BuildPlatform.yml & Steps/PrGate.yml: Introduce publish_artifac…

…ts parameter (#355)
microsoft · Jul 25, 2024 · 7f95228 · 7f95228
1 parent 84958c8
commit 7f95228
Show file tree

Hide file tree

Showing 5 changed files with 50 additions and 21 deletions.
diff --git a/Steps/BinaryCopyAndPublish.yml b/Steps/BinaryCopyAndPublish.yml
@@ -15,6 +15,10 @@ parameters:
   displayName: Artifacts Identifier
   type: string
   default: 'Artifacts'
+- name: publish_artifacts
+  displayName: Publish Artifacts
+  type: boolean
+  default: true
 
 steps:
 - bash: |
@@ -37,10 +41,11 @@ steps:
   condition: and(succeededOrFailed(), eq(variables.artifacts_present, 'true'))
 
 # Publish build artifacts to Azure Artifacts/TFS or a file share
-- task: PublishBuildArtifacts@1
-  continueOnError: true
-  displayName: Publish Build Binaries
-  inputs:
-    pathtoPublish: "$(Build.ArtifactStagingDirectory)/Binaries"
-    artifactName: "Binaries ${{ parameters.artifacts_identifier }}"
-  condition: and(succeededOrFailed(), eq(variables.artifacts_present, 'true'))
+- ${{ if eq(parameters.publish_artifacts, true) }}:
+  - task: PublishPipelineArtifact@1
+    continueOnError: true
+    displayName: Publish Build Binaries
+    inputs:
+      targetPath: "$(Build.ArtifactStagingDirectory)/Binaries"
+      artifactName: "Binaries ${{ parameters.artifacts_identifier }}"
+    condition: and(succeededOrFailed(), eq(variables.artifacts_present, 'true'))
diff --git a/Steps/BuildPlatform.yml b/Steps/BuildPlatform.yml
@@ -66,6 +66,10 @@ parameters:
   displayName: Perform self checkout step
   type: boolean
   default: true
+- name: publish_artifacts
+  displayName: Publish Artifacts
+  type: boolean
+  default: true
 
 steps:
 - ${{ if eq(parameters.checkout_self, true) }}:
@@ -139,15 +143,18 @@ steps:
 - template: CommonLogCopyAndPublish.yml
   parameters:
     artifacts_identifier: ${{ parameters.artifacts_identifier }}
+    publish_artifacts: ${{ parameters.publish_artifacts }}
 
 # Copy build binaries to the artifact staging directory
 - template: BinaryCopyAndPublish.yml
   parameters:
     artifacts_binary: ${{ parameters.artifacts_binary }}
     artifacts_identifier: ${{ parameters.artifacts_identifier }}
+    publish_artifacts: ${{ parameters.publish_artifacts }}
 
 # Copy other files to the artifact staging directory
 - template: OtherCopyAndPublish.yml
   parameters:
     artifacts_other: ${{ parameters.artifacts_other }}
     artifacts_identifier: ${{ parameters.artifacts_identifier }}
+    publish_artifacts: ${{ parameters.publish_artifacts }}
diff --git a/Steps/CommonLogCopyAndPublish.yml b/Steps/CommonLogCopyAndPublish.yml
@@ -11,6 +11,10 @@ parameters:
   displayName: Artifacts Identifier
   type: string
   default: 'Artifacts'
+- name: publish_artifacts
+  displayName: Publish Artifacts
+  type: boolean
+  default: true
 
 steps:
 - task: CopyFiles@2
@@ -38,10 +42,11 @@ steps:
     flattenFolders: true
   condition: succeededOrFailed()
 
-- task: PublishBuildArtifacts@1
-  continueOnError: true
-  displayName: Publish Build Logs
-  inputs:
-    pathtoPublish: '$(Build.ArtifactStagingDirectory)/Logs'
-    artifactName: 'Logs ${{ parameters.artifacts_identifier }}'
-  condition: succeededOrFailed()
+- ${{ if eq(parameters.publish_artifacts, true) }}:
+  - task: PublishPipelineArtifact@1
+    continueOnError: true
+    displayName: Publish Build Logs
+    inputs:
+      targetPath: '$(Build.ArtifactStagingDirectory)/Logs'
+      artifactName: 'Logs ${{ parameters.artifacts_identifier }}'
+    condition: succeededOrFailed()
diff --git a/Steps/OtherCopyAndPublish.yml b/Steps/OtherCopyAndPublish.yml
@@ -15,6 +15,10 @@ parameters:
   displayName: Other Artifacts to Publish
   type: string
   default: ''
+- name: publish_artifacts
+  displayName: Publish Artifacts
+  type: boolean
+  default: true
 
 steps:
 - bash: |
@@ -37,10 +41,11 @@ steps:
   condition: and(succeededOrFailed(), eq(variables.artifacts_present, 'true'))
 
 # Publish build artifacts to Azure Artifacts/TFS or a file share
-- task: PublishBuildArtifacts@1
-  continueOnError: true
-  displayName: Publish Other Files
-  inputs:
-    pathtoPublish: "$(Build.ArtifactStagingDirectory)/Other"
-    artifactName: "Other ${{ parameters.artifacts_identifier }}"
-  condition: and(succeededOrFailed(), eq(variables.artifacts_present, 'true'))
+- ${{ if eq(parameters.publish_artifacts, true) }}:
+  - task: PublishPipelineArtifact@1
+    continueOnError: true
+    displayName: Publish Other Files
+    inputs:
+      targetPath: "$(Build.ArtifactStagingDirectory)/Other"
+      artifactName: "Other ${{ parameters.artifacts_identifier }}"
+    condition: and(succeededOrFailed(), eq(variables.artifacts_present, 'true'))
diff --git a/Steps/PrGate.yml b/Steps/PrGate.yml
@@ -88,6 +88,10 @@ parameters:
   displayName: Indicate if this is a selfhost agent
   type: boolean
   default: false
+- name: publish_artifacts
+  displayName: Publish Artifacts
+  type: boolean
+  default: true
 
 steps:
 - ${{ if eq(parameters.checkout_self, true) }}:
@@ -206,15 +210,18 @@ steps:
 - template: CommonLogCopyAndPublish.yml
   parameters:
     artifacts_identifier: ${{ parameters.artifacts_identifier }}
+    publish_artifacts: ${{ parameters.publish_artifacts }}
 
 # Copy build binaries to the artifact staging directory
 - template: BinaryCopyAndPublish.yml
   parameters:
     artifacts_binary: ${{ parameters.artifacts_binary }}
     artifacts_identifier: ${{ parameters.artifacts_identifier }}
+    publish_artifacts: ${{ parameters.publish_artifacts }}
 
 # Copy other files to the artifact staging directory
 - template: OtherCopyAndPublish.yml
   parameters:
     artifacts_other: ${{ parameters.artifacts_other }}
     artifacts_identifier: ${{ parameters.artifacts_identifier }}
+    publish_artifacts: ${{ parameters.publish_artifacts }}