testing: the wall-time scheduler case flakes under a full run

Đang mở
  • Có gì không ổn?
#924 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 2 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
65/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
rust
Lĩnh vực
testing

Hướng nghiên cứu

Bắt đầu tại crates/flexiq-core/src/scheduler/mod.rs, ở scheduler::tests::test_outcomes_carry_measured_wall_time và get_job unwrap quanh dòng 1229. Chạy cargo test --workspace, sau đó chạy test với --exact để tái hiện sự khác biệt và kiểm tra thứ tự sweep, archival và claim/complete của scheduler. Hoàn tất khi test wall-time vẫn đáng tin cậy trong toàn bộ test suite và báo assertion thay vì một unwrap panic khó hiểu.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

P2 rust tests

scheduler::tests::test_outcomes_carry_measured_wall_time fails intermittently under a full cargo test --workspace, and passes in isolation and on a clean re-run.

---- scheduler::tests::test_outcomes_carry_measured_wall_time stdout ----
thread 'scheduler::tests::test_outcomes_carry_measured_wall_time' panicked at crates/flexiq-core/src/scheduler/mod.rs:1229:34:
called `Result::unwrap()` on an `Err` value: JobNotFound("01a08e22-66df-7132-ad29-ef0e61e41642")

Seen once in 479 tests during the #916 gate, then green on a re-run of the same suite and green with --exact on its own. The branch it surfaced on had an empty diff against crates/flexiq-core/.

Where to look

scheduler/mod.rs:1229 unwraps a get_job inside test scaffolding rather than in the scheduler itself. Under load the job appears to be gone by the time the assertion reads it back — the likely candidates being a concurrent sweep or archival having moved it, or a claim/complete ordering the test assumes but does not enforce.

Two things worth fixing regardless of the root cause:

  • The unwrap is the reason this reads as a panic rather than an assertion. A test reading a job it expects to exist should say so, so a failure names the expectation instead of the Result.
  • A test whose subject is wall-clock measurement is a likely candidate for timing assumptions that hold on an idle machine and not on a loaded one.

Low priority — it does not gate anything today and re-runs pass — but it is the kind of flake that eventually costs someone an afternoon on an unrelated PR, which is how it surfaced here.

Ngôn ngữ chính
Rust
Star
20
Fork
2
Merge trung bình
1 ngày 1 giờ
Pull request đã merge (30 ngày)
11

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của ByteVeda/flexiq

Tất cả issue của ByteVeda/flexiq

Issue tương tự

Thêm issue về Rust

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.