ensure results are returned even when `code-review` task doesn't run automatically
メンテナーはふだん 1 日以内に返信
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 25/100
調査の方向性
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.
索引モデルが issue の本文から書いたものです。
説明
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-reviewtask to run or directly schedule the publication task if it is.
- 主要言語
- Python
- スター
- 68
- フォーク
- 53
- 平均マージ
- 1日 13時間
- マージ済み PR（30日）
- 52
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
mozilla/code-review のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
mozilla/code-review#3399 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 55/100
mozilla/code-review#3739 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
mozilla/code-review#3738 ·
メンテナーはふだん 1 日以内に返信
-
support performance profiles in tasks対応中かも @La0 が 2 日前に担当しました。 オープン
難易度 4/5 3〜5日 初心者へのやさしさ 30/100
mozilla/code-review#3737 · コメント 3 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 3/5 1〜2日 初心者へのやさしさ 55/100
mozilla/code-review#3710 ·
メンテナーはふだん 1 日以内に返信
mozilla/code-review の issue をすべて見る
似ている issue
-
Validate non-divisible effective batch sizes in the Unsloth LoRA backend対応中かも @debesh-biswas が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
-
`rx.Model` event arguments emit `PydanticDeprecatedSince20` once per payload key対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
reflex-dev/reflex#7634 · コメント 1 件 ·
メンテナーはふだん 2 日以内に返信
-
documentation
難易度 1/5 1〜3時間 初心者へのやさしさ 82/100
topoteretes/cognee#6239 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
ranaroussi/yfinance#3009 ·
メンテナーはふだん 1 日以内に返信
-
Test discovery aborts on GPU builds without cuStateVec: check_cuStateVec expects a failed result but set_config() now raises RuntimeError対応中かも @dmeliksetian が今日担当しました。 オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
Qiskit/qiskit-aer#2495 ·
メンテナーはふだん 1 日以内に返信