Skip to content

Commit

Permalink
Do not attempt to publish TLS report during releases (#1070)
Browse files Browse the repository at this point in the history
  • Loading branch information
achamayou authored Apr 15, 2020
1 parent d2bbc79 commit 53e5648
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 3 deletions.
4 changes: 4 additions & 0 deletions .azure-pipelines-templates/common.yml
Original file line number Diff line number Diff line change
@@ -1,3 +1,6 @@
parameters:
tls_report: false

jobs:
- job: ${{ parameters.target }}_${{ parameters.suffix }}
variables:
Expand Down Expand Up @@ -35,6 +38,7 @@ jobs:
- template: publish_build.yml
parameters:
artifact_name: ${{ parameters.artifact_name }}
tls_report: ${{ parameters.tls_report }}

- template: install.yml
parameters:
Expand Down
3 changes: 2 additions & 1 deletion .azure-pipelines-templates/daily-matrix.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,4 +36,5 @@ jobs:
suffix: 'Release'
artifact_name: 'SGX_Release'
ctest_filter: '-LE "benchmark|perf"'
install_prefix: '${{ parameters.build.install.install_prefix }}'
install_prefix: '${{ parameters.build.install.install_prefix }}'
tls_report: true
6 changes: 4 additions & 2 deletions .azure-pipelines-templates/publish_build.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
parameters:
parameters:
artifact_name: ''
tls_report: false

steps:
- script: |
Expand All @@ -15,4 +16,5 @@ steps:
- task: PublishPipelineArtifact@1
inputs:
artifactName: '${{ parameters.artifact_name }} TLS report'
targetPath: build/tls_report.html
targetPath: build/tls_report.html
condition: eq('${{ parameters.tls_report }}', true)

0 comments on commit 53e5648

Please sign in to comment.