Skip to content

Conversation

@Guest0x0
Copy link
Collaborator

@Guest0x0 Guest0x0 commented Oct 24, 2025

Recently the sanitizer check CI keeps failing randomly. Address sanitizer slows down the program, which makes the test program fail to finish all ready tasks before the next round of timeout expiration, eventually result in very high latency and incorrect test result. The solution here is to add a max concurrent test restriction to tests in moonbitlang/async. Currently the value is set to 20.

Another failing test is walk_test.mbt in @fs. With the recent introduction of README.mbt.md for the @fs package, a lot of tests in the README will be running at the same time at the beginning, again causing high latency. Since there is only one time-sensitive test in @fs, the solution here is to add a delay before that time-sensitive test, so that it won't race with the other tests without timeout.

@Guest0x0 Guest0x0 marked this pull request as draft October 24, 2025 09:45
@Guest0x0 Guest0x0 force-pushed the debug-CI-failure branch 2 times, most recently from 8453ec7 to d04ed24 Compare October 27, 2025 09:22
@Guest0x0 Guest0x0 changed the title debug CI faliure fix sanitizer check failure Oct 27, 2025
@Guest0x0 Guest0x0 changed the title fix sanitizer check failure fix sanitizer check failure via rate limit Oct 27, 2025
@coveralls
Copy link

coveralls commented Oct 27, 2025

Pull Request Test Coverage Report for Build 688

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 78.543%

Totals Coverage Status
Change from base Build 682: 0.0%
Covered Lines: 1164
Relevant Lines: 1482

💛 - Coveralls

@Guest0x0 Guest0x0 marked this pull request as ready for review October 27, 2025 09:29
@Guest0x0 Guest0x0 merged commit 6f67902 into main Oct 27, 2025
10 checks passed
@Guest0x0 Guest0x0 deleted the debug-CI-failure branch October 27, 2025 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants