Skip to content

Renovate の予備の workflow は置かない、と書き直す - #391

Merged
5ym merged 2 commits into
mainfrom
renovate-no-fallback
Oct 16, 2026
Merged

5ym merged 2 commits into
mainfrom
renovate-no-fallback

Conversation

@5ym

@5ym 5ym commented Oct 16, 2026

Copy link
Copy Markdown
Collaborator

5ym/repo-config#96 と doa/renovate で、もとの Renovate の workflow を消す。予備として手で回せるように残すと書いていた apps/renovate/ の説明を直す (コメントと README だけ。マニフェストの中身は変えない)。

クラスタが止まっているときは更新先のサーバも止まっているので、そのあいだに PR を出しても意味が無い。

🤖 Generated with Claude Code

https://claude.ai/code/session_01JbmWyUB9jcBcYCnNyiGBSV

@claude

claude Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Claude finished @5ym's task in 21s —— View job


レビュー結果

コメントと README だけの変更で、マニフェストの中身は変わっていません。不具合とセキュリティの問題はありません。テストも対象外です。指摘は記述の正確さについて 2 点です。

  • 差分を確認
  • 周辺ファイルとの整合を確認

指摘

  1. 「更新先のサーバもこのクラスタの上」は Forgejo にしか当てはまらない (要確認)

    • apps/renovate/README.md の「クラスタが落ちているとき」と apps/renovate/renovate.yaml の冒頭コメントに、この文があります。
    • README の冒頭によると、対象は GitHub の danything と 5ym、Forgejo (fj.doany.io) の doa です。GitHub は外部サービスなので、クラスタが止まっていても動いています。
    • 「止まっているあいだに PR を出しても意味が無い」は、Forgejo 側の理由としては正しいです。GitHub 側の理由としては通りません。
    • 実際に予備を置かない理由が別にあるのなら、その理由を書いたほうが誤解がありません。たとえば「GitHub 側は手で回せるが、CronJob と二重に走るのを避けるために置かない」のような理由です。
    • 「Forgejo は同じクラスタ上にあるので」と範囲を限定するだけでも足ります。
  2. 日付 (要確認)

    • README に「2026-10-17 に消した」とあります。
    • CronJob の開始は同じファイルで「2026-10-16〜」と書かれており、今日 (実行時点) も 2026-10-16 です。
    • 消した日が 10-17 だと、まだ起きていないことを過去形で書くことになります。実際は Renovate の workflow を消す(k8s の CronJob に移した) 5ym/repo-config#96 と doa/renovate の側で消した日を確認してください。
    • 日付が確かでなければ、日付を書かずに「消した」だけにしても意味は変わりません。

補足

  • README の「(どちらも消した)」と、renovate.yaml の「もとの 2 つの workflow は消した」は内容が一致しています。
  • 「予備は置かない」への書き換えは、旧記述にあった「CronJob と同時に走らせないこと」の注意書きが不要になったことと矛盾しません。

@claude

claude Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Claude finished @5ym's task in 13s —— View job


レビュー結果

コメントと README だけの変更で、マニフェストの中身は変わっていません。不具合とセキュリティの問題はありません。テストも対象外です。

  • 差分を確認 (2 コミット: e813b19、56ff445)
  • 前回の指摘が直っているか確認

前回の指摘

  1. 「更新先のサーバもクラスタの上」は Forgejo にしか当てはまらない: 解消済みです。56ff445 で「(Forgejo はそもそもクラスタの上)」と範囲を限定していて、GitHub 側に誤解を与えません。

  2. 日付 (要確認、未解消): apps/renovate/README.md に「2026-10-17 に消した」とありますが、今日は 2026-10-16 です。まだ来ていない日付を過去形で書いています。実際に Renovate の workflow を消す(k8s の CronJob に移した) 5ym/repo-config#96 と doa/renovate で消した日を確認して直してください。日付が確かでなければ、日付を書かずに「消した」だけにしても意味は変わりません。

補足

  • README の「(どちらも消した)」と、renovate.yaml の「もとの 2 つの workflow は消した」は一致しています。
  • 旧記述にあった「CronJob と同時に走らせないこと」の注意書きは、予備を置かなくなったので不要です。削除は妥当です。

@5ym
5ym merged commit 93afb40 into main Oct 16, 2026
1 check passed
@5ym
5ym deleted the renovate-no-fallback branch October 16, 2026 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant