Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

ensure results are returned even when `code-review` task doesn't run automatically

Đang mở
  • Có gì không ổn?
#3,740 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 1 ngày

@bhearsum đang làm issue này rồi.

Từ ngày 21/10/2026.

  • #3751 của @bhearsum — đang mở

Đánh giá

Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
Mức phù hợp với người mới
25/100
Loại issue
Tính năng
Độ rõ ràng
Cần làm rõ
Mức độ hoạt động
Sôi nổi
Công nghệ
python
Lĩnh vực
ci-cd, devtools

Hướng nghiên cứu

Start by reading the discussion in pull request 3720 and trace how the code-review task and publication task are scheduled when upstream tasks do not run. Verify the behavior of Taskcluster's taskGroupResolved exchange and compare it with the cron option described here. Done means a chosen approach reliably publishes results and resolves Phabricator builds when an upstream task is skipped.

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

Mô tả

Some discussion in https://github.com/mozilla/code-review/pull/3720 has highlighted the fact that it's possible for us never to publish status of a phab revision or github pull request. Because this happens in the publication task, and that publication task only runs when a code-review task completes, any issues that cause any of the direct upstream of that task not to run will result in the publication task never running. This, in turn, causes no feedback to be posted to github or phabricator, and worse yet, builds on phabricator not to be resolved. This is a more likely problem for builds and tests, but it is still possible to happen for linting as well.

It's not immediately clear to me how to deal with this. There is no way that I'm aware of to make the code-review tasks run when a direct upstream does not. Some ideas could be:

  • Run something on a cron schedule that looks for reviewbot task groups that are not doing anything, but haven't run code-review. Then, somehow kick that task to run, or directly schedule the publication task for them. The main downside to this is that unless we run this very frequently, we delay results.
  • Fire something on task-group-resolved. I believe this fires whenever a task group is idle (even if not all tasks have completed) - but this needs to be verified. This event doesn't have enough information in its payload to immediately fire a publication task; we'd need to run some small task to determine if it's a group we care about, and then to force the code-review task to run or directly schedule the publication task if it is.
Ngôn ngữ chính
Python
Star
68
Fork
53
Merge trung bình
1 ngày 13 giờ
Pull request đã merge (30 ngày)
52

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 mozilla/code-review

Tất cả issue của mozilla/code-review

Issue tương tự

Thêm issue về Python

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.