perf(pr-pipeline): end the demotion alert's re-run wait as soon as main recovers
メンテナーはふだん 1 日以内に返信
関連するプルリクエストがすでにマージされています。
- #9082 @kyle-sexton による — マージ済み
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 76/100
調査の方向性
.github/workflows/post-merge-demotion-alert.ymlを読み、待機ループ後にある既存の復旧チェックをたどってください。待機ループのテストを更新し、main上により新しい成功したpr-require-checksの実行がある場合に最初のポーリングで終了すること、また出力recovered=trueと通知が現在のループ後の処理経路に一致することを確認してください。
索引モデルが issue の本文から書いたものです。
説明
Refs: #8897, #8898
Problem
post-merge-demotion-alert.yml waits up to 40 one-minute polls for a re-run of a bot-dispatched red run (polls=40), and only after that loop checks whether a newer pr-require-checks run on main succeeded. pr-merge does not re-run these dispatched reds (no red reached attempt 2), so the wait always times out.
All seven dispatched reds between 2026-10-21 20:37Z and 2026-10-22 03:40Z ran the full 40 minutes, then filed nothing because main had already recovered 8 to 30 minutes after the red. For example, alert run 39688650626 ran 22:44:24 to 23:24:43, while main passed at run 39689621793 before 23:00. That is about 280 runner-minutes for no filing. Details: #8898.
Fix
Inside the wait loop, also run the recovery check for a newer successful pr-require-checks run on main after the red run. When one exists, exit with the same "main already recovered" notice and recovered=true output as today. Filing behavior does not change.
Done when a test of the wait loop shows it exits on the first poll where a newer success exists, and the output and notice match the current post-loop path.
- 主要言語
- Shell
- スター
- 22
- フォーク
- 2
- 平均マージ
- 6時間 58分
- マージ済み PR（30日）
- 951
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
melodic-software/claude-code-plugins のほかの issue
-
priority: medium status: needs-decision work-class: scoped
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
melodic-software/claude-code-plugins#9339 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
pipeline-canary priority: low status: ready work-class: mechanical
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
melodic-software/claude-code-plugins#9263 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
good first issue priority: low
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
melodic-software/claude-code-plugins#9251 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
area: security needs-triage priority: low
難易度 2/5 1〜3時間 初心者へのやさしさ 79/100
melodic-software/claude-code-plugins#9211 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
priority: low
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
melodic-software/claude-code-plugins#9202 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
melodic-software/claude-code-plugins の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
MISP/misp-docker#478 ·
メンテナーはふだん 1 日以内に返信
-
roadmap-100
難易度 2/5 1〜3時間 初心者へのやさしさ 80/100
メンテナーはふだん 1 日以内に返信
-
数据界面的横向滚动条缺失オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
zilliztech/attu#1037 ·
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
community-scripts/ProxmoxVE#18073 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Uptime "Test Monitor" always fails with an empty HTTP 400 (uptime-checker API endpoint not configured)対応中かも @ayoubjemai が今日担当しました。 オープンWaiting for: Product Owner
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
getsentry/self-hosted#4582 ·
メンテナーはふだん 1 日以内に返信