Flaky test: tests/migrate/test_memory_import.py::test_routes sees the import still running on Windows CI
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 76/100
Hướng nghiên cứu
Đọc tests/migrate/test_memory_import.py, đặc biệt là test_routes quanh dòng 497, và xem cách kiểm tra này thăm dò GET /api/import/memory/job sau khi áp dụng quá trình nhập. Chạy kiểm tra đó và xác minh rằng nó chờ hoàn tất một cách đáng tin cậy, thay vì thất bại khi tác vụ nền mất nhiều thời gian hơn trên Windows. Kết quả mong đợi là kiểm tra vượt qua mà không cần chạy lại một cách chập chờn.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
tests/migrate/test_memory_import.py::test_routes failed once on Windows CI and passed on a re-run.
Where: branch feat/386-outcome-checks-2, run 38908685415 attempt 1, Engine tests (windows-latest): 1 failed, 2304 passed in 20:11.
What failed: line 497, assert now["id"] == job["id"] and now["status"] == "done" and now["facts"] == 3: the import job was still running.
Likely cause: after POST /api/import/chatgpt/apply the test polls GET /api/import/memory/job at most 200 times with time.sleep(0.02), about 4 seconds plus request time. The import runs in the background on the app's event loop and writes memories to SQLite; on a busy Windows runner with -n auto that took longer than the polling budget. Nothing was wrong with the result, only the wait.
Possible fix: poll against a deadline (e.g. 30 s) rather than a fixed number of 20 ms sleeps, or wait on the job's completion event.
Until fixed it is marked @pytest.mark.flaky(reruns=2), and CI's rerun summary lists it whenever it needs a rerun.
- Ngôn ngữ chính
- Python
- Star
- 702
- Fork
- 113
- Merge trung bình
- 5 giờ 8 phút
- Pull request đã merge (30 ngày)
- 171
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của existence-master/Sentient
-
area: channels enhancement good first issue priority: P2 size: S
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
existence-master/Sentient#457 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area: desktop bug good first issue priority: P2 size: S
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
existence-master/Sentient#454 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area: desktop bug priority: P2 size: S
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
existence-master/Sentient#440 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area: desktop bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
existence-master/Sentient#426 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area: browser bug flaky
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
existence-master/Sentient#421 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của existence-master/Sentient
Issue tương tự
-
documentation
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
ianktoo/project-monitor#14 ·
-
A wrong descriptor checksum is refused with the right one, also for an xprv or WIF descriptorĐang mởsecurity
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
btclib-org/btclib-wallet#478 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
security
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
btclib-org/btclib#2727 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
good first issue lead variant:container
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
radialmonster/orf-cleanroom#10 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
gdsa opencode: keep-alive cleanup trap fails with 'keepalive_pid: unbound variable'Có thể đã có người làm @darribas đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100