evaluate-default-paths.yml: Add paths for wasi (#81817)
authorAnkit Jain <radical@gmail.com>
Thu, 9 Feb 2023 01:20:08 +0000 (20:20 -0500)
committerGitHub <noreply@github.com>
Thu, 9 Feb 2023 01:20:08 +0000 (20:20 -0500)
* evaluate-default-paths.yml: Add paths for wasi

.. and exclude wasm specific paths from coreclr.

* disable runtime-wasm-optional automatic trigger for PRs

* CI: skip debugger tests on runtime-wasm if they run on runtime pipeline

.. too.

* fix condition

* address review feedback from @kg

* Fix yml issue

Using `Was?Runner*` failed to validate on azdo with:

`/eng/pipelines/common/evaluate-default-paths.yml (Line: 10, Col: 24): While parsing a flow sequence, did not find expected ',' or ']'.`

* fix syntax error in yml

eng/pipelines/common/evaluate-default-paths.yml
eng/pipelines/common/templates/wasm-debugger-tests.yml
eng/pipelines/extra-platforms/runtime-extra-platforms-wasm.yml
eng/pipelines/runtime-wasm-optional.yml

index fd0697b..d77e8a9 100644 (file)
@@ -7,9 +7,14 @@ parameters:
   _const_paths:
     _wasm_specific_only: [
         eng/testing/ProvisioningVersions.props
+        eng/testing/WasmRunner*
+        eng/testing/WasiRunner*
         eng/testing/scenarios/BuildWasmAppsJobsList.txt
-        eng/testing/tests.wasm.targets
+        eng/testing/tests.browser.targets
+        eng/testing/tests.was*.targets
+        eng/testing/was*provisioning.targets
         src/libraries/sendtohelix-wasm.targets
+        src/libraries/sendtohelix-wasi.targets
         src/mono/mono/**/*wasm*
         src/mono/nuget/Microsoft.NET.Runtime.WebAssembly.Sdk/*
         src/mono/nuget/Microsoft.NET.Runtime.wasm.Sample.Mono/*
@@ -70,6 +75,7 @@ jobs:
       - eng/pipelines/installer/*
       - eng/pipelines/mono/*
       - eng/pipelines/libraries/*
+      - ${{ parameters._const_paths._wasm_specific_only }}
       - ${{ parameters._const_paths._wasm_pipelines }}
       - ${{ parameters._const_paths._always_exclude }}
 
index 994b169..4d8ec6e 100644 (file)
@@ -4,6 +4,7 @@ parameters:
   isWasmOnlyBuild: false
   browser: 'chrome'
   shouldContinueOnError: false
+  runOnlyOnWasmOnlyPipelines: false
   extraBuildArgs: ''
   nameSuffix: ''
   platforms: []
@@ -23,6 +24,8 @@ jobs:
       # map dependencies variables to local variables
       - name: alwaysRunVar
         value: ${{ parameters.alwaysRun }}
+      - name: runOnlyOnWasmPipelinesVar
+        value: ${{ parameters.runOnlyOnWasmOnlyPipelines }}
       - name: shouldRunOnDefaultPipelines
         value: $[
           or(
@@ -47,7 +50,9 @@ jobs:
       condition: >-
         or(
           eq(variables['alwaysRunVar'], true),
-          eq(variables['isDefaultPipeline'], variables['shouldRunOnDefaultPipelines']))
+          and(
+            eq(variables['isDefaultPipeline'], variables['shouldRunOnDefaultPipelines']),
+            ne(variables['runOnlyOnWasmPipelinesVar'], 'true')))
       extraStepsTemplate: /eng/pipelines/libraries/helix.yml
       extraStepsParameters:
         creator: dotnet-bot
index d1b25fd..de1be93 100644 (file)
@@ -241,6 +241,7 @@ jobs:
       alwaysRun: ${{ parameters.isWasmOnlyBuild }}
       isExtraPlatformsBuild: ${{ parameters.isExtraPlatformsBuild }}
       isWasmOnlyBuild: ${{ parameters.isWasmOnlyBuild }}
+      runOnlyOnWasmOnlyPipelines: true
 
   # Disable for now
   #- template: /eng/pipelines/coreclr/perf-wasm-jobs.yml
index 86840d3..a1392e6 100644 (file)
@@ -1,8 +1,5 @@
-# This is a wrapper yml for `extra-platforms/runtime-extra-platforms-wasm.yml`, which
-# has all the wasm jobs. This file is essentially so we can have point
-# the pipeline in azdo UI to this, and thus avoid any scheduled triggers
-
 trigger: none
+pr: none
 
 variables:
   - template: /eng/pipelines/common/variables.yml