Skip to content

docs: ストレージ方針のレビュー指摘(pv.sh restore-all と明記、thin プールの溢れ) - #469

Merged
5ym merged 1 commit into
mainfrom
docs/storage-plan-fix
Oct 20, 2026
Merged

5ym merged 1 commit into
mainfrom
docs/storage-plan-fix

Conversation

@5ym

@5ym 5ym commented Oct 20, 2026

Copy link
Copy Markdown
Collaborator

#468 のレビュー指摘の反映(マージに間に合わなかった分)。

  • ROADMAP: 「k8up の restore-all」→「recovery/pv.sh restore-all(k8up の restic から)」
  • decisions.md: Talos の LVM 設定が入った版の書き方を一次情報(v1.14 のドキュメント、support for LVM volumes siderolabs/talos#12309)に合わせた。確かめていないことに「thin プールが溢れたときの挙動と見張り」を足した

🤖 Generated with Claude Code

https://claude.ai/code/session_01NN4TmSXHrMP8dkTbtkjfz1

@claude

claude Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

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


レビュー結果

問題なしです。ドキュメントのみの小さな修正でした。

  • ROADMAP.md: recovery/pv.sh restore-all は実在し、k8up の restic から戻す作りです(recovery/pv.sh:239、recovery/README.md:167)。書き換え後の記述はこれと合っています。
  • docs/decisions.md: Talos の LVM 設定の書き方を「v1.14 のドキュメントにある」と言い切らない形に直しています。「確かめていないこと」に thin プール溢れを足したのも妥当です。

要確認(任意): siderolabs/talos#12309 が 2026-06 に閉じられた、という点は私の環境からは確かめていません。一次情報に合わせた箇所なので、マージ前に日付だけ見直してください。

@5ym
5ym merged commit 79e620b into main Oct 20, 2026
1 check passed
@5ym
5ym deleted the docs/storage-plan-fix branch October 20, 2026 05:34
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