Skip to content

docs(recovery): 戻したあとに、戻すのに使ったスナップショットを消す - #787

Merged
5ym merged 1 commit into
mainfrom
volsync-restore-cleanup
Nov 11, 2026
Merged

5ym merged 1 commit into
mainfrom
volsync-restore-cleanup

Conversation

@5ym

@5ym 5ym commented Nov 11, 2026

Copy link
Copy Markdown
Collaborator

VolSync の populator で戻したあと、<pvc>-dst が戻した中身をスナップショットで持ち続ける。LVM はスナップショットのある元のボリュームを消せないので、作業用の PV (volsync-<pvc>-dst-dest) が Released のまま残り、VolumeFailedDelete (with 1 active snapshots) を出し続けていた (2026-11-02 の入れ直しの 21 個)。

  • 1 個 (netbird-routing-peer-data) で試し、スナップショットを消すと PV が数分で消え、アプリの PVC と Pod に響かないことを確かめた
  • 手順書の「6. PV データを戻す」に、戻したあとに消すことを書き足す。マニフェストは変えない

🤖 Generated with Claude Code

https://claude.ai/code/session_01T8TqVdo6LUrWKdVQgbScVU

<pvc>-dst が持ち続けるスナップショットがあると、LVM が作業用のボリュームを消せず、
PV (volsync-<pvc>-dst-dest) が Released のまま VolumeFailedDelete を出し続ける。
2026-11-02 の入れ直しの 21 個が残っていた。

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

claude Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

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


レビュー結果

recovery/README.md に 5 行足すだけの変更を見ました。ブロックする問題はありません。

  • 不具合: 書いてある内容に矛盾はありません。
    • 消すのは全 PVC が Bound になってからです。
    • 消したあとで PVC を作り直すときは、先に <pvc>-dst を戻し直させます。<pvc>-dst は消えたスナップショットを指したままになるので、この順番が必要です。
    • 後半の手順は変更箇所のすぐ下にあり、記述は整合しています。
  • セキュリティ: 該当なしです。
  • 設計: 6 節の「戻す」流れの中にそのまま入っていて、一貫しています。マニフェストを変えないのも妥当です。
  • テスト: ドキュメントのみの変更です。PR 本文によると netbird-routing-peer-data の 1 個で実機確認済みです。

要確認(任意): 21 個あるので、消す手順の例があると楽かもしれません。

  • 例: kubectl get volumesnapshot -A | grep -- '-dst-dest-' で一覧を出し、kubectl delete volumesnapshot -n <ns> <name> で消す。
  • 全 namespace に -dst-dest- を含む別のスナップショットがないことは、私の側では確認できていません。

指摘は以上です。

@5ym
5ym merged commit 1942ae8 into main Nov 11, 2026
3 checks passed
@5ym
5ym deleted the volsync-restore-cleanup branch November 11, 2026 02: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