Skip to content

[Wasm] Add AOT CI Leg #46651

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 19 commits into from
Jan 20, 2021
Merged

[Wasm] Add AOT CI Leg #46651

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
1753bca
Quick add of a wasm aot leg
Jan 6, 2021
abea0a1
Try to get more legs to run?
Jan 6, 2021
f7f5046
yml tweak
Jan 6, 2021
a0b9e2c
Add initial wasm AOT library test project exclusions
Jan 7, 2021
5384618
Add a few more project exclusions
Jan 8, 2021
82d65db
Add all other test suites failing on wasm AOT CI leg
Jan 8, 2021
7eff656
Merge remote-tracking branch 'upstream/master' into wasm-aot-ci-leg
Jan 11, 2021
20d6e82
Fix AllSubsets_Mono AOT condition to run lane
Jan 11, 2021
e233cf5
Temporarily include linker update and remote executor exlusion to see…
Jan 11, 2021
d2a29e7
Add another project exclusion
Jan 11, 2021
433ff25
Merge remote-tracking branch 'upstream/master' into wasm-aot-ci-leg
Jan 12, 2021
ad7fab0
Run small set of test suites on CI
Jan 12, 2021
51c9a5c
Merge remote-tracking branch 'upstream/master' into wasm-aot-ci-leg
Jan 12, 2021
fa0f75a
Merge remote-tracking branch 'upstream/master' into wasm-aot-ci-leg
Jan 13, 2021
322daf3
Reduce number of AOT tests on CI to one
Jan 13, 2021
c7173ec
Move AOT CI lane to staging
Jan 13, 2021
a87bfaa
Remove temporary change that ran the CI lane
Jan 13, 2021
c3c8d4b
Merge remote-tracking branch 'upstream/master' into wasm-aot-ci-leg
Jan 19, 2021
6f60613
Fix lane condition
Jan 19, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
41 changes: 41 additions & 0 deletions eng/pipelines/runtime.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -271,6 +271,47 @@ jobs:
eq(variables['monoContainsChange'], true),
eq(variables['isFullMatrix'], true))

#
# Build the whole product using Mono and run libraries tests, multi-scenario
#
- template: /eng/pipelines/common/platform-matrix.yml
Copy link
Member

@safern safern Jan 7, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would suggest adding this to runtime-staging once this is ready to be merged (I know we're on early process gathering info on how many tests fail, etc) to get it stable.

Copy link
Member

@mdh1418 mdh1418 Jan 13, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The lane passed after running just System.Buffers.Tests.csproj :)
https://dev.azure.com/dnceng/public/_build/results?buildId=948229&view=logs&j=f217f715-90ad-5404-53bd-b93e3b80f465&t=9df76bd4-3041-5537-002b-9e29a53263b9
Moved the block to runtime-staging.yml

parameters:
jobTemplate: /eng/pipelines/common/global-build-job.yml
helixQueuesTemplate: /eng/pipelines/libraries/helix-queues-setup.yml
buildConfig: Release
runtimeFlavor: mono
platforms:
- Browser_wasm
variables:
# map dependencies variables to local variables
- name: librariesContainsChange
value: $[ dependencies.evaluate_paths.outputs['SetPathVars_libraries.containsChange'] ]
- name: monoContainsChange
value: $[ dependencies.evaluate_paths.outputs['SetPathVars_mono.containsChange'] ]
jobParameters:
testGroup: innerloop
nameSuffix: AllSubsets_Mono_AOT
buildArgs: -s mono+libs+host+packs+libs.tests -c $(_BuildConfig) /p:ArchiveTests=true /p:EnableAggressiveTrimming=true /p:RunAOTCompilation=true
timeoutInMinutes: 120
condition: >-
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_installer.containsChange'], true),
eq(variables['isFullMatrix'], true))
# extra steps, run tests
extraStepsTemplate: /eng/pipelines/libraries/helix.yml
extraStepsParameters:
creator: dotnet-bot
testRunNamePrefixSuffix: Mono_$(_BuildConfig)
scenarios:
- normal
#- wasmtestonbrowser
condition: >-
or(
eq(variables['librariesContainsChange'], true),
eq(variables['monoContainsChange'], true),
eq(variables['isFullMatrix'], true))

#
# Build the whole product using Mono and run libraries tests
Expand Down
2 changes: 1 addition & 1 deletion src/mono/wasm/runtime/driver.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -469,7 +469,7 @@ mono_wasm_load_runtime (const char *unused, int debug_level)

#ifdef DEBUG
monoeg_g_setenv ("MONO_LOG_LEVEL", "debug", 0);
monoeg_g_setenv ("MONO_LOG_MASK", "gc", 0);
monoeg_g_setenv ("MONO_LOG_MASK", "all", 0);
// Setting this env var allows Diagnostic.Debug to write to stderr. In a browser environment this
// output will be sent to the console. Right now this is the only way to emit debug logging from
// corlib assemblies.
Expand Down