From 1742da6fe60b287566b7cd3a6968160467d6033e Mon Sep 17 00:00:00 2001 From: Ankit Jain Date: Wed, 16 Feb 2022 14:49:51 -0500 Subject: [PATCH 1/5] [wasm] Trigger all wasm jobs, if eng/Version.Details.xml has any changes, so we run a full check on emsdk updates --- .../common/evaluate-default-paths.yml | 3 ++ .../runtime-extra-platforms-wasm.yml | 29 +++++++++++++++++-- eng/pipelines/runtime-staging.yml | 28 +++++++++++++++--- eng/pipelines/runtime.yml | 19 ++++++++++++ 4 files changed, 72 insertions(+), 7 deletions(-) diff --git a/eng/pipelines/common/evaluate-default-paths.yml b/eng/pipelines/common/evaluate-default-paths.yml index bf6fdc7c4d1732..e0f2c40bcda9a2 100644 --- a/eng/pipelines/common/evaluate-default-paths.yml +++ b/eng/pipelines/common/evaluate-default-paths.yml @@ -124,6 +124,9 @@ jobs: - src/mono/wasm/debugger/* - src/mono/wasm/runtime/* - src/mono/mono/* + - subset: allwasm + - eng/Version.Details.xml + - src/mono/wasm/emscripten-version.txt - ${{ if ne(parameters.extraSubsets, '') }}: - ${{ parameters.extraSubsets }} diff --git a/eng/pipelines/runtime-extra-platforms-wasm.yml b/eng/pipelines/runtime-extra-platforms-wasm.yml index af51523090fca1..1673723a79263a 100644 --- a/eng/pipelines/runtime-extra-platforms-wasm.yml +++ b/eng/pipelines/runtime-extra-platforms-wasm.yml @@ -26,6 +26,8 @@ jobs: value: $[ dependencies.evaluate_paths.outputs['SetPathVars_libraries.containsChange'] ] - name: monoContainsChange value: $[ dependencies.evaluate_paths.outputs['SetPathVars_mono.containsChange'] ] + - name: allWasmContainsChange + value: $[ dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'] ] jobParameters: testGroup: innerloop nameSuffix: LibraryTests @@ -39,6 +41,7 @@ jobs: eq(variables['isWasmOnlyBuild'], true), eq(variables['isRollingBuild'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_libraries.containsChange'], true), + eq(dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_mono.containsChange'], true)) # extra steps, run tests extraStepsTemplate: /eng/pipelines/libraries/helix.yml @@ -68,6 +71,8 @@ jobs: value: $[ dependencies.evaluate_paths.outputs['SetPathVars_libraries.containsChange'] ] - name: monoContainsChange value: $[ dependencies.evaluate_paths.outputs['SetPathVars_mono.containsChange'] ] + - name: allWasmContainsChange + value: $[ dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'] ] jobParameters: isExtraPlatforms: ${{ parameters.isExtraPlatformsBuild }} testGroup: innerloop @@ -83,6 +88,7 @@ jobs: eq(variables['isRollingBuild'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_libraries.containsChange'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_mono.containsChange'], true), + eq(dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_installer.containsChange'], true)) # extra steps, run tests extraStepsTemplate: /eng/pipelines/libraries/helix.yml @@ -111,6 +117,8 @@ jobs: value: $[ dependencies.evaluate_paths.outputs['SetPathVars_libraries.containsChange'] ] - name: monoContainsChange value: $[ dependencies.evaluate_paths.outputs['SetPathVars_mono.containsChange'] ] + - name: allWasmContainsChange + value: $[ dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'] ] jobParameters: isExtraPlatforms: ${{ parameters.isExtraPlatformsBuild }} testGroup: innerloop @@ -126,6 +134,7 @@ jobs: eq(variables['isRollingBuild'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_libraries.containsChange'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_mono.containsChange'], true), + eq(dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_installer.containsChange'], true)) # extra steps, run tests extraStepsTemplate: /eng/pipelines/libraries/helix.yml @@ -150,6 +159,9 @@ jobs: platforms: - Browser_wasm - Browser_wasm_win + variables: + - name: allWasmContainsChange + value: $[ dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'] ] jobParameters: testGroup: innerloop nameSuffix: WasmBuildTests @@ -157,7 +169,9 @@ jobs: timeoutInMinutes: 180 # always run for runtime-wasm builds (triggered manually) condition: >- - eq(variables['isWasmOnlyBuild'], true) + or( + eq(dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'], true), + eq(variables['isWasmOnlyBuild'], true)) # extra steps, run tests extraStepsTemplate: /eng/pipelines/libraries/helix.yml extraStepsParameters: @@ -179,6 +193,9 @@ jobs: platforms: - Browser_wasm - Browser_wasm_win + variables: + - name: allWasmContainsChange + value: $[ dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'] ] jobParameters: testGroup: innerloop nameSuffix: Mono_DebuggerTests @@ -186,7 +203,9 @@ jobs: timeoutInMinutes: 180 # always run for runtime-wasm builds (triggered manually) condition: >- - eq(variables['isWasmOnlyBuild'], true) + or( + eq(variables['isWasmOnlyBuild'], true), + eq(dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'], true)) # extra steps, run tests extraStepsTemplate: /eng/pipelines/libraries/helix.yml extraStepsParameters: @@ -218,6 +237,8 @@ jobs: value: 10 - name: timeoutPerTestCollectionInMinutes value: 200 + - name: allWasmContainsChange + value: $[ dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'] ] jobParameters: testGroup: innerloop nameSuffix: AllSubsets_Mono_RuntimeTests @@ -225,7 +246,9 @@ jobs: timeoutInMinutes: 180 # always run for runtime-wasm builds (triggered manually) condition: >- - eq(variables['isWasmOnlyBuild'], true) + or( + eq(variables['isWasmOnlyBuild'], true), + eq(dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'], true)) extraStepsTemplate: /eng/pipelines/common/templates/runtimes/wasm-runtime-and-send-to-helix.yml extraStepsParameters: creator: dotnet-bot diff --git a/eng/pipelines/runtime-staging.yml b/eng/pipelines/runtime-staging.yml index 2eb1ee489f6eb1..aebe8af2720b17 100644 --- a/eng/pipelines/runtime-staging.yml +++ b/eng/pipelines/runtime-staging.yml @@ -325,6 +325,8 @@ jobs: # map dependencies variables to local variables - name: wasmbuildtestsContainsChange value: $[ dependencies.evaluate_paths.outputs['SetPathVars_wasmbuildtests.containsChange'] ] + - name: allWasmContainsChange + value: $[ dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'] ] jobParameters: isExtraPlatforms: ${{ variables.isExtraPlatformsBuild }} testGroup: innerloop @@ -332,7 +334,9 @@ jobs: buildArgs: -s mono+libs+host+packs+libs.tests -c $(_BuildConfig) /p:ArchiveTests=true /p:TestWasmBuildTests=true /p:TestAssemblies=false /p:BrowserHost=$(_hostedOs) timeoutInMinutes: 180 condition: >- - eq(dependencies.evaluate_paths.outputs['SetPathVars_wasmbuildtests.containsChange'], true) + or( + eq(dependencies.evaluate_paths.outputs['SetPathVars_wasmbuildtests.containsChange'], true), + eq(dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'], true)) # extra steps, run tests extraStepsTemplate: /eng/pipelines/libraries/helix.yml extraStepsParameters: @@ -342,7 +346,9 @@ jobs: scenarios: - buildwasmapps condition: >- - eq(variables['wasmbuildtestsContainsChange'], true) + or( + eq(variables['wasmbuildtestsContainsChange'], true), + eq(variables['allWasmContainsChange'], true), # # Build Browser_wasm, on windows, run console and browser tests @@ -361,6 +367,8 @@ jobs: value: $[ dependencies.evaluate_paths.outputs['SetPathVars_libraries.containsChange'] ] - name: monoContainsChange value: $[ dependencies.evaluate_paths.outputs['SetPathVars_mono.containsChange'] ] + - name: allWasmContainsChange + value: $[ dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'] ] jobParameters: testGroup: innerloop nameSuffix: LibraryTests @@ -370,6 +378,7 @@ jobs: or( eq(dependencies.evaluate_paths.outputs['SetPathVars_libraries.containsChange'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_mono.containsChange'], true), + eq(dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_installer.containsChange'], true)) # extra steps, run tests extraStepsTemplate: /eng/pipelines/libraries/helix.yml @@ -384,6 +393,7 @@ jobs: or( eq(variables['librariesContainsChange'], true), eq(variables['monoContainsChange'], true), + eq(variables['allWasmContainsChange'], true), eq(variables['isRollingBuild'], true)) # @@ -403,6 +413,8 @@ jobs: value: $[ dependencies.evaluate_paths.outputs['SetPathVars_libraries.containsChange'] ] - name: monoContainsChange value: $[ dependencies.evaluate_paths.outputs['SetPathVars_mono.containsChange'] ] + - name: allWasmContainsChange + value: $[ dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'] ] jobParameters: isExtraPlatforms: ${{ variables.isExtraPlatformsBuild }} testGroup: innerloop @@ -413,6 +425,7 @@ jobs: or( eq(dependencies.evaluate_paths.outputs['SetPathVars_libraries.containsChange'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_mono.containsChange'], true), + eq(dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_installer.containsChange'], true)) # extra steps, run tests extraStepsTemplate: /eng/pipelines/libraries/helix.yml @@ -424,8 +437,9 @@ jobs: - normal condition: >- or( - eq(variables['librariesContainsChange'], true), - eq(variables['monoContainsChange'], true)) + eq(variables['librariesContainsChange'], true), + eq(variables['monoContainsChange'], true), + eq(variables['allWasmContainsChange'], true)) # Wasm debugger tests - windows - template: /eng/pipelines/common/platform-matrix.yml @@ -440,6 +454,8 @@ jobs: # map dependencies variables to local variables - name: wasmdebuggertestsContainsChange value: $[ dependencies.evaluate_paths.outputs['SetPathVars_wasmdebuggertests.containsChange'] ] + - name: allWasmContainsChange + value: $[ dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'] ] jobParameters: testGroup: innerloop nameSuffix: Mono_DebuggerTests @@ -448,6 +464,7 @@ jobs: condition: >- or( eq(dependencies.evaluate_paths.outputs['SetPathVars_wasmdebuggertests.containsChange'], true), + eq(dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'], true), eq(variables['isRollingBuild'], true)) # extra steps, run tests extraStepsTemplate: /eng/pipelines/libraries/helix.yml @@ -471,6 +488,8 @@ jobs: # map dependencies variables to local variables - name: wasmdebuggertestsContainsChange value: $[ dependencies.evaluate_paths.outputs['SetPathVars_wasmdebuggertests.containsChange'] ] + - name: allWasmContainsChange + value: $[ dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'] ] jobParameters: testGroup: innerloop nameSuffix: Mono_DebuggerTests @@ -479,6 +498,7 @@ jobs: condition: >- or( eq(dependencies.evaluate_paths.outputs['SetPathVars_wasmdebuggertests.containsChange'], true), + eq(dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'], true), eq(variables['isRollingBuild'], true)) # extra steps, run tests extraStepsTemplate: /eng/pipelines/libraries/helix.yml diff --git a/eng/pipelines/runtime.yml b/eng/pipelines/runtime.yml index b854aafbff4f41..5cf1e8abff5db5 100644 --- a/eng/pipelines/runtime.yml +++ b/eng/pipelines/runtime.yml @@ -386,6 +386,8 @@ jobs: value: $[ dependencies.evaluate_paths.outputs['SetPathVars_libraries.containsChange'] ] - name: monoContainsChange value: $[ dependencies.evaluate_paths.outputs['SetPathVars_mono.containsChange'] ] + - name: allWasmContainsChange + value: $[ dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'] ] jobParameters: testGroup: innerloop nameSuffix: LibraryTests @@ -396,6 +398,7 @@ jobs: eq(dependencies.evaluate_paths.outputs['SetPathVars_libraries.containsChange'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_mono.containsChange'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_installer.containsChange'], true), + eq(dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'], true), eq(variables['isRollingBuild'], true)) # extra steps, run tests extraStepsTemplate: /eng/pipelines/libraries/helix.yml @@ -409,6 +412,7 @@ jobs: or( eq(variables['librariesContainsChange'], true), eq(variables['monoContainsChange'], true), + eq(variables['allWasmContainsChange'], true), eq(variables['isRollingBuild'], true)) # @@ -510,6 +514,8 @@ jobs: # map dependencies variables to local variables - name: wasmbuildtestsContainsChange value: $[ dependencies.evaluate_paths.outputs['SetPathVars_wasmbuildtests.containsChange'] ] + - name: allWasmContainsChange + value: $[ dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'] ] jobParameters: testGroup: innerloop nameSuffix: WasmBuildTests @@ -518,6 +524,7 @@ jobs: condition: >- or( eq(dependencies.evaluate_paths.outputs['SetPathVars_wasmbuildtests.containsChange'], true), + eq(dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'], true), eq(variables['isRollingBuild'], true)) # extra steps, run tests extraStepsTemplate: /eng/pipelines/libraries/helix.yml @@ -529,6 +536,7 @@ jobs: condition: >- or( eq(variables['wasmbuildtestsContainsChange'], true), + eq(variables['allWasmContainsChange'], true), eq(variables['isRollingBuild'], true)) # @@ -548,6 +556,8 @@ jobs: value: $[ dependencies.evaluate_paths.outputs['SetPathVars_libraries.containsChange'] ] - name: monoContainsChange value: $[ dependencies.evaluate_paths.outputs['SetPathVars_mono.containsChange'] ] + - name: allWasmContainsChange + value: $[ dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'] ] jobParameters: testGroup: innerloop nameSuffix: LibraryTests_EAT @@ -558,6 +568,7 @@ jobs: eq(dependencies.evaluate_paths.outputs['SetPathVars_libraries.containsChange'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_mono.containsChange'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_installer.containsChange'], true), + eq(dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'], true), eq(variables['isRollingBuild'], true)) # extra steps, run tests extraStepsTemplate: /eng/pipelines/libraries/helix.yml @@ -571,6 +582,7 @@ jobs: or( eq(variables['librariesContainsChange'], true), eq(variables['monoContainsChange'], true), + eq(variables['allWasmContainsChange'], true), eq(variables['isRollingBuild'], true)) # @@ -590,6 +602,8 @@ jobs: value: $[ dependencies.evaluate_paths.outputs['SetPathVars_libraries.containsChange'] ] - name: monoContainsChange value: $[ dependencies.evaluate_paths.outputs['SetPathVars_mono.containsChange'] ] + - name: allWasmContainsChange + value: $[ dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'] ] jobParameters: testGroup: innerloop nameSuffix: LibraryTests_AOT @@ -600,6 +614,7 @@ jobs: eq(dependencies.evaluate_paths.outputs['SetPathVars_libraries.containsChange'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_mono.containsChange'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_installer.containsChange'], true), + eq(dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'], true), eq(variables['isRollingBuild'], true)) # extra steps, run tests extraStepsTemplate: /eng/pipelines/libraries/helix.yml @@ -613,6 +628,7 @@ jobs: or( eq(variables['librariesContainsChange'], true), eq(variables['monoContainsChange'], true), + eq(variables['allWasmContainsChange'], true), eq(variables['isRollingBuild'], true)) # @@ -637,6 +653,8 @@ jobs: value: 10 - name: timeoutPerTestCollectionInMinutes value: 200 + - name: allWasmContainsChange + value: $[ dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'] ] jobParameters: testGroup: innerloop nameSuffix: AllSubsets_Mono_RuntimeTests @@ -646,6 +664,7 @@ jobs: or( eq(dependencies.evaluate_paths.outputs['SetPathVars_mono.containsChange'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_runtimetests.containsChange'], true), + eq(dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'], true), eq(variables['isRollingBuild'], true)) extraStepsTemplate: /eng/pipelines/common/templates/runtimes/wasm-runtime-and-send-to-helix.yml extraStepsParameters: From c53495685bb00e93a61069c866cd5618c2db7596 Mon Sep 17 00:00:00 2001 From: Ankit Jain Date: Wed, 16 Feb 2022 14:58:36 -0500 Subject: [PATCH 2/5] fix yml --- eng/pipelines/common/evaluate-default-paths.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/eng/pipelines/common/evaluate-default-paths.yml b/eng/pipelines/common/evaluate-default-paths.yml index e0f2c40bcda9a2..d2589666773451 100644 --- a/eng/pipelines/common/evaluate-default-paths.yml +++ b/eng/pipelines/common/evaluate-default-paths.yml @@ -125,6 +125,7 @@ jobs: - src/mono/wasm/runtime/* - src/mono/mono/* - subset: allwasm + include: - eng/Version.Details.xml - src/mono/wasm/emscripten-version.txt From 245ce7714b4a0c6d4ac3cea807f9dab710fce278 Mon Sep 17 00:00:00 2001 From: Ankit Jain Date: Fri, 18 Feb 2022 11:34:26 -0500 Subject: [PATCH 3/5] Fix merge --- eng/pipelines/common/templates/wasm-build-tests.yml | 3 +++ eng/pipelines/common/templates/wasm-debugger-tests.yml | 3 +++ eng/pipelines/common/templates/wasm-library-tests.yml | 3 +++ eng/pipelines/common/templates/wasm-runtime-tests.yml | 3 +++ 4 files changed, 12 insertions(+) diff --git a/eng/pipelines/common/templates/wasm-build-tests.yml b/eng/pipelines/common/templates/wasm-build-tests.yml index cd142b55a3ea13..54926c5da253f2 100644 --- a/eng/pipelines/common/templates/wasm-build-tests.yml +++ b/eng/pipelines/common/templates/wasm-build-tests.yml @@ -19,6 +19,8 @@ jobs: # map dependencies variables to local variables - name: wasmbuildtestsContainsChange value: $[ dependencies.evaluate_paths.outputs['SetPathVars_wasmbuildtests.containsChange'] ] + - name: allWasmContainsChange + value: $[ dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'] ] - name: alwaysRunVar value: ${{ parameters.alwaysRun }} jobParameters: @@ -30,6 +32,7 @@ jobs: condition: >- or( eq(variables['alwaysRunVar'], true), + eq(dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_wasmbuildtests.containsChange'], true)) # extra steps, run tests extraStepsTemplate: /eng/pipelines/libraries/helix.yml diff --git a/eng/pipelines/common/templates/wasm-debugger-tests.yml b/eng/pipelines/common/templates/wasm-debugger-tests.yml index 9d5bb43606608a..49ad67739f0cb5 100644 --- a/eng/pipelines/common/templates/wasm-debugger-tests.yml +++ b/eng/pipelines/common/templates/wasm-debugger-tests.yml @@ -17,6 +17,8 @@ jobs: # map dependencies variables to local variables - name: wasmdebuggertestsContainsChange value: $[ dependencies.evaluate_paths.outputs['SetPathVars_wasmdebuggertests.containsChange'] ] + - name: allWasmContainsChange + value: $[ dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'] ] - name: alwaysRunVar value: ${{ parameters.alwaysRun }} jobParameters: @@ -28,6 +30,7 @@ jobs: condition: >- or( eq(variables['alwaysRunVar'], true), + eq(dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_wasmdebuggertests.containsChange'], true)) # extra steps, run tests extraStepsTemplate: /eng/pipelines/libraries/helix.yml diff --git a/eng/pipelines/common/templates/wasm-library-tests.yml b/eng/pipelines/common/templates/wasm-library-tests.yml index 56bf0740cd64ce..539e2ffa693590 100644 --- a/eng/pipelines/common/templates/wasm-library-tests.yml +++ b/eng/pipelines/common/templates/wasm-library-tests.yml @@ -28,6 +28,8 @@ jobs: value: $[ dependencies.evaluate_paths.outputs['SetPathVars_mono.containsChange'] ] - name: alwaysRunVar value: ${{ parameters.alwaysRun }} + - name: allWasmContainsChange + value: $[ dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'] ] jobParameters: isExtraPlatforms: ${{ parameters.isExtraPlatformsBuild }} testGroup: innerloop @@ -41,6 +43,7 @@ jobs: or( eq(variables['alwaysRunVar'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_libraries.containsChange'], true), + eq(dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_mono.containsChange'], true)) # extra steps, run tests extraStepsTemplate: /eng/pipelines/libraries/helix.yml diff --git a/eng/pipelines/common/templates/wasm-runtime-tests.yml b/eng/pipelines/common/templates/wasm-runtime-tests.yml index 58ace9127f9e68..aa7e4257764bde 100644 --- a/eng/pipelines/common/templates/wasm-runtime-tests.yml +++ b/eng/pipelines/common/templates/wasm-runtime-tests.yml @@ -16,6 +16,8 @@ jobs: runtimeFlavor: mono platforms: ${{ parameters.platforms }} variables: + - name: allWasmContainsChange + value: $[ dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'] ] - name: alwaysRunVar value: ${{ parameters.alwaysRun }} - name: timeoutPerTestInMinutes @@ -32,6 +34,7 @@ jobs: or( eq(variables['alwaysRunVar'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_mono.containsChange'], true), + eq(dependencies.evaluate_paths.outputs['SetPathVars_allwasm.containsChange'], true), eq(dependencies.evaluate_paths.outputs['SetPathVars_runtimetests.containsChange'], true)) extraStepsTemplate: /eng/pipelines/common/templates/runtimes/wasm-runtime-and-send-to-helix.yml extraStepsParameters: From 88e333b0908b68ef650e3b2f0fa7c0210e967268 Mon Sep 17 00:00:00 2001 From: Ankit Jain Date: Fri, 18 Feb 2022 16:17:21 -0500 Subject: [PATCH 4/5] [wasm] Pass isRollingBuild to runtime-extra-platforms-wasm template, to correctly trigger the EAT/AOT builds --- eng/pipelines/runtime-extra-platforms.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/eng/pipelines/runtime-extra-platforms.yml b/eng/pipelines/runtime-extra-platforms.yml index 025d5d27e462ca..226b563d3036cd 100644 --- a/eng/pipelines/runtime-extra-platforms.yml +++ b/eng/pipelines/runtime-extra-platforms.yml @@ -38,6 +38,7 @@ jobs: - template: /eng/pipelines/runtime-extra-platforms-wasm.yml parameters: isExtraPlatformsBuild: ${{ variables.isExtraPlatformsBuild }} + isRollingBuild: ${{ variables.isRollingBuild }} # Any jobs that are not specific to any platform - ${{ if eq(variables.isNotSpecificPlatformOnlyBuild, true) }}: From 28da030726159fce983157adccdf35599179fb9b Mon Sep 17 00:00:00 2001 From: Ankit Jain Date: Fri, 18 Feb 2022 16:20:24 -0500 Subject: [PATCH 5/5] .. and pass isWasmOnlyBuild also --- eng/pipelines/runtime-extra-platforms.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/eng/pipelines/runtime-extra-platforms.yml b/eng/pipelines/runtime-extra-platforms.yml index 226b563d3036cd..02591d91b6b0e3 100644 --- a/eng/pipelines/runtime-extra-platforms.yml +++ b/eng/pipelines/runtime-extra-platforms.yml @@ -37,7 +37,7 @@ jobs: # include this unconditionally, because it has per job conditions - template: /eng/pipelines/runtime-extra-platforms-wasm.yml parameters: - isExtraPlatformsBuild: ${{ variables.isExtraPlatformsBuild }} + isWasmOnlyBuild: ${{ variables.isWasmOnlyBuild }} isRollingBuild: ${{ variables.isRollingBuild }} # Any jobs that are not specific to any platform