Skip to content

Commit 78dd9ae

Browse files
authored
Enable ExcludePaths in the pullrequest pipeline (Azure#39231)
* Enable ExcludePaths in the pullrequest pipeline * Temporary commit for testing, changes will be reverted before merge * Reverting the testing changes to cosmos and template's MD files * Really revert this time. VS Code trimming spaces caused it to not be reverted
1 parent f5f9c86 commit 78dd9ae

File tree

4 files changed

+23
-2
lines changed

4 files changed

+23
-2
lines changed

eng/pipelines/templates/jobs/ci.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,9 @@ parameters:
88
- name: TestPipeline
99
type: boolean
1010
default: false
11+
- name: ExcludePaths
12+
type: object
13+
default: []
1114
- name: BeforePublishSteps
1215
type: object
1316
default: []
@@ -77,6 +80,7 @@ jobs:
7780
Artifacts: ${{ parameters.Artifacts }}
7881
ArtifactSuffix: linux
7982
BuildTargetingString: ${{ parameters.BuildTargetingString }}
83+
ExcludePaths: ${{parameters.ExcludePaths}}
8084

8185
- job: 'Build_Windows'
8286
timeoutInMinutes: 90
@@ -95,6 +99,7 @@ jobs:
9599
Artifacts: ${{ parameters.Artifacts }}
96100
ArtifactSuffix: windows
97101
BuildTargetingString: ${{ parameters.BuildTargetingString }}
102+
ExcludePaths: ${{parameters.ExcludePaths}}
98103

99104
- job: 'Build_MacOS'
100105
timeoutInMinutes: 90
@@ -113,6 +118,7 @@ jobs:
113118
Artifacts: ${{ parameters.Artifacts }}
114119
ArtifactSuffix: mac
115120
BuildTargetingString: ${{ parameters.BuildTargetingString }}
121+
ExcludePaths: ${{parameters.ExcludePaths}}
116122

117123
- job: 'Build_Extended'
118124
displayName: Build Extended
@@ -260,6 +266,7 @@ jobs:
260266
- template: /eng/common/pipelines/templates/steps/save-package-properties.yml
261267
parameters:
262268
ServiceDirectory: ${{parameters.ServiceDirectory}}
269+
ExcludePaths: ${{parameters.ExcludePaths}}
263270
CloudConfig:
264271
Cloud: Public
265272
AdditionalParameters:

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,9 @@ parameters:
1414
- name: AfterTestSteps
1515
type: object
1616
default: []
17+
- name: ExcludePaths
18+
type: object
19+
default: []
1720
- name: BeforePublishSteps
1821
type: object
1922
default: []
@@ -88,6 +91,7 @@ extends:
8891
TestPipeline: true
8992
BeforeTestSteps: ${{ parameters.BeforeTestSteps }}
9093
AfterTestSteps: ${{ parameters.AfterTestSteps }}
94+
ExcludePaths: ${{ parameters.ExcludePaths }}
9195
BeforePublishSteps: ${{ parameters.BeforePublishSteps }}
9296
TestMarkArgument: ${{ parameters.TestMarkArgument }}
9397
BuildTargetingString: ${{ parameters.BuildTargetingString }}

eng/pipelines/templates/steps/build-package-artifacts.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,9 @@ parameters:
2020
- name: BuildTargetingString
2121
type: string
2222
default: '*'
23+
- name: ExcludePaths
24+
type: object
25+
default: []
2326

2427
steps:
2528
- task: UsePythonVersion@0
@@ -62,6 +65,7 @@ steps:
6265
- template: /eng/common/pipelines/templates/steps/save-package-properties.yml
6366
parameters:
6467
ServiceDirectory: ${{ parameters.ServiceDirectory }}
68+
ExcludePaths: ${{ parameters.ExcludePaths }}
6569

6670
- template: /eng/pipelines/templates/steps/resolve-package-targeting.yml
6771
parameters:

sdk/pullrequest.yml

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,9 @@ pr:
1010
paths:
1111
include:
1212
- "*"
13-
13+
# Note: The ExcludePaths template below needs to duplicate
14+
# any excludes here. The reason being is that we can't access
15+
# pr->paths->exclude
1416
exclude:
1517
- sdk/cosmos
1618

@@ -25,4 +27,8 @@ extends:
2527
ServiceDirectory: ${{ parameters.Service }}
2628
BuildTargetingString: "*"
2729
TestProxy: true
28-
TestTimeOutInMinutes: 180
30+
TestTimeOutInMinutes: 180
31+
# See pr->paths->exclude comment above. Anything added/removed there
32+
# needs to be added/removed here.
33+
ExcludePaths:
34+
- sdk/cosmos

0 commit comments

Comments
 (0)