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.

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

Revert "Revert "CI: Include stdlib build for Wasm in the Linux buildbot"" #77872

Conversation

MaxDesiatov
Copy link
Contributor

@MaxDesiatov MaxDesiatov commented Nov 29, 2024

Reverts #76597

After performance improvements landed in WasmKit, this no longer introduces a 1hr regression, but only 10-20 minutes slowdown to Ubuntu jobs.

Additional solutions are investigated to shave time off SwiftPM build jobs, which could potentially reduce total build time by 5-10 minutes in swiftlang/swift-package-manager#8145.

@MaxDesiatov
Copy link
Contributor Author

@swift-ci build toolchain

@MaxDesiatov
Copy link
Contributor Author

@swift-ci smoke test

@MaxDesiatov MaxDesiatov marked this pull request as ready for review November 29, 2024 16:41
@MaxDesiatov
Copy link
Contributor Author

@swift-ci test webassembly

@MaxDesiatov
Copy link
Contributor Author

@swift-ci smoke test

@MaxDesiatov
Copy link
Contributor Author

@swift-ci test webassembly

@MaxDesiatov
Copy link
Contributor Author

@swift-ci build toolchain

@MaxDesiatov MaxDesiatov added continuous integration WebAssembly Platform: WebAssembly WASI WebAssembly System Interface support labels Dec 2, 2024
@MaxDesiatov
Copy link
Contributor Author

@swift-ci test

@shahmishal shahmishal merged commit 34dffc5 into main Dec 4, 2024
9 checks passed
@shahmishal shahmishal deleted the revert-76597-revert-72010-yt/include-wasm-build-linux-buildbot branch December 4, 2024 17:12
MaxDesiatov added a commit that referenced this pull request Dec 5, 2024
* **Explanation**: "Swift Test Linux Platform" jobs on ci.swift.org required as PR checks should cross-compile and run tests for WebAssembly. Effectively, what was previously an optional `@swift-ci test webassembly` trigger should become an implicit and required part of PR testing to prevent occasional regressions in Wasm and WASI support.
* **Scope**: Only affects the WASI platform
* **Risk**: Low. It increases job run time by ~10% and this platform has active maintainers.
* **Testing**: stdlib test suite is now enabled.
* **Issue**: N/A
* **Reviewer**:  @shahmishal 
* **Main branch PR**: #77872
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
continuous integration WASI WebAssembly System Interface support WebAssembly Platform: WebAssembly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants