probation_start is set while the criterion is still failing
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 53/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
調査の方向性
Start with state_machine._probation_start and the outlived_grace branch in transition(), which the issue identifies as stamping probation_start; trace how confirmed status changes from fail to pass. Check the existing backfill approach for rows stamped mid-failure. Done means probation starts only on recovery, the read-side compensations can be removed, and existing affected rows are corrected.
索引モデルが issue の本文から書いたものです。
説明
probation_start is stamped as soon as a failure is confirmed, then re-stamped to "tomorrow" on every failing night after that. So a set probation_start does not mean the criterion is serving probation — it may still be failing.
Every reader has to compensate with the same extra check:
| Reader | Compensation |
|---|---|
is_serving_probation (API) |
confirmed_status == 'pass' |
feed_search.sql |
confirmed_status = 'pass' |
roll_up_on_probation |
cancels on any confirmed failure |
state_machine.phase() |
returns ON_PROBATION for both, leaves it to the caller |
Recent fixes have all been on the read side: #1840, #1841, #1842, #1843.
Fix
Stamp probation_start only on the recovery — the run where confirmed_status goes fail → pass. last_confirmed_failure_at already records that a failure happened, so the "stamp tomorrow on every failing night" trick is no longer needed to land the start on the repair day.
Then probation_start is not None means serving probation, and the four compensations above come out.
state_machine._probation_start, plus theoutlived_gracebranch intransition(), which stamps too- backfill for rows currently stamped mid-failure
- 主要言語
- Python
- スター
- 34
- フォーク
- 7
- 平均マージ
- 2日 4時間
- マージ済み PR（30日）
- 8
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
MobilityData/mobility-feed-api のほかの issue
-
fix(notifications): dispatcher ignores a subscription's feeds and sends every feed.url_updated eventオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
MobilityData/mobility-feed-api#1875 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
MobilityData/mobility-feed-api#1846 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
MobilityData/mobility-feed-api#1755 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
MobilityData/mobility-feed-api#1647 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
MobilityData/mobility-feed-api#836 ·
メンテナーはふだん 1 日以内に返信
MobilityData/mobility-feed-api の issue をすべて見る
似ている issue
-
[oblt-aw][autodoc] Update workflow docs for gh-aw v0.92.5 and detector permission scope対応中かも @github-actions が今日担当しました。 オープン
難易度 1/5 1〜3時間 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
roadmap-100
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
comp/tools P2 tool/browser tool/file type/bug
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
NousResearch/hermes-agent#143418 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
python-hyper/uritemplate#164 ·