Skip to content

Commit 70c62d5

Browse files
committed
Join in non-source-only even in PR.
1 parent be6b487 commit 70c62d5

File tree

3 files changed

+68
-60
lines changed

3 files changed

+68
-60
lines changed

eng/pipelines/templates/stages/vmr-build-with-join.yml

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -54,13 +54,16 @@ parameters:
5454
stages:
5555
# Insert all stages that build verticals
5656
# and publish artifacts for each vertical
57-
- ${{ each vertical in parameters.verticalsStages }}:
58-
- ${{ insert }}: ${{ vertical }}
57+
- ${{ each stage in parameters.verticalsStages }}:
58+
- ${{ insert }}: ${{ stage }}
5959
- ${{ if parameters.finalJoinCondition }}:
6060
# This stage should contain all jobs that take inputs from all verticals and determine the exact final outputs.
6161
- stage: VMR_Final_Join
6262
displayName: 'VMR Final Join'
63-
dependsOn: ${{ parameters.verticalsStages.*.stage }}
63+
dependsOn:
64+
- ${{ each stage in parameters.verticalsStages }}:
65+
- ${{ if ne(stage.templateContext.validationOnly, 'true') }}:
66+
- ${{ stage.stage }}
6467
jobs:
6568
- job: FinalJoin
6669
displayName: Final Build Pass
@@ -70,10 +73,12 @@ stages:
7073
outputParentDirectory: $(Build.ArtifactStagingDirectory)/artifacts
7174
inputs:
7275
# Download the artifacts from every job in every vertical stage under VerticalArtifacts
73-
- ${{ each vertical in parameters.verticalsStages.*.jobs.*.job }}:
74-
- input: pipelineArtifact
75-
path: $(Build.ArtifactStagingDirectory)/VerticalArtifacts/${{ vertical }}
76-
artifactName: ${{ vertical }}_Artifacts
76+
- ${{ each stage in parameters.verticalsStages }}:
77+
- ${{ if ne(stage.templateContext.validationOnly, 'true') }}:
78+
- ${{ each vertical in stage.jobs.*.job }}:
79+
- input: pipelineArtifact
80+
path: $(Build.ArtifactStagingDirectory)/VerticalArtifacts/${{ vertical.job }}
81+
artifactName: ${{ vertical.job }}_Artifacts
7782
- input: pipelineArtifact
7883
path: $(Build.ArtifactStagingDirectory)/VerticalManifests
7984
artifactName: VerticalManifests

eng/pipelines/templates/stages/vmr-build.yml

Lines changed: 54 additions & 53 deletions
Original file line numberDiff line numberDiff line change
@@ -85,65 +85,66 @@ stages:
8585
isBuiltFromVmr: ${{ parameters.isBuiltFromVmr }}
8686
scope: ${{ parameters.scope }}
8787
isSourceOnlyBuild: ${{ parameters.isSourceOnlyBuild }}
88-
finalJoinCondition: ${{ and(parameters.isBuiltFromVmr, not(parameters.isSourceOnlyBuild), eq(variables['System.TeamProject'], 'internal'), ne(variables['Build.Reason'], 'PullRequest')) }}
88+
finalJoinCondition: ${{ and(not(parameters.isSourceOnlyBuild), eq(parameters.scope, 'full')) }}
8989
postJoinStages:
90-
- stage: Publish_Build_Assets
91-
displayName: Publish Assets
92-
jobs:
93-
- template: /eng/common/templates-official/job/publish-build-assets.yml@self
94-
parameters:
95-
publishUsingPipelines: true
96-
publishAssetsImmediately: true
97-
pool: ${{ parameters.pool_Linux }}
90+
- ${{ if and(parameters.isBuiltFromVmr, not(parameters.isSourceOnlyBuild), eq(variables['System.TeamProject'], 'internal'), ne(variables['Build.Reason'], 'PullRequest')) }}:
91+
- stage: Publish_Build_Assets
92+
displayName: Publish Assets
93+
jobs:
94+
- template: /eng/common/templates-official/job/publish-build-assets.yml@self
95+
parameters:
96+
publishUsingPipelines: true
97+
publishAssetsImmediately: true
98+
pool: ${{ parameters.pool_Linux }}
9899

99-
- stage: VMR_Validation
100-
displayName: VMR Validation
101-
variables:
102-
- template: ../variables/vmr-build.yml
103-
parameters:
104-
vmrBranch: ${{ parameters.vmrBranch }}
105-
desiredSigning: ${{ parameters.desiredSigning }}
106-
- group: Release-Pipeline
107-
- group: DotNetBot-GitHub-AllBranches
108-
jobs:
109-
- job: ValidateAssetBaselines
110-
displayName: Validate Asset Baselines
111-
pool: ${{ parameters.pool_Windows }}
112-
timeoutInMinutes: 180
113-
steps:
114-
- template: ../steps/vmr-validate-asset-baseline.yml
100+
- stage: VMR_Validation
101+
displayName: VMR Validation
102+
variables:
103+
- template: ../variables/vmr-build.yml
115104
parameters:
116-
continueOnError: true
117-
- ${{ if eq(variables.signEnabled, 'true') }}:
118-
- job: ValidateSigning_Windows
119-
displayName: Validate Signing - Windows
105+
vmrBranch: ${{ parameters.vmrBranch }}
106+
desiredSigning: ${{ parameters.desiredSigning }}
107+
- group: Release-Pipeline
108+
- group: DotNetBot-GitHub-AllBranches
109+
jobs:
110+
- job: ValidateAssetBaselines
111+
displayName: Validate Asset Baselines
120112
pool: ${{ parameters.pool_Windows }}
121-
timeoutInMinutes: 240
113+
timeoutInMinutes: 180
122114
steps:
123-
- template: ../steps/vmr-validate-signing.yml
115+
- template: ../steps/vmr-validate-asset-baseline.yml
124116
parameters:
125-
artifactName: PackageArtifacts
126117
continueOnError: true
127-
OS: Windows_NT
118+
- ${{ if eq(variables.signEnabled, 'true') }}:
119+
- job: ValidateSigning_Windows
120+
displayName: Validate Signing - Windows
121+
pool: ${{ parameters.pool_Windows }}
122+
timeoutInMinutes: 240
123+
steps:
124+
- template: ../steps/vmr-validate-signing.yml
125+
parameters:
126+
artifactName: PackageArtifacts
127+
continueOnError: true
128+
OS: Windows_NT
128129

129-
- job: ValidateSigning_Mac
130-
displayName: Validate Signing - Mac
131-
pool: ${{ parameters.pool_Mac }}
132-
timeoutInMinutes: 240
133-
steps:
134-
- template: ../steps/vmr-validate-signing.yml
135-
parameters:
136-
artifactName: BlobArtifacts
137-
continueOnError: true
138-
OS: Darwin
130+
- job: ValidateSigning_Mac
131+
displayName: Validate Signing - Mac
132+
pool: ${{ parameters.pool_Mac }}
133+
timeoutInMinutes: 240
134+
steps:
135+
- template: ../steps/vmr-validate-signing.yml
136+
parameters:
137+
artifactName: BlobArtifacts
138+
continueOnError: true
139+
OS: Darwin
139140

140-
- job: ValidateSigning_Linux
141-
displayName: Validate Signing - Linux
142-
pool: ${{ parameters.pool_Linux }}
143-
timeoutInMinutes: 240
144-
steps:
145-
- template: ../steps/vmr-validate-signing.yml
146-
parameters:
147-
artifactName: BlobArtifacts
148-
continueOnError: true
149-
OS: Linux
141+
- job: ValidateSigning_Linux
142+
displayName: Validate Signing - Linux
143+
pool: ${{ parameters.pool_Linux }}
144+
timeoutInMinutes: 240
145+
steps:
146+
- template: ../steps/vmr-validate-signing.yml
147+
parameters:
148+
artifactName: BlobArtifacts
149+
continueOnError: true
150+
OS: Linux

eng/pipelines/templates/stages/vmr-verticals.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -355,6 +355,8 @@ stages:
355355
- ${{ if and(not(parameters.isSourceOnlyBuild), or(eq(variables['Build.Reason'], 'PullRequest'), ne(variables['System.TeamProject'], 'internal'))) }}:
356356
- stage: VMR_Vertical_Build_Validation
357357
displayName: VMR Vertical Build Validation
358+
templateContext:
359+
validationOnly: true
358360
dependsOn: []
359361
variables:
360362
- template: ../variables/vmr-build.yml

0 commit comments

Comments
 (0)