Skip to content

k8up: rybbit の Schedule が check の文書と混ざって消えていたのを直す - #186

Merged
5ym merged 1 commit into
mainfrom
rybbit-schedule
Sep 19, 2026
Merged

5ym merged 1 commit into
mainfrom
rybbit-schedule

Conversation

@5ym

@5ym 5ym commented Sep 19, 2026

Copy link
Copy Markdown
Collaborator

#182 で足した rybbit の Schedule の後ろに --- が無く、続く check の文書と 1 つの YAML 文書に混ざっていた(同じキーが 2 回出て後ろが勝ち、rybbit は無かったことになる)。クラスタに Schedule rybbit が無いのを見て気づいた。dry-run の出力に rybbit が出ていないのを見落としたのが原因。区切りを 1 行足しただけ。dry-run で schedule.k8up.io/rybbit createdcheck configured の両方が出ることを確認。

🤖 Generated with Claude Code

https://claude.ai/code/session_01JgGYf5qbjDXYhtzcpLsGvv

kubectl の dry-run で rybbit が出ていないのを見落としていた(check だけ出ていた)。

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JgGYf5qbjDXYhtzcpLsGvv
@claude

claude Bot commented Sep 19, 2026

Copy link
Copy Markdown

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


レビュー完了

  • コンテキスト収集 (diff 確認)
  • 変更内容の検証 (YAML 文書区切りの妥当性)
  • 周辺の Schedule 定義との整合性確認
  • レビューコメント投稿

変更内容: rybbit の Schedule の末尾に --- が抜けていたため、後続の check 用コメント文書と 1 つの YAML 文書にマージされ、check の内容(spec.check のみ)が rybbitspec(backup/prune)を上書きしてしまっていた不具合の修正。

検証:

  • 他の 12 個の Schedule 定義(mattermostforgejo)はすべて末尾に --- があり、今回追加された行 (apps/k8up/schedules.yaml:354) でこのパターンに揃っている。
  • 修正後、rybbit は独立した YAML 文書となり、check(namespace: k8up)とはキーの競合が起きない。
  • PR 本文の通り dry-run で schedule.k8up.io/rybbit createdcheck configured の両方が出ることを確認済みとのこと。

1 行の区切り追加のみで、意図通りの修正になっています。問題無しです。

@5ym
5ym merged commit 2805277 into main Sep 19, 2026
1 check passed
@5ym
5ym deleted the rybbit-schedule branch September 19, 2026 14:16
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