Skip to content
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

Jump to bottom

CI: Reduce the unrelated jobs running on wasm only PRs #74601

Merged
merged 17 commits into from
Sep 1, 2022
Merged

CI: Reduce the unrelated jobs running on wasm only PRs #74601

merged 17 commits into from
Sep 1, 2022

Conversation

radical
Copy link
Member

@radical radical commented Aug 25, 2022 •
edited
Loading

  • For PRs with strictly wasm only changes, disables running non-wasm jobs like mobile, or coreclr
  • For Wasm.Build.Tests, and debugger only PRs, runs only those jobs
  • Run coreclr jobs only if the PR has non-mono/wasm changes

@radical radical added NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) NO-REVIEW Experimental/testing PR, do NOT review it area-Infrastructure labels Aug 25, 2022
@ghost ghost assigned radical Aug 25, 2022
@ghost
Copy link

ghost commented Aug 25, 2022

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

Issue Details

null

Author: radical
Assignees: -
Labels:

NO-MERGE, NO-REVIEW, area-Infrastructure
Milestone: -

@radical
Copy link
Member Author

radical commented Aug 26, 2022

/azp run runtime-extra-platforms

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@radical radical force-pushed the ci-job-fix-onlywasm branch 7 times, most recently from 74f71dd to a0d0472 Compare August 30, 2022 21:43
@radical radical changed the title [IGNORE][wasm] Testing reducing CI load CI: Reduce the unrelated jobs running on wasm only PRs Aug 30, 2022
@radical radical removed NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) NO-REVIEW Experimental/testing PR, do NOT review it labels Aug 30, 2022
@radical radical marked this pull request as ready for review August 30, 2022 21:47
@radical radical mentioned this pull request Aug 30, 2022
Open
23 tasks
@radical
Copy link
Member Author

radical commented Aug 30, 2022

/azp run runtime-wasm,runtime-extra-platforms

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@lewing lewing closed this Aug 31, 2022
@lewing lewing reopened this Aug 31, 2022
@steveisok steveisok requested a review from a team August 31, 2022 02:03
@radical
Copy link
Member Author

radical commented Aug 31, 2022

/azp run runtime-wasm,runtime-extra-platforms

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@radical
CI: move isDefaultPipeline to variables.yml
5a9819f
@radical
CI: add support for running wasm jobs as only libtests, and only non-…
0688e04 
…libtests
@radical
Update README with CI matrix for wasm
7cf965a
@radical
[wasm] Fix timeout for runtime tests to 180, which was changed earlie…
4f51e01 
…r by mistake
@radical
Merge remote-tracking branch 'origin/main' into wasm-libtests-ci-2
b220c9a
@radical
evaluate-default-paths: update, and rename some subsets, and track ch…
d9a3c53 
…anges in ymls
@radical
Enable evaluate-paths job for runtime-dev-innerloop, and dotnet-linke…
0326236 
…r-tests pipelines
@radical
Add conditions for coreclr, and other non-mono jobs
a37bb2c
@radical
evaluate-default-paths.yml: naming change
0cf18c0 
mono -> mono_excluding_wasm
non_mono -> non_mono_and_wasm

this helps simplify the conditions, and make it easier to read.
@radical
cleanup
513a56e
@radical
cleanup
faea7e3
@radical
cleanup
f028458
@radical
fix typo
20f634d
@radical
fix merge
438769f
@radical
dummy
5c3bb74
@radical
fix merge
7d23598
@radical
Merge remote-tracking branch 'origin/main' into ci-job-fix-onlywasm
36bb4c8
@radical
Copy link
Member Author

radical commented Aug 31, 2022

Rebased on main.

@radical
Copy link
Member Author

radical commented Aug 31, 2022

/azp run runtime-wasm-libtests,runtime-wasm-non-libtests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@radical
Copy link
Member Author

radical commented Aug 31, 2022

runtime-wasm-libtests, and runtime-wasm-non-libtests runs can be ignored here. I ran them just to confirm that the correct jobs are run, and skipped.

@radical radical merged commit d684343 into dotnet:main Sep 1, 2022
@radical radical deleted the ci-job-fix-onlywasm branch September 1, 2022 01:51
@fanyang-mono fanyang-mono mentioned this pull request Sep 15, 2022
Closed
@ghost ghost locked as resolved and limited conversation to collaborators Oct 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@lewing lewing lewing approved these changes

@steveisok steveisok steveisok approved these changes

@akoeplinger akoeplinger Awaiting requested review from akoeplinger akoeplinger is a code owner

@marek-safar marek-safar Awaiting requested review from marek-safar

Assignees

@radical radical

Labels
area-Infrastructure
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@radical @lewing @steveisok