Skip to content

Commit f10b138

Browse files
authored
Default to live tests for only scheduled runs (Azure#2344)
1 parent a76bb44 commit f10b138

File tree

3 files changed

+11
-3
lines changed

3 files changed

+11
-3
lines changed

eng/pipelines/templates/jobs/live.tests.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -102,6 +102,7 @@ jobs:
102102
- ${{ if parameters.UseFederatedAuth }}:
103103
- task: AzurePowerShell@5
104104
displayName: "Test Packages - Federated Auth"
105+
condition: and(succeeded(), eq(variables['CI_HAS_DEPLOYED_RESOURCES'], 'true')))
105106
timeoutInMinutes: ${{ parameters.TimeoutInMinutes }}
106107
env:
107108
CIBW_BUILD_VERBOSITY: 3
@@ -126,6 +127,7 @@ jobs:
126127
- ${{ else }}:
127128
- task: Powershell@2
128129
displayName: "Test Packages"
130+
condition: and(succeeded(), eq(variables['CI_HAS_DEPLOYED_RESOURCES'], 'true')))
129131
timeoutInMinutes: ${{ parameters.TimeoutInMinutes }}
130132
env:
131133
CIBW_BUILD_VERBOSITY: 3

eng/pipelines/templates/stages/archetype-sdk-client.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ parameters:
99
default: true
1010
- name: RunLiveTests
1111
type: boolean
12-
default: true
12+
default: false
1313
# Test Parameters
1414
- name: TestTimeoutInMinutes
1515
type: number
@@ -93,8 +93,8 @@ extends:
9393
MatrixFilters: ${{ parameters.MatrixFilters }}
9494
MatrixReplace: ${{ parameters.MatrixReplace }}
9595

96-
# Run live tests for internal only, not public CI builds. This can be triggered manually for via an `/azp run` comment.
97-
- ${{if eq(parameters.RunLiveTests, 'true') }}:
96+
# Run live tests for internal only, not public CI builds. This can be triggered manually via an `/azp run` comment.
97+
- ${{if and(eq(variables['System.TeamProject'], 'internal')), eq(parameters.RunLiveTests, 'true')) }}:
9898
- ${{ each cloud in parameters.CloudConfig }}:
9999
# Run all clouds by default for weekly test pipeline, except for clouds specifically unsupported by the calling pipeline
100100
- ${{ if or(contains(parameters.Clouds, cloud.key), and(contains(variables['Build.DefinitionName'], 'weekly'), contains(parameters.SupportedClouds, cloud.key))) }}:

sdk/keyvault/ci.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,9 @@
11
# NOTE: Please refer to https://aka.ms/azsdk/engsys/ci-yaml before editing this file.
2+
parameters:
3+
- name: RunLiveTests
4+
displayName: 'Run live tests'
5+
type: boolean
6+
default: false
27

38
trigger:
49
branches:
@@ -14,6 +19,7 @@ extends:
1419
template: /eng/pipelines/templates/stages/archetype-sdk-client.yml
1520
parameters:
1621
ServiceDirectory: keyvault
22+
RunLiveTests: ${{ or(parameters.RunLiveTests, eq(variables['Build.Reason'], 'Schedule')) }}
1723
Artifacts:
1824
- name: azure_security_keyvault_secrets
1925
safeName: AzureSecurityKeyvaultSecrets

0 commit comments

Comments
 (0)