Skip to content

Commit

Permalink
Only build the GC simulator tests in the GC simulator leg
Browse files Browse the repository at this point in the history
Only build the GC simluator tests instead of all Pri1 tests. When we build all Pri1 tests in this leg, nearly all tests are skipped as only the GCSimluator tests are actually run. These skipped tests leave a ton of files on disk, causing issues in Helix. By only building the actual tests we are going to run, we can avoid this additional infrastructure cost.
  • Loading branch information
jkoritzinsky committed Apr 18, 2022
1 parent 3b6b812 commit 24168eb
Showing 1 changed file with 13 additions and 1 deletion.
14 changes: 13 additions & 1 deletion eng/pipelines/common/templates/runtimes/build-test-job.yml
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,18 @@ jobs:
- name: runtimeFlavorArgs
value: '-mono'

- name: testTreeFilterArg
value: ''

# Only build GCSimulator tests when the gc-simulator group is specified.
- ${{ if eq(parameters.testGroup, 'gc-simulator') }}:
- ${{ if eq(parameters.osGroup, 'windows') }}:
- name: testTreeFilterArg
value: '-tree GC/Scenarios/GCSimulator'
- ${{ if ne(parameters.osGroup, 'windows') }}:
- name: testTreeFilterArg
value: '-tree:GC/Scenarios/GCSimulator'

steps:

# Install test build dependencies
Expand All @@ -113,7 +125,7 @@ jobs:
displayName: Disk Usage before Build
# Build managed test components
- script: $(Build.SourcesDirectory)/src/tests/build$(scriptExt) $(logRootNameArg)Managed allTargets skipnative skipgeneratelayout skiptestwrappers $(buildConfig) $(archType) $(runtimeFlavorArgs) $(crossArg) $(priorityArg) ci /p:TargetOS=AnyOS
- script: $(Build.SourcesDirectory)/src/tests/build$(scriptExt) $(logRootNameArg)Managed allTargets skipnative skipgeneratelayout skiptestwrappers $(buildConfig) $(archType) $(runtimeFlavorArgs) $(crossArg) $(priorityArg) $(testTreeFilterArg) ci /p:TargetOS=AnyOS
displayName: Build managed test components

- ${{ if in(parameters.osGroup, 'OSX', 'iOS', 'tvOS') }}:
Expand Down

0 comments on commit 24168eb

Please sign in to comment.