CI: a superseded dispatched run's pending status can overwrite the newer run's result (typescript.yml, python.yml)
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
- 82/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- github-actions, typescript
- Lĩnh vực
- ci-cd
Hướng nghiên cứu
Bắt đầu bằng cách so sánh dispatch-status-pending trong typescript.yml và python.yml với bước kiểm tra lần chạy mới hơn trong dispatch-status, sau đó tìm script pending mà các job đó chạy. Thêm bước kiểm tra tương tự vào luồng pending và chạy script pending với một gh giả lập như mô tả trong issue. Hoàn tất khi không đăng trạng thái pending nếu có một lần chạy mới hơn được dispatch, và bài kiểm thử bao quát trường hợp đó.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
In typescript.yml and python.yml, the dispatch-status-pending job posts its (dispatched) status as pending without first checking for a newer dispatched run on the same commit. The dispatch-status result job does check (#5178), and it posts nothing when a newer run exists.
How it happens
- Run A is dispatched on commit X, and then run B.
- B finishes and posts
success(orfailure). - A's pending job runs after that and posts
pending. Two ways this can happen:- A is re-run after B finished. A re-run keeps its run ID, so it is still the older run.
- A's pending job starts late, which the ordering allows when the two runs are not in one concurrency group.
- A's result job finds B, the newer run, and exits without posting.
The status stays pending until someone dispatches the workflow again.
Expected
The pending job runs the same newer-run check as the result job and posts nothing when it finds one. A test runs the pending script against a stub gh to pin this.
Found by Copilot on #5222. That PR fixes the two reporters it adds (dco.yml, everything-mcp-diff.yml). This issue covers the two existing ones, which that PR leaves alone.
- Ngôn ngữ chính
- TypeScript
- Star
- 91.4k
- Fork
- 11.8k
- Merge trung bình
- 7 giờ 26 phút
- Pull request đã merge (30 ngày)
- 130
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 modelcontextprotocol/servers
-
git: README should state plainly that without --repository every reachable repository is in scopeĐang mởdocumentation server-git v2
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 84/100
modelcontextprotocol/servers#5225 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug v2
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
modelcontextprotocol/servers#5217 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
filesystem: search_files does not expand non-wildcard excludePatterns like directory_tree doesĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
modelcontextprotocol/servers#5216 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug v2
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 79/100
modelcontextprotocol/servers#5215 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
documentation server-git server-time v2
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
modelcontextprotocol/servers#5213 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của modelcontextprotocol/servers
Issue tương tự
-
level/task type/bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
wazuh/wazuh-dashboard-plugins#9508 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Testnet EOL page links Zonescan mainnetCó thể đã có người làm @napelvs đã nhận hôm nay. Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 76/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Azure-Samples/azure-search-openai-demo#3249 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
[Docs] docs/architecture.mermaid leaves out 21 of the 51 src/ directoriesCó thể đã có người làm @uyg7x đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement status:needs-triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
dongdongbh/Mindwtr#1438 ·
Maintainer thường phản hồi trong vòng 1 ngày