Skip to content

Commit 8054e90

Browse files
authored
Add trailing glob to improve sparse checkout performance (#31042)
1 parent a25ee97 commit 8054e90

File tree

2 files changed

+12
-12
lines changed

2 files changed

+12
-12
lines changed

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

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -47,19 +47,19 @@ jobs:
4747
Paths:
4848
- "/*"
4949
- "!SessionRecords"
50-
- "/sdk/${{ parameters.ServiceDirectory }}/**/SessionRecords"
50+
- "/sdk/${{ parameters.ServiceDirectory }}/**/SessionRecords/*"
5151
${{ else }}:
5252
Paths:
5353
- "/*"
5454
- "!SessionRecords"
55-
- "/sdk/*/**/SessionRecords"
55+
- "/sdk/*/**/SessionRecords/*"
5656
- ${{ if ne(parameters.ProjectListOverrideFilePropertyName, '') }}:
5757
- task: DownloadPipelineArtifact@2
5858
inputs:
5959
artifact: DependencyTestProjectReferences
6060
patterns: "**/$(${{ parameters.ProjectListOverrideFilePropertyName }})"
6161
path: $(Build.SourcesDirectory)
62-
62+
6363
- pwsh: |
6464
$(Build.SourcesDirectory)/eng/common/scripts/trust-proxy-certificate.ps1
6565
displayName: 'Language Specific Certificate Trust'
@@ -76,12 +76,12 @@ jobs:
7676
- template: /eng/pipelines/templates/steps/install-dotnet.yml
7777
parameters:
7878
NuGetCacheKey: Test
79-
- pwsh: |
79+
- pwsh: |
8080
if ("${{ parameters.ProjectListOverrideFilePropertyName }}" -ne '') {
8181
Write-Host "##vso[task.setvariable variable=ProjectListOverrideFileName]$(${{ parameters.ProjectListOverrideFilePropertyName }})"
8282
Write-Host "##vso[task.setvariable variable=CodeCoverageFilePattern]$(Build.SourcesDirectory)\sdk\**\coverage.cobertura.xml"
8383
}
84-
else
84+
else
8585
{
8686
Write-Host "##vso[task.setvariable variable=ProjectListOverrideFileName]"
8787
Write-Host "##vso[task.setvariable variable=CodeCoverageFilePattern]$(Build.SourcesDirectory)\sdk\${{parameters.ServiceDirectory}}\**\coverage.cobertura.xml"
@@ -99,7 +99,7 @@ jobs:
9999
/p:RunApiCompat=false /p:InheritDocEnabled=false
100100
/p:Configuration=$(BuildConfiguration)
101101
/p:CollectCoverage=$(CollectCoverage)
102-
/p:EnableSourceLink=false
102+
/p:EnableSourceLink=false
103103
/p:ProjectListOverrideFile=$(ProjectListOverrideFileName)
104104
$(AdditionalTestArguments)
105105
displayName: "Build & Test ($(TestTargetFramework))"

eng/pipelines/templates/jobs/ci.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -238,21 +238,21 @@ jobs:
238238
SDKType: ${{ parameters.SDKType }}
239239
ServiceDirectory: ${{ parameters.ServiceDirectory }}
240240
TestSetupSteps: ${{ parameters.TestSetupSteps }}
241-
TestTimeoutInMinutes: ${{ parameters.TestTimeoutInMinutes }}
241+
TestTimeoutInMinutes: ${{ parameters.TestTimeoutInMinutes }}
242242
- ${{ if ne(parameters.TestDependsOnDependency, 'not-specified') }}:
243243
- template: /eng/common/pipelines/templates/jobs/archetype-sdk-tests-generate.yml
244244
parameters:
245-
GenerateJobName: generate_target_dependencies_test_matrix
245+
GenerateJobName: generate_target_dependencies_test_matrix
246246
JobTemplatePath: /eng/pipelines/templates/jobs/ci.tests.yml
247247
MatrixConfigs: ${{ parameters.MatrixConfigs }}
248248
MatrixFilters: ${{ parameters.MatrixFilters }}
249249
MatrixReplace: ${{ parameters.MatrixReplace }}
250250
CloudConfig:
251251
Cloud: public
252-
SparseCheckoutPaths:
252+
SparseCheckoutPaths:
253253
- "/*"
254-
- "!/sdk/*/**/SessionRecords"
255-
PreGenerationSteps:
254+
- "!/sdk/*/**/SessionRecords/*"
255+
PreGenerationSteps:
256256
- ${{ each config in parameters.MatrixConfigs }}:
257257
- template: /eng/pipelines/templates/steps/dependency.tests.yml
258258
parameters:
@@ -269,4 +269,4 @@ jobs:
269269
TestSetupSteps: ${{ parameters.TestSetupSteps }}
270270
TestTimeoutInMinutes: ${{ parameters.TestTimeoutInMinutes }}
271271
# The value for ProjectListOverrideFilePropertyName should be the same as dependency.tests.yml parameter ProjectListOverrideFilePropertyName
272-
ProjectListOverrideFilePropertyName: 'ProjectListOverrideFile'
272+
ProjectListOverrideFilePropertyName: 'ProjectListOverrideFile'

0 commit comments

Comments
 (0)