From e965f1d18e260e326c8b2dad76c7c3fe686e9d8f Mon Sep 17 00:00:00 2001 From: "takemi.ohama" Date: Thu, 24 Sep 2026 12:02:43 +0900 Subject: [PATCH 01/14] =?UTF-8?q?feat(PLAN68):=20=E3=82=B9=E3=83=8A?= =?UTF-8?q?=E3=83=83=E3=83=97=E3=82=B7=E3=83=A7=E3=83=83=E3=83=88=E3=81=AE?= =?UTF-8?q?=E4=B8=96=E4=BB=A3=E3=82=92=E3=82=A2=E3=82=AB=E3=82=A6=E3=83=B3?= =?UTF-8?q?=E3=83=88=E3=82=B0=E3=83=AB=E3=83=BC=E3=83=97=E3=81=94=E3=81=A8?= =?UTF-8?q?=E3=81=AE=E7=B3=BB=E5=88=97=E3=81=A7=E6=8C=81=E3=81=A4?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - devbase up の自動スナップショットは、起動したグループの系列の最新の世代へ差分を積む。 最小間隔も系列ごとに判定し、ログに系列のグループ名と新しい世代にする理由を出す - rotate は系列ごとに keep 世代を残し、全体の上限 (既定 keep × 3) を超えた分を系列を またいで古い順に消す。各系列の最新の世代は消さない。devbase snapshot rotate に --max-total を足し、--keep はグループごとの数にする - _safe_snap_dir はシンボリックリンクの世代を拒み、包含判定をパスの要素の単位にする。 rotate は消す前に検証し、拒否されたエントリは一覧からだけ外す - 利用者向け文書と CHANGELOG を更新し、実装計画にタスク分解を追記 Co-Authored-By: Claude Opus 5.5 (1M context) --- CHANGELOG.md | 25 ++ docs/user/cli-reference/02-project.md | 4 +- docs/user/cli-reference/05-snapshot.md | 14 +- docs/user/container-operations.md | 4 +- docs/user/snapshot-guide.md | 112 ++++-- docs/user/troubleshooting.md | 4 +- issues/PLAN68_snapshot-series.md | 23 +- lib/devbase/cli.py | 6 +- lib/devbase/commands/container.py | 21 +- lib/devbase/commands/snapshot.py | 8 +- lib/devbase/snapshot/manager.py | 268 ++++++++++--- lib/devbase/tui/actions_snapshot.py | 8 +- tests/cli/tui/test_actions_snapshot.py | 11 +- tests/snapshot/test_auto_snapshot_series.py | 101 +++++ tests/snapshot/test_manager_series.py | 420 ++++++++++++++++++++ 15 files changed, 917 insertions(+), 112 deletions(-) create mode 100644 tests/snapshot/test_auto_snapshot_series.py create mode 100644 tests/snapshot/test_manager_series.py diff --git a/CHANGELOG.md b/CHANGELOG.md index 375a35f1..51b12818 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -12,6 +12,31 @@ 止まります。`lfm` / `snapshot` は base を継がないため入りません。 **反映には `devbase build base --no-cache` と、使っている派生イメージの建て直しが要ります。** +### Changed +- **スナップショットの世代を、アカウントグループ（対象ボリュームの組）ごとの系列で持つように + しました（PLAN68 / #248）。** グループの違うプロジェクトを行き来しても、`devbase up` は + 起動したグループの系列の最新の世代へ差分を積み、フルバックアップを取り直しません。 + 新しい世代を作るのは、その系列に世代が無いときと、差分が上限（10）に達したときだけです。 + 自動スナップショットの最小間隔（`DEVBASE_SNAPSHOT_MIN_INTERVAL_MINUTES`）も系列ごとに判定します。 + 起動時の出力には扱った系列のグループ名が出て、「対象ボリュームの構成が変わったため」の行は + 出なくなりました。 +- **ローテーションは、グループごとに 3 世代を残し、全体で 9 世代を上限にします。** 全体の上限を + 超えると系列をまたいで古い世代から消しますが、**各系列の最新の世代は自動では消えません** + （次の差分の積み先のため）。使わなくなったグループの世代が不要なら `devbase snapshot delete` + で消してください。 +- **`devbase snapshot rotate --keep N` の N は「全体で残す数」から「グループごとに残す数」に + なりました。** 全体の上限は新しい `--max-total M`（省略時は `N × 3`）で指定します。同じ `--keep` の + 値で残る世代の数は減りませんが、どの世代が残るかは変わることがあります。どちらの指定も + その 1 回の実行だけに効き、`devbase up` / `down` の自動ローテーションは既定の数で動きます。 + +### Fixed +- **ローテーションが、`backups/` の外を指す名前の世代を消さず、シンボリックリンクの世代で + 止まらなくなりました（PLAN68 / #248）。** `snapshot.yml` のそうしたエントリは、ディレクトリも + リンク先も消さずに一覧から外し、警告を出します。 +- **`devbase snapshot create` / `restore` / `copy` / `delete` が、シンボリックリンクの世代を + エラーで止めるようになりました。** これまで `delete` はリンク先（`backups/` の外や別の世代）の + 中身を消していました。 + ## [3.7.0] - 2026-09-23 ### Added diff --git a/docs/user/cli-reference/02-project.md b/docs/user/cli-reference/02-project.md index 9b1f44d5..8d250bbb 100644 --- a/docs/user/cli-reference/02-project.md +++ b/docs/user/cli-reference/02-project.md @@ -94,8 +94,8 @@ devbase up [name] [--context NAME] - 自動スナップショットは作らない（控えたいボリュームがリモートにあるため） - `DOCKER_HOST` が設定されていれば警告して外す（docker は `DOCKER_HOST` を `DOCKER_CONTEXT` より優先するため） -- 起動時にスナップショットを自動作成（新世代 or 差分追加） - - 直近のスナップショット取得から既定 60 分以内のときはスキップします +- 起動時にスナップショットを自動作成（起動するアカウントグループの系列の最新の世代へ差分追加。系列に世代が無いか差分が上限に達していれば新世代） + - 同じアカウントグループ（系列）の直近のスナップショット取得から既定 60 分以内のときはスキップします。別のグループのスナップショットの時刻は見ません - 間隔は `DEVBASE_SNAPSHOT_MIN_INTERVAL_MINUTES` 環境変数で上書き可能（既定 60、`0` で無効化＝毎回取得、不正値は警告して既定値） - `project.yml` の `scale` に基づいてコンテナ数を決定（既定: 2） - `project.yml` の `repos` を clone プランへ正規化してコンテナへ渡す（コンテナ内で `/work` 配下へ clone される） diff --git a/docs/user/cli-reference/05-snapshot.md b/docs/user/cli-reference/05-snapshot.md index d55ad16c..ce7c6765 100644 --- a/docs/user/cli-reference/05-snapshot.md +++ b/docs/user/cli-reference/05-snapshot.md @@ -66,12 +66,20 @@ devbase snapshot delete ## `devbase snapshot rotate` -古い世代のスナップショットを削除します。 +古い世代のスナップショットを削除します。世代はアカウントグループ（対象ボリュームの組）ごとの +系列に分かれ、系列ごとに `--keep` 世代を残したうえで、全体の上限を超えた分を系列をまたいで +古い順に削除します。各系列の最新の世代は削除しません。 ``` -devbase snapshot rotate [--keep N] +devbase snapshot rotate [--keep N] [--max-total M] ``` | オプション | 説明 | |-----------|------| -| `--keep N` | 保持する世代数（デフォルト: `3`） | +| `--keep N` | アカウントグループ（系列）ごとに保持する世代数（デフォルト: `3`） | +| `--max-total M` | すべてのグループを合わせて保持する世代数の上限（デフォルト: `N × 3`）。各グループの最新の世代は上限を超えても残す | + +`N` と `M` は 1 以上です。0 以下はエラーで終了コード 1 になります。 + +**どちらの指定も、手動で実行したその 1 回だけに効きます。** 値は保存されず、`devbase up` / +`devbase down` の自動ローテーションは既定（グループごとに 3・全体で 9）で動きます。 diff --git a/docs/user/container-operations.md b/docs/user/container-operations.md index 579d6a35..6ec3d315 100644 --- a/docs/user/container-operations.md +++ b/docs/user/container-operations.md @@ -60,8 +60,8 @@ devbase down | タイミング | 動作 | 条件 | |-----------|------|------| -| `devbase up` | フルバックアップ or 差分追加 | 前回のフルバックアップからの経過日数で判定 | -| `devbase down` | 古い世代のローテーション | `DEFAULT_MAX_GENERATIONS` を超えた世代を削除 | +| `devbase up` | フルバックアップ or 差分追加 | 起動するアカウントグループの系列の最新の世代へ差分を追加。系列に世代が無いか差分が上限に達していればフルバックアップで新しい世代 | +| `devbase down` | 古い世代のローテーション | グループ（系列）ごとに `DEFAULT_MAX_GENERATIONS`（3）を超えた世代と、全体の上限（9）を超えた古い世代を削除。各系列の最新の世代は残す | 詳細は [スナップショットガイド](snapshot-guide.md) を参照してください。 diff --git a/docs/user/snapshot-guide.md b/docs/user/snapshot-guide.md index f0f8cd7b..be4944e5 100644 --- a/docs/user/snapshot-guide.md +++ b/docs/user/snapshot-guide.md @@ -57,31 +57,52 @@ graph LR | パラメータ | デフォルト値 | 説明 | |-----------|------------|------| | `DEFAULT_MAX_INCREMENTALS` | `10` | 1 世代あたりの最大差分バックアップ数 | -| `DEFAULT_MAX_GENERATIONS` | `3` | 保持する最大世代数 | +| `DEFAULT_MAX_GENERATIONS` | `3` | アカウントグループ（系列）ごとに保持する世代数 | +| 全体の上限 | `DEFAULT_MAX_GENERATIONS × 3`（`9`） | すべてのグループを合わせて保持する世代数の上限 | -デフォルト設定では、差分バックアップが 10 回溜まるとフルバックアップが新たに作成され、最大 3 世代が保持されます。 +デフォルト設定では、差分バックアップが 10 回溜まるとフルバックアップが新たに作成され、グループごとに最大 3 世代、全体で最大 9 世代が保持されます。 + +### 系列 + +世代は、控える**ボリュームの組**（共通ボリューム `devbase_home_ubuntu` とグループの +ボリューム `devbase_home_`）ごとの**系列**に分かれます。アカウントグループ 1 つにつき +系列が 1 つできます。アカウントグループ分離より前に作られた世代（共通ボリュームだけを控えたもの）は、 +まとめて 1 つの系列になります。 + +- 差分は、起動したグループの系列の**最新の世代**へ積みます。別のグループのプロジェクトを + 起動した後に戻ってきても、フルバックアップを取り直しません +- 保持の数は系列ごとに数えます。グループを切り替えても、他のグループの世代が保持の枠から押し出されません +- 全体の上限を超えると、系列をまたいで最も古い世代から削除します +- **各系列の最新の世代は自動では削除されません。** 次の差分の積み先になるためです。使わなくなった + グループや、アカウントグループ分離より前の系列の最新の世代も残ります。不要になったら + `devbase snapshot delete <名前>` で削除してください。系列の数が全体の上限を超えているときは、 + 上限を超えたまま残し、警告を出します ### 世代の概念 ```mermaid graph TD - subgraph 世代 1（最古） - A1[full.tar.zst] - A2[incr-001.tar.zst] - A3[incr-002.tar.zst] + subgraph 系列 default + subgraph 世代 1（最古） + A1[full.tar.zst] + A2[incr-001.tar.zst] + A3[incr-002.tar.zst] + end + subgraph 世代 2（系列の最新） + B1[full.tar.zst] + B2[incr-001.tar.zst] + end end - subgraph 世代 2 - B1[full.tar.zst] - B2[incr-001.tar.zst] - end - subgraph 世代 3（最新） - C1[full.tar.zst] + subgraph 系列 with + subgraph 世代 3（系列の最新） + C1[full.tar.zst] + end end ``` - 1 つの世代は 1 つのフルバックアップと 0 個以上の差分バックアップで構成される -- 差分バックアップが `DEFAULT_MAX_INCREMENTALS` 回に達すると新しい世代が開始される -- `DEFAULT_MAX_GENERATIONS` を超えた古い世代は自動的に削除される +- 系列の最新の世代の差分バックアップが `DEFAULT_MAX_INCREMENTALS` 回に達すると、その系列に新しい世代が開始される +- 系列ごとに `DEFAULT_MAX_GENERATIONS` を超えた古い世代と、全体の上限を超えた古い世代は自動的に削除される ## 自動実行 @@ -91,22 +112,37 @@ graph TD ```mermaid flowchart TD - A[devbase up 実行] --> B{現世代の差分バックアップが
DEFAULT_MAX_INCREMENTALS 回以上?} - B -->|はい| C[新世代のフルバックアップを作成] - B -->|いいえ| D[現世代に差分バックアップを追加] - C --> E[コンテナを起動] - D --> E + A[devbase up 実行] --> S{起動するグループの系列に
世代がある?} + S -->|いいえ| C[新世代のフルバックアップを作成] + S -->|はい| B{系列の最新の世代の差分バックアップが
DEFAULT_MAX_INCREMENTALS 回以上?} + B -->|はい| C + B -->|いいえ| D[系列の最新の世代に差分バックアップを追加] + C --> R[ローテーション] + D --> R + R --> E[コンテナを起動] +``` + +起動時の出力には、扱った系列のグループ名が出ます。新しい世代を作るときは、その理由 +（系列に世代が無い / 差分が上限に達した）を 1 行出します。 + +```text +[0/6] スナップショットを差分更新中: 20260920-212546 (グループ with) ``` +直近のスナップショットから `DEVBASE_SNAPSHOT_MIN_INTERVAL_MINUTES`（既定 60 分）以内なら +スナップショットを飛ばします。この間隔も系列ごとに判定するため、default のプロジェクトを +起動した直後に with のプロジェクトを起動しても、with の系列は控えます。 + ### `devbase down` 時の動作 ```mermaid flowchart TD A[devbase down 実行] --> B[コンテナを停止・削除] - B --> C{世代数 > DEFAULT_MAX_GENERATIONS?} - C -->|はい| D[最古の世代を削除] - D --> C - C -->|いいえ| E[完了] + B --> C[系列ごとに DEFAULT_MAX_GENERATIONS を
超えた古い世代を削除] + C --> T{全体の世代数 > 全体の上限?} + T -->|はい| D[系列の最新ではない世代のうち
最も古いものを削除] + D --> T + T -->|いいえ| E[完了] ``` ## バックアップデータ構造 @@ -190,9 +226,11 @@ before-upgrade 2026-02-21 14:00:00 1 2.1GB devbase_ho ### 対象ボリュームが変わったとき アカウントグループを切り替えたり、分離前の環境から更新したりすると、対象ボリュームの構成が -変わります。このとき devbase は**新しい世代を作ります**。旧世代の差分状態ファイル -（`snapshot.snar`）は別のレイアウトを記録しているため、そこへ差分を積むと全ファイルが -移動したものとして扱われ、差分が壊れるからです。世代を分けることで旧世代はそのまま復元できます。 +変わります。devbase は、切り替えた先のグループの系列（[系列](#系列)）の最新の世代へ差分を +積みます。**新しい世代を作るのは、その系列にまだ世代が無いときだけです。** 構成の違う世代へは +差分を積みません。差分状態ファイル（`snapshot.snar`）は別のレイアウトを記録しているため、 +そこへ差分を積むと全ファイルが移動したものとして扱われ、差分が壊れるからです。 +系列を分けることで、どの世代もそのまま復元できます。 構成の違う世代を明示的に指定して差分を作ろうとした場合は、理由を示して中断します。 @@ -332,11 +370,20 @@ devbase snapshot delete 20260218-080000 # デフォルトの保持数で実行 devbase snapshot rotate -# 保持する世代数を指定 +# グループごとに保持する世代数を指定 devbase snapshot rotate --keep 5 + +# 全体の上限も指定 +devbase snapshot rotate --keep 5 --max-total 10 ``` -`--keep N` で指定した世代数より古い世代を削除します。名前付きスナップショット（`--name` で作成したもの）はローテーション対象外です。 +`--keep N` はアカウントグループ（系列）ごとに残す世代数です。各系列で古い世代から削除し、 +残りの合計が `--max-total M`（省略時は `N × 3`）を超えていれば、系列をまたいで古い世代から +削除します。各系列の最新の世代は削除しません。名前付きスナップショット（`--name` で作成したもの）や +`copy` で作った世代も、対象ボリュームの組でいずれかの系列に入り、同じ規則で削除の対象になります。 + +**`--keep` と `--max-total` は、その 1 回の実行だけに効きます。** 値は保存されず、 +`devbase up` / `devbase down` の自動ローテーションは既定（グループごとに 3・全体で 9）で動きます。 ## 運用のベストプラクティス @@ -357,9 +404,8 @@ devbase snapshot rotate --keep 5 du -sh projects//backups/ ``` -5. **ローテーションの保持数はプロジェクトに合わせて調整する** +5. **長期保持したい世代は `backups/` の外へ複製する** - ```bash - # 長期保持が必要な場合 - devbase snapshot rotate --keep 7 - ``` + `devbase snapshot rotate --keep` の指定はその 1 回だけに効き、次の `devbase up` / `devbase down` + の自動ローテーションで既定の数まで削除されます。`snapshot copy` で作った世代もローテーションの + 対象です。長く残したい世代は、世代のディレクトリを `backups/` の外へ複製してください。 diff --git a/docs/user/troubleshooting.md b/docs/user/troubleshooting.md index 85c48837..d3260d15 100644 --- a/docs/user/troubleshooting.md +++ b/docs/user/troubleshooting.md @@ -358,9 +358,11 @@ devbase snapshot list # バックアップディレクトリのサイズ確認 du -sh ${DEVBASE_ROOT}/backups/ -# 不要な世代を削除（2世代のみ保持） +# 不要な世代を削除（グループごとに 2 世代、全体で 6 世代まで保持。この 1 回だけに効く） devbase snapshot rotate --keep 2 +# 各グループの最新の世代はローテーションで消えない。使わないグループの世代は個別に削除する + # 個別のスナップショットを削除 devbase snapshot delete ``` diff --git a/issues/PLAN68_snapshot-series.md b/issues/PLAN68_snapshot-series.md index 9fdf3237..af3eb977 100644 --- a/issues/PLAN68_snapshot-series.md +++ b/issues/PLAN68_snapshot-series.md @@ -202,7 +202,28 @@ default（差分 0、3.9 GB）で、合計は 42 GB である。 ## 実装計画 設計は [PLAN68_snapshot-series-design.md](PLAN68_snapshot-series-design.md)。 -**タスクへの分解は実装の持ち場で `/ndf:implementation-plan` が行う。** +タスクは受け入れ条件の単位で分け、どれも「失敗するテスト → 通す最小実装 → 整理」で進める。 + +### タスク分解 + +| # | タスク | 対象ファイル | 変更内容 | 満たす受け入れ条件 | +| --- | --- | --- | --- | --- | +| 1 | 世代の場所の検証 | `snapshot/manager.py`、`tests/snapshot/test_manager_series.py` | `_safe_snap_dir` にシンボリックリンクの拒否と `Path.is_relative_to` の包含判定を入れる（決定 7） | 26（`_safe_snap_dir`）・27・28 | +| 2 | 系列の解決と積み先 | 同上 | `series_key` / `series_label` / `_entry_volumes` / `series_latest` / `auto_snapshot_target` を足し、`should_start_new_generation` を包むだけにする | 1〜5 | +| 3 | 系列ごとの最小間隔 | 同上 | `last_snapshot_time(volumes=None)` | 6・7 | +| 4 | 系列ごとの保持と全体の上限 | 同上 | `rotate(keep, max_total)` を系列ごと + 全体の上限 + 各系列の最新を残す形へ。消す前に `_safe_snap_dir` で検証し、拒否されたエントリは一覧からだけ外す | 8〜14・17・22・25・26 | +| 5 | `_auto_snapshot` の流れ | `commands/container.py`、`tests/snapshot/test_auto_snapshot_series.py` | 最小間隔を系列で判定し、`auto_snapshot_target` の結果で `create` を呼ぶ。ログに系列の名前を入れる | 1・2・6・7・16 | +| 6 | CLI と TUI | `cli.py`、`commands/snapshot.py`、`tui/actions_snapshot.py`、`tests/cli/tui/test_actions_snapshot.py`、`tests/snapshot/test_manager_series.py` | `--max-total` の追加、`--keep` の help、振り分けの `getattr(args, 'max_total', None)`、TUI の問いの文言 | 14（CLI）・15・18 | +| 7 | 文書と CHANGELOG | 文書 4 本、`CHANGELOG.md` | 設計の「文書の変更」の表のとおり | 19〜21 | +| 8 | 全体の確認 | — | `uv run --locked pytest tests/ -q`、`ruff check --select=E9,F63,F7,F82 lib` | 23・24 | + +### リスクと対処 + +| リスク | 対処 | +| --- | --- | +| `manager.py`（818 行）に規則が集まる | 系列の解決は小さな関数に分け、`rotate` の削除候補の計算を副作用の無い補助に切り出す。構造は保ち、タスクごとにテストを通す | +| 既存テスト（`test_auto_snapshot.py` の `last_snapshot_time`、`test_manager_volumes.py`）の退行 | 引数の既定値で現行の振る舞いを保ち、タスクごとに `tests/snapshot/` を回す | +| 実際の tar による、別グループを挟んだ差分の復元が未確認 | この持ち場ではコンテナを起動しない（並行する #253 の検査と重ねない）。検査の持ち場へ回し、Pull Request 本文に書く | ### 修正対象 diff --git a/lib/devbase/cli.py b/lib/devbase/cli.py index e6c0c34f..288290e0 100644 --- a/lib/devbase/cli.py +++ b/lib/devbase/cli.py @@ -713,7 +713,11 @@ def _add_snapshot_parser(subparsers): s_delete.add_argument('name', help='Snapshot name') s_rotate = ss_sub.add_parser('rotate', help='Rotate old snapshots') - s_rotate.add_argument('--keep', type=int, default=3, help='Generations to keep') + s_rotate.add_argument('--keep', type=int, default=3, + help='Generations to keep per account group') + s_rotate.add_argument('--max-total', type=int, default=None, metavar='M', + help='Upper limit of generations across all groups ' + '(default: 3 x --keep)') def _add_shortcuts(subparsers): diff --git a/lib/devbase/commands/container.py b/lib/devbase/commands/container.py index f5901a71..d7bdc650 100644 --- a/lib/devbase/commands/container.py +++ b/lib/devbase/commands/container.py @@ -804,7 +804,7 @@ def _snapshot_min_interval_minutes() -> int: def _auto_snapshot(remote: bool = False) -> None: - """デプロイ前の自動スナップショット (差分世代数ベース世代管理)。 + """デプロイ前の自動スナップショット (系列ごとの差分世代数ベース世代管理。PLAN68)。 失敗してもデプロイは続行する (warning のみ)。DEVBASE_ROOT 未設定なら no-op。 リモート扱い (PLAN52 決定 12) では作らない。控えたいボリュームがリモートにあり、 @@ -824,8 +824,11 @@ def _auto_snapshot(remote: bool = False) -> None: from devbase.snapshot.manager import SnapshotManager mgr = SnapshotManager(Path(devbase_root)) + # 最小間隔・積み先はどちらも系列 (起動するグループのボリュームの組) の + # 単位で判定する (PLAN68 決定 2・4) + label = mgr.series_label(mgr.volumes) min_interval = _snapshot_min_interval_minutes() - last = mgr.last_snapshot_time() + last = mgr.last_snapshot_time(mgr.volumes) if min_interval > 0 and last is not None: # 経過時間が負 (last が未来) の場合はスキップしない。システム時計の # ズレや他環境からのリストアで last が未来になると delta が負になり、 @@ -834,17 +837,17 @@ def _auto_snapshot(remote: bool = False) -> None: delta = datetime.now(timezone.utc) - last if timedelta(0) <= delta < timedelta(minutes=min_interval): logger.info( - "[0/6] 直近のスナップショット (%s) から%d分以内のためスキップします", - last.astimezone().strftime('%Y-%m-%d %H:%M:%S'), min_interval, + "[0/6] %s の直近のスナップショット (%s) から%d分以内のためスキップします", + label, last.astimezone().strftime('%Y-%m-%d %H:%M:%S'), min_interval, ) return - if mgr.should_start_new_generation(): - logger.info("[0/6] 新しいスナップショット世代を作成中...") + target = mgr.auto_snapshot_target() + if target is None: + logger.info("[0/6] 新しいスナップショット世代を作成中 (%s)...", label) mgr.create() else: - latest = mgr.list()[-1]['name'] - logger.info("[0/6] スナップショットを差分更新中: %s", latest) - mgr.create(name=latest, full=False) + logger.info("[0/6] スナップショットを差分更新中: %s (%s)", target, label) + mgr.create(name=target, full=False) mgr.rotate() except Exception as e: logger.warning("スナップショットの自動作成に失敗しましたがデプロイは続行します: %s", e) diff --git a/lib/devbase/commands/snapshot.py b/lib/devbase/commands/snapshot.py index d24f208c..b0ce330e 100644 --- a/lib/devbase/commands/snapshot.py +++ b/lib/devbase/commands/snapshot.py @@ -36,7 +36,9 @@ def cmd_snapshot(devbase_root: Path, args) -> int: name=getattr(args, 'name', ''), new_name=getattr(args, 'new_name', '')), 'delete': lambda: _snapshot_delete(mgr, name=getattr(args, 'name', '')), - 'rotate': lambda: _snapshot_rotate(mgr, keep=getattr(args, 'keep', 3)), + # TUI の dispatch_group は keep だけを持つ引数を渡すため getattr で受ける + 'rotate': lambda: _snapshot_rotate(mgr, keep=getattr(args, 'keep', 3), + max_total=getattr(args, 'max_total', None)), } handler = handlers.get(subcmd) @@ -101,8 +103,8 @@ def _snapshot_delete(mgr, name='') -> int: return 0 -def _snapshot_rotate(mgr, keep=3) -> int: - deleted = mgr.rotate(keep=keep) +def _snapshot_rotate(mgr, keep=3, max_total=None) -> int: + deleted = mgr.rotate(keep=keep, max_total=max_total) if deleted == 0: logger.info("ローテーション不要です") return 0 diff --git a/lib/devbase/snapshot/manager.py b/lib/devbase/snapshot/manager.py index ada680e6..4e5b4746 100644 --- a/lib/devbase/snapshot/manager.py +++ b/lib/devbase/snapshot/manager.py @@ -158,10 +158,26 @@ def _validate_name(name: str) -> None: ) def _safe_snap_dir(self, name: str) -> Path: - """名前からスナップショットディレクトリを安全に解決する""" + """名前からスナップショットディレクトリを安全に解決する。 + + 次の 3 つを ``SnapshotError`` で止める (PLAN68 決定 7)。 + + - 名前が不正 (``../outside`` など) + - 世代がシンボリックリンク。devbase はリンクの世代を作らず、リンク先が + ``backups/`` の外でも中の別の世代でも、消すと実体を失う + - 解決後のパスが ``backups/`` の中に無い。文字列の前方一致ではなく + パスの要素の単位で比べる (兄弟の ``backups-outside/`` を通さないため) + + ``backups/`` 自体をリンクにした構成は、解決後の ``backups/`` と比べるため使える。 + """ self._validate_name(name) - snap_dir = (self.backups_dir / name).resolve() - if not str(snap_dir).startswith(str(self.backups_dir.resolve())): + raw = self.backups_dir / name + if raw.is_symlink(): + raise SnapshotError( + f"スナップショット '{name}' はシンボリックリンクのため扱えません: " + f"{raw} -> {raw.readlink()}") + snap_dir = raw.resolve() + if not snap_dir.is_relative_to(self.backups_dir.resolve()): raise SnapshotError(f"無効なスナップショットパス: '{name}'") return snap_dir @@ -208,9 +224,12 @@ def list(self) -> list[dict]: snap['size_bytes'] = 0 return snapshots - def last_snapshot_time(self) -> Optional[datetime]: + def last_snapshot_time(self, volumes: Optional[dict] = None) -> Optional[datetime]: """直近のスナップショット取得 (フル/差分) 日時を返す。 + ``volumes`` を渡すと、その組の系列に属する世代 (``snapshot.yml`` のエントリ) + のディレクトリだけを見る (PLAN68 決定 4)。省けば全ディレクトリを見る。 + 各スナップショットディレクトリ内のアーカイブ実体 (``full.tar.zst`` / ``incr-*.tar.zst``) の mtime のうち最新のものを採用する。 差分更新は既存ディレクトリ名を再利用するため (ディレクトリ名の日付は世代 @@ -226,9 +245,18 @@ def last_snapshot_time(self) -> Optional[datetime]: """ if not self.backups_dir.exists(): return None + if volumes is None: + snap_dirs = list(self.backups_dir.iterdir()) + else: + key = self.series_key(volumes) + snap_dirs = [ + self.backups_dir / s['name'] for s in self._entries() + if self.series_key(self._entry_volumes(s)) == key + and is_single_segment_name(s['name']) + ] latest: Optional[float] = None - for snap_dir in self.backups_dir.iterdir(): - if not snap_dir.is_dir(): + for snap_dir in snap_dirs: + if snap_dir.is_symlink() or not snap_dir.is_dir(): continue for f in snap_dir.iterdir(): if not f.is_file(): @@ -448,69 +476,209 @@ def delete(self, name: str) -> None: self._save_metadata(meta) logger.info("削除完了: %s", name) - def rotate(self, keep: int = DEFAULT_MAX_GENERATIONS) -> int: - """古い世代を削除する。 + def rotate(self, keep: int = DEFAULT_MAX_GENERATIONS, + max_total: Optional[int] = None) -> int: + """古い世代を削除する (PLAN68 決定 1・6・7)。 + + 系列 (対象ボリュームの組) ごとに ``keep`` 世代を残し、残りの総数が + ``max_total`` (省けば ``keep × 3``) を超えたら、系列をまたいで最も古い + 世代から消す。**各系列の最新の世代は消さない** (次の差分の積み先のため)。 + 消す前に :meth:`_safe_snap_dir` で名前を検証し、拒否されたエントリは + ディレクトリを消さずに一覧からだけ外す。 + + Args: + keep: 系列ごとに残す世代の数 + max_total: 全体で残す世代の上限。省けば ``keep × 3`` Returns: 削除された世代数 + + Raises: + SnapshotError: ``keep`` か ``max_total`` が 1 未満の場合 (何も消さない) """ - meta = self._load_metadata() - snapshots = meta.get('snapshots', []) + if max_total is None: + max_total = keep * 3 + if keep < 1: + raise SnapshotError(f"--keep は 1 以上である必要があります: {keep}") + if max_total < 1: + raise SnapshotError(f"--max-total は 1 以上である必要があります: {max_total}") - if len(snapshots) <= keep: + meta = self._load_metadata() + snapshots = meta.get('snapshots', []) or [] + plan = self._rotation_plan(snapshots, keep, max_total) + if not plan: return 0 - # 古い順にソート（created_atベース） - snapshots.sort(key=lambda s: s.get('created_at', '')) - to_delete = snapshots[:-keep] - - deleted = 0 - for snap in to_delete: - snap_dir = self.backups_dir / snap['name'] + removed_ids = set() + deleted_ids = set() + for index, reason in plan: + snap = snapshots[index] + name = snap.get('name', '') + removed_ids.add(index) + try: + snap_dir = self._safe_snap_dir(name) + except SnapshotError as e: + logger.warning( + "snapshot.yml の世代 '%s' は場所が不正なため、ディレクトリを消さずに" + "一覧からだけ外します: %s", name, e) + continue if snap_dir.exists(): shutil.rmtree(snap_dir) - deleted += 1 + deleted_ids.add(index) + if reason is not None: + logger.info( + "ローテーション: 全体の上限 %d 世代を超えたため、%s の %s を削除しました", + max_total, self.series_label(self._entry_volumes(snap)), name) + + per_series: dict = {} + for index, reason in plan: + if reason is None and index in deleted_ids: + label = self.series_label(self._entry_volumes(snapshots[index])) + per_series[label] = per_series.get(label, 0) + 1 + for label, count in per_series.items(): + logger.info( + "ローテーション: %s の %d 世代を削除しました（グループごとに %d 世代保持）", + label, count, keep) - meta['snapshots'] = snapshots[-keep:] + remaining = [s for i, s in enumerate(snapshots) if i not in removed_ids] + order = {id(s): i for i, s in enumerate(snapshots)} + remaining.sort(key=lambda s: (s.get('created_at', '') or '', order[id(s)])) + meta['snapshots'] = remaining meta['max_generations'] = keep self._save_metadata(meta) + return len(deleted_ids) - if deleted: - logger.info("ローテーション: %d 世代を削除しました（%d 世代保持）", deleted, keep) - return deleted + def _rotation_plan(self, snapshots: list, keep: int, max_total: int) -> list: + """ローテーションで消すエントリを決める (副作用なし)。 - def should_start_new_generation( - self, max_incrementals: int = DEFAULT_MAX_INCREMENTALS, - ) -> bool: - """最新世代の差分バックアップ数が上限に達しているか判定する。 + Returns: + ``(snapshots の添字, 理由)`` の並び。理由は系列ごとの保持なら ``None``、 + 全体の上限なら ``'total'``。 + """ + groups: dict = {} + for index, snap in enumerate(snapshots): + key = self.series_key(self._entry_volumes(snap)) + groups.setdefault(key, []).append(index) + + def age(index: int) -> tuple: + # created_at が同じなら snapshot.yml で前にあるものを古いとみなす + return (snapshots[index].get('created_at', '') or '', index) + + plan: list = [] + kept: dict = {} + for key, indexes in groups.items(): + indexes.sort(key=age) + excess = max(0, len(indexes) - keep) + plan.extend((i, None) for i in indexes[:excess]) + kept[key] = indexes[excess:] + + total = sum(len(v) for v in kept.values()) + while total > max_total: + candidates = [v for v in kept.values() if len(v) >= 2] + if not candidates: + logger.warning( + "全体の上限 %d 世代を超えていますが、各グループの最新の世代は" + "消さないため %d 世代を残します", max_total, total) + break + oldest = min(candidates, key=lambda v: age(v[0])) + plan.append((oldest.pop(0), 'total')) + total -= 1 + return plan - Args: - max_incrementals: 1世代あたりの最大差分バックアップ数 + # ------------------------------------------------------------------ + # 系列 (PLAN68) + # ------------------------------------------------------------------ + + @staticmethod + def _entry_volumes(entry: dict) -> dict: + """``snapshot.yml`` のエントリの対象ボリュームの組。 + + ``volumes`` が無い・空・dict でないエントリは PLAN39 より前の旧レイアウトで、 + 共通ボリューム 1 本の組とみなす。値は検証しない (系列のキーにするだけで、 + マウントには使わないため)。 + """ + volumes = entry.get('volumes') if isinstance(entry, dict) else None + if isinstance(volumes, dict) and volumes: + return dict(volumes) + return {'': HOME_UBUNTU_VOLUME} + + @staticmethod + def series_key(volumes: dict) -> tuple: + """系列の識別子。対象ボリュームの組を並べ替えたタプル。""" + return tuple(sorted((str(k), str(v)) for k, v in volumes.items())) + + @staticmethod + def series_label(volumes: dict) -> str: + """系列の表示名 (ログ用)。例: ``グループ default``。""" + group = volumes.get(GROUP_MOUNT) + if isinstance(group, str) and group: + if group.startswith(SHARED_VOLUME_PREFIX): + group = group[len(SHARED_VOLUME_PREFIX):] + return f"グループ {group}" + return "旧レイアウト（共通ボリュームのみ）" + + def _entries(self) -> list: + return [s for s in (self._load_metadata().get('snapshots') or []) + if isinstance(s, dict) and 'name' in s] + + def series_latest(self, volumes: Optional[dict] = None) -> Optional[dict]: + """系列の最新の世代のエントリ (``created_at`` が最大)。無ければ ``None``。 + + ``created_at`` が同じなら ``snapshot.yml`` で後ろのものを新しいとみなす。 + """ + key = self.series_key(self.volumes if volumes is None else volumes) + latest = None + latest_age = None + for index, snap in enumerate(self._entries()): + if self.series_key(self._entry_volumes(snap)) != key: + continue + age = (snap.get('created_at', '') or '', index) + if latest_age is None or age > latest_age: + latest, latest_age = snap, age + return latest + + def auto_snapshot_target( + self, max_incrementals: int = DEFAULT_MAX_INCREMENTALS, + ) -> Optional[str]: + """自動スナップショットの積み先を返す (PLAN68 決定 2)。 Returns: - True: 新世代を作成すべき（スナップショットなし or 差分数が上限以上） - False: 既存世代に差分を追加すべき + 系列の最新の世代の名前。新しい世代を作るべきなら ``None`` + (その理由を INFO で 1 行出す)。 """ - meta = self._load_metadata() - snapshots = meta.get('snapshots', []) - if not snapshots: - return True - latest = snapshots[-1] - - # 対象ボリュームの構成が変わったら新世代にする (PLAN39 の移行やグループ - # 切替)。旧世代の snar は別のレイアウトを記録しているので、そこへ差分を - # 積むと全ファイルが移動したものとして扱われ差分が壊れる。世代を分ければ - # 旧世代はそのまま復元できる。 - snap_dir = self.backups_dir / latest.get('name', '') - if snap_dir.is_dir() and self.snapshot_volumes(snap_dir) != self.volumes: + label = self.series_label(self.volumes) + latest = self.series_latest() + if latest is None: + logger.info("%s の世代がまだ無いため、新しい世代を作成します", label) + return None + + name = latest['name'] + snap_dir = self.backups_dir / name + if (is_single_segment_name(name) and not snap_dir.is_symlink() + and snap_dir.is_dir()): + recorded = self.snapshot_volumes(snap_dir) + if recorded != self.volumes: + logger.info( + "世代 %s の meta.yml の対象ボリューム (%s) が %s と一致しないため、" + "新しい世代を作成します", name, ', '.join(recorded.values()), label) + return None + + if latest.get('incremental_count', 0) >= max_incrementals: logger.info( - "対象ボリュームの構成が変わったため新しい世代を作成します " - "(旧: %s / 新: %s)", - ', '.join(self.snapshot_volumes(snap_dir).values()), - ', '.join(self.volumes.values())) - return True + "世代 %s（%s）の差分が上限 (%d) に達したため、新しい世代を作成します", + name, label, max_incrementals) + return None + return name - return latest.get('incremental_count', 0) >= max_incrementals + def should_start_new_generation( + self, max_incrementals: int = DEFAULT_MAX_INCREMENTALS, + ) -> bool: + """新しい世代を作るべきか (``auto_snapshot_target(...) is None``)。 + + 系列 (対象ボリュームの組) の最新の世代に世代が無い、または差分数が上限以上 + なら True。グループを切り替えただけでは True にならない (PLAN68)。 + """ + return self.auto_snapshot_target(max_incrementals) is None # ------------------------------------------------------------------ # Private helpers @@ -647,7 +815,7 @@ def _create_incremental(self, name: str, snap_dir: Path) -> None: """差分バックアップを作成""" recorded = self.snapshot_volumes(snap_dir) if recorded != self.volumes: - # 通常はここへ来ない (should_start_new_generation が新世代へ倒す)。 + # 通常はここへ来ない (auto_snapshot_target が新世代へ倒す)。 # 明示的に古い世代を指定されたときだけ到達する。黙って壊れた差分を # 積むより、理由を出して止める方がよい。 raise SnapshotError( diff --git a/lib/devbase/tui/actions_snapshot.py b/lib/devbase/tui/actions_snapshot.py index b863b807..1b3434bb 100644 --- a/lib/devbase/tui/actions_snapshot.py +++ b/lib/devbase/tui/actions_snapshot.py @@ -9,7 +9,7 @@ - restore: ``name``, ``point`` (None=全差分適用 / manager は 1 以上のみ受理) - copy: ``name``, ``new_name`` - delete: ``name`` -- rotate: ``keep`` (3) +- rotate: ``keep`` (3。グループごと。``max_total`` は渡さず既定の ``keep × 3``) 破壊的な restore / delete は実行前に確認する (plan 3.4)。restore は ``cmd_snapshot`` 側にも TTY 時の input() 確認が残るが、TUI の規約として @@ -143,8 +143,10 @@ def _op_delete(devbase_root: Path): def _op_rotate(devbase_root: Path): - # keep=0 は manager 実装上 no-op (空スライス) のため 1 以上を要求する。 - keep = flow.need(menu.integer("保持する世代数 (--keep)", default=3, min_value=1)) + # keep=0 は manager が SnapshotError で拒むため 1 以上を要求する。 + # 全体の上限 (--max-total) は問わず、既定 (keep × 3) で動かす (PLAN68)。 + keep = flow.need(menu.integer("グループごとに保持する世代数 (--keep)", + default=3, min_value=1)) return dispatch_group(cmd_snapshot, devbase_root, "rotate", keep=keep) diff --git a/tests/cli/tui/test_actions_snapshot.py b/tests/cli/tui/test_actions_snapshot.py index 0804a162..61c43240 100644 --- a/tests/cli/tui/test_actions_snapshot.py +++ b/tests/cli/tui/test_actions_snapshot.py @@ -44,7 +44,7 @@ def _capture_dispatch(monkeypatch): def _spy(devbase_root, args): captured["devbase_root"] = devbase_root captured["subcommand"] = args.subcommand - for k in ("name", "full", "point", "new_name", "keep"): + for k in ("name", "full", "point", "new_name", "keep", "max_total"): if hasattr(args, k): captured[k] = getattr(args, k) return 0 @@ -314,14 +314,17 @@ def test_run_operation_rotate_collects_keep(monkeypatch, tmp_path): seen = {} def fake_integer(message, *, default=None, min_value=None, max_value=None): - seen.update(default=default, min_value=min_value) + seen.update(message=message, default=default, min_value=min_value) return 5 monkeypatch.setattr(menu, "integer", fake_integer) assert actions_snapshot._run_operation(tmp_path, "rotate") == 0 assert captured["subcommand"] == "rotate" and captured["keep"] == 5 - # CLI 既定 (--keep 3) と同じ既定値を提示し、no-op な 0 以下は弾く。 - assert seen == {"default": 3, "min_value": 1} + # 全体の上限は問わない (既定の keep × 3 で動く。PLAN68)。 + assert "max_total" not in captured + # CLI 既定 (--keep 3) と同じ既定値を提示し、0 以下は弾く。問いはグループごと。 + assert seen == {"message": "グループごとに保持する世代数 (--keep)", + "default": 3, "min_value": 1} @pytest.mark.parametrize("int_ret", ["BACK", None]) diff --git a/tests/snapshot/test_auto_snapshot_series.py b/tests/snapshot/test_auto_snapshot_series.py new file mode 100644 index 00000000..9b46441a --- /dev/null +++ b/tests/snapshot/test_auto_snapshot_series.py @@ -0,0 +1,101 @@ +"""``devbase up`` の自動スナップショットを系列で扱う (PLAN68) + +``DEVBASE_ROOT`` を ``tmp_path`` に向け、``SnapshotManager._run_docker_tar`` を +差し替える。実データの ``backups/`` と Docker に触らない。 +""" + +from __future__ import annotations + +import logging +import os +import re +import time + +import pytest +import yaml + +from devbase.commands import container +from devbase.snapshot.manager import SnapshotManager + +from .test_manager_series import names, write_state + + +@pytest.fixture +def root(tmp_path, monkeypatch): + monkeypatch.setenv("DEVBASE_ROOT", str(tmp_path)) + monkeypatch.setenv("DEVBASE_SNAPSHOT_MIN_INTERVAL_MINUTES", "0") + monkeypatch.delenv("DEVBASE_ACCOUNT_GROUP", raising=False) + def fake(self, snap_dir, mode, command, volumes=None): + if mode == "backup": + archive = re.search(r"/backup/(full\.tar\.zst|incr-\d+\.tar\.zst)", command) + (snap_dir / archive.group(1)).write_text("archive") + (snap_dir / "snapshot.snar").write_text("snar") + + monkeypatch.setattr(SnapshotManager, "_run_docker_tar", fake) + return tmp_path + + +def _age(path, seconds): + t = time.time() - seconds + os.utime(path, (t, t)) + + +def test_returning_group_appends_to_its_generation(root, caplog): + """1・16: default → with → default で、default の世代へ incr を積む。""" + backups = write_state(root, [("A", "default", 0), ("B", "with", 0)]) + + with caplog.at_level(logging.INFO, logger="devbase"): + container._auto_snapshot() + + assert (backups / "A" / "incr-001.tar.zst").exists() + assert not (backups / "B" / "incr-001.tar.zst").exists() + assert names(root) == ["A", "B"] + messages = [r.getMessage() for r in caplog.records] + assert not any("構成が変わった" in m for m in messages) + assert any("差分更新中: A (グループ default)" in m for m in messages) + + +def test_group_without_generation_creates_new(root, monkeypatch, caplog): + """2・16: with の世代が無ければ full の世代を作り、理由を出す。""" + write_state(root, [("A", "default", 0)]) + monkeypatch.setenv("DEVBASE_ACCOUNT_GROUP", "with") + + with caplog.at_level(logging.INFO, logger="devbase"): + container._auto_snapshot() + + entries = yaml.safe_load((root / "backups" / "snapshot.yml").read_text())["snapshots"] + assert len(entries) == 2 + new = entries[-1] + assert new["volumes"]["group"] == "devbase_home_with" + assert (root / "backups" / new["name"] / "full.tar.zst").exists() + messages = [r.getMessage() for r in caplog.records] + assert any("グループ with の世代がまだ無いため" in m for m in messages) + assert any("新しいスナップショット世代を作成中 (グループ with)" in m for m in messages) + + +def test_min_interval_is_per_series(root, monkeypatch, caplog): + """6: default は 10 分前なら飛ばし、with は 2 時間前なので積む。""" + backups = write_state(root, [("W", "with", 0), ("D", "default", 0)]) + _age(backups / "D" / "full.tar.zst", 600) + _age(backups / "W" / "full.tar.zst", 7200) + monkeypatch.setenv("DEVBASE_SNAPSHOT_MIN_INTERVAL_MINUTES", "60") + + monkeypatch.setenv("DEVBASE_ACCOUNT_GROUP", "with") + container._auto_snapshot() + assert (backups / "W" / "incr-001.tar.zst").exists() + + monkeypatch.setenv("DEVBASE_ACCOUNT_GROUP", "default") + with caplog.at_level(logging.INFO, logger="devbase"): + container._auto_snapshot() + assert not (backups / "D" / "incr-001.tar.zst").exists() + assert any("グループ default の直近のスナップショット" in r.getMessage() + and "スキップします" in r.getMessage() for r in caplog.records) + + +def test_zero_interval_never_skips(root): + """7: 間隔 0 なら 10 分前の系列でも積む。""" + backups = write_state(root, [("D", "default", 0)]) + _age(backups / "D" / "full.tar.zst", 600) + + container._auto_snapshot() + assert (backups / "D" / "incr-001.tar.zst").exists() diff --git a/tests/snapshot/test_manager_series.py b/tests/snapshot/test_manager_series.py new file mode 100644 index 00000000..01d6f19b --- /dev/null +++ b/tests/snapshot/test_manager_series.py @@ -0,0 +1,420 @@ +"""スナップショットの世代を、ボリュームの組ごとの系列で持つ (PLAN68) + +系列の解決・差分の積み先・系列ごとの最小間隔・系列ごとの保持と全体の上限、 +および世代の場所の検証 (決定 7) を固定する。Docker は起動しない +(``_run_docker_tar`` を差し替える。``test_manager_volumes.py`` と同じ流儀)。 +""" + +from __future__ import annotations + +import logging +import os +import time +import types +from pathlib import Path + +import pytest +import yaml + +from devbase.commands.snapshot import cmd_snapshot +from devbase.errors import SnapshotError +from devbase.snapshot.manager import SnapshotManager + + +@pytest.fixture(autouse=True) +def _clean_group_env(monkeypatch): + monkeypatch.delenv("DEVBASE_ACCOUNT_GROUP", raising=False) + + +class RecordingManager(SnapshotManager): + """``docker run`` を実行せず、渡された引数だけを記録する。""" + + def __init__(self, *args, **kwargs): + super().__init__(*args, **kwargs) + self.calls: list[dict] = [] + + def _run_docker_tar(self, snap_dir, mode, command, volumes=None): + self.calls.append({"mode": mode, "command": command}) + if mode == "backup": + (snap_dir / "full.tar.zst").write_text("archive") + (snap_dir / "snapshot.snar").write_text("snar") + + +def vols(group: str) -> dict: + return {"ai": "devbase_home_ubuntu", "group": f"devbase_home_{group}"} + + +def write_state(root: Path, entries: list) -> Path: + """``snapshot.yml`` と各世代のディレクトリを直接書く。 + + entries は ``(名前, グループ | None, 差分数)`` の並び。``None`` は旧レイアウト + (``volumes`` を持たないエントリ)。並びの順に ``created_at`` を古い方から振る。 + """ + backups = root / "backups" + backups.mkdir(parents=True, exist_ok=True) + snaps = [] + for i, (name, group, incr) in enumerate(entries): + created = f"2026-09-01T00:{i:02d}:00" + snap_dir = backups / name + if not snap_dir.exists() and not snap_dir.is_symlink() and "/" not in name: + snap_dir.mkdir() + (snap_dir / "full.tar.zst").write_text("archive") + (snap_dir / "snapshot.snar").write_text("snar") + meta = {"name": name, "type": "full", "files": ["full.tar.zst"], + "incremental_count": incr} + if group is None: + meta["volume"] = "devbase_home_ubuntu" + else: + meta["volumes"] = vols(group) + (snap_dir / "meta.yml").write_text(yaml.safe_dump(meta)) + entry = {"name": name, "created_at": created, "updated_at": created, + "incremental_count": incr} + if group is not None: + entry["volumes"] = vols(group) + snaps.append(entry) + (backups / "snapshot.yml").write_text(yaml.safe_dump( + {"max_generations": 3, "snapshots": snaps})) + return backups + + +def names(root: Path) -> list: + data = yaml.safe_load((root / "backups" / "snapshot.yml").read_text()) + return [s["name"] for s in data["snapshots"]] + + +# --------------------------------------------------------------------------- +# 系列の解決 +# --------------------------------------------------------------------------- + +def test_series_label_names_the_group(): + assert SnapshotManager.series_label(vols("with")) == "グループ with" + assert SnapshotManager.series_label({"": "devbase_home_ubuntu"}) == \ + "旧レイアウト（共通ボリュームのみ）" + + +def test_series_key_ignores_order(): + a = {"ai": "devbase_home_ubuntu", "group": "devbase_home_x"} + b = {"group": "devbase_home_x", "ai": "devbase_home_ubuntu"} + assert SnapshotManager.series_key(a) == SnapshotManager.series_key(b) + + +# --------------------------------------------------------------------------- +# 差分の積み先 (受け入れ条件 1〜5) +# --------------------------------------------------------------------------- + +def test_returning_group_appends_to_its_latest_generation(tmp_path): + """1: default → with → default で、default の世代へ差分を積む。""" + RecordingManager(tmp_path, group="default").create(name="A") + RecordingManager(tmp_path, group="with").create(name="B") + + mgr = RecordingManager(tmp_path, group="default") + assert mgr.auto_snapshot_target() == "A" + assert mgr.should_start_new_generation() is False + + +def test_group_without_generation_starts_a_new_one(tmp_path, caplog): + """2: 系列に世代が無ければ新しい世代にする。理由を出す。""" + write_state(tmp_path, [("A", "default", 0)]) + mgr = RecordingManager(tmp_path, group="with") + + with caplog.at_level(logging.INFO, logger="devbase"): + assert mgr.auto_snapshot_target() is None + assert any("グループ with の世代がまだ無いため" in r.getMessage() + for r in caplog.records) + + +def test_incremental_limit_is_counted_per_series(tmp_path, caplog): + """3: 差分の上限は系列ごと。他の系列の差分数を見ない。""" + write_state(tmp_path, [("W", "with", 0), ("D", "default", 10)]) + + with caplog.at_level(logging.INFO, logger="devbase"): + assert RecordingManager(tmp_path, group="default").auto_snapshot_target() is None + assert any("上限 (10) に達した" in r.getMessage() and "グループ default" in r.getMessage() + for r in caplog.records) + assert RecordingManager(tmp_path, group="with").auto_snapshot_target() == "W" + + +def test_legacy_generation_is_not_appended_to(tmp_path): + """4: 旧レイアウトの世代だけなら新しい世代にする。""" + write_state(tmp_path, [("old", None, 0)]) + assert RecordingManager(tmp_path).auto_snapshot_target() is None + + +def test_meta_mismatch_starts_a_new_generation(tmp_path, caplog): + """snapshot.yml と meta.yml の組が食い違えば積まない (判定の順 2)。""" + backups = write_state(tmp_path, [("D", "default", 0)]) + meta = yaml.safe_load((backups / "D" / "meta.yml").read_text()) + meta["volumes"] = vols("kkg") + (backups / "D" / "meta.yml").write_text(yaml.safe_dump(meta)) + + with caplog.at_level(logging.INFO, logger="devbase"): + assert RecordingManager(tmp_path, group="default").auto_snapshot_target() is None + assert any("meta.yml の対象ボリューム" in r.getMessage() for r in caplog.records) + + +def test_series_latest_uses_created_at(tmp_path): + write_state(tmp_path, [("D1", "default", 0), ("W", "with", 0), ("D2", "default", 0)]) + mgr = RecordingManager(tmp_path, group="default") + assert mgr.series_latest()["name"] == "D2" + assert mgr.series_latest(vols("with"))["name"] == "W" + assert mgr.series_latest(vols("kkg")) is None + + +# --------------------------------------------------------------------------- +# 系列ごとの最小間隔 (受け入れ条件 6) +# --------------------------------------------------------------------------- + +def test_last_snapshot_time_per_series(tmp_path): + backups = write_state(tmp_path, [("W", "with", 0), ("D", "default", 0)]) + now = time.time() + os.utime(backups / "D" / "full.tar.zst", (now - 600, now - 600)) + os.utime(backups / "W" / "full.tar.zst", (now - 7200, now - 7200)) + + mgr = SnapshotManager(tmp_path) + d = mgr.last_snapshot_time(vols("default")).timestamp() + w = mgr.last_snapshot_time(vols("with")).timestamp() + assert abs(d - (now - 600)) < 2 + assert abs(w - (now - 7200)) < 2 + assert mgr.last_snapshot_time(vols("kkg")) is None + # 省けば全体 (現行どおり) + assert abs(mgr.last_snapshot_time().timestamp() - (now - 600)) < 2 + + +# --------------------------------------------------------------------------- +# 保持 (受け入れ条件 8〜14・17・22) +# --------------------------------------------------------------------------- + +def test_rotate_keeps_per_series(tmp_path, caplog): + """8・17: default 4・with 1 で default の最古だけを消す。""" + backups = write_state(tmp_path, [ + ("D1", "default", 0), ("D2", "default", 0), ("W1", "with", 0), + ("D3", "default", 0), ("D4", "default", 0)]) + + with caplog.at_level(logging.INFO, logger="devbase"): + assert SnapshotManager(tmp_path).rotate() == 1 + assert names(tmp_path) == ["D2", "W1", "D3", "D4"] + assert not (backups / "D1").exists() + assert (backups / "W1").exists() + assert any("グループ default の 1 世代を削除しました" in r.getMessage() + for r in caplog.records) + + +def test_alternating_groups_keep_three_each(tmp_path): + """9: 交互に 4 つずつ作っても各 3 世代が残る。""" + entries = [] + for i in range(4): + entries += [(f"D{i}", "default", 0), (f"W{i}", "with", 0)] + write_state(tmp_path, entries) + + assert SnapshotManager(tmp_path).rotate() == 2 + assert names(tmp_path) == ["D1", "W1", "D2", "W2", "D3", "W3"] + + +def test_total_limit_removes_oldest_across_series(tmp_path, caplog): + """10・17: 4 系列 × 3 世代 (交互) で A1・B1・C1 を消す。""" + entries = [] + for i in (1, 2, 3): + for g in "abcd": + entries.append((f"{g.upper()}{i}", f"g{g}", 0)) + write_state(tmp_path, entries) + + with caplog.at_level(logging.INFO, logger="devbase"): + assert SnapshotManager(tmp_path).rotate() == 3 + remaining = names(tmp_path) + assert len(remaining) == 9 + assert not {"A1", "B1", "C1"} & set(remaining) + assert any("全体の上限 9 世代を超えたため、グループ ga の A1 を削除しました" + in r.getMessage() for r in caplog.records) + + +def test_total_limit_keeps_latest_of_each_series(tmp_path): + """11: A の 3 世代が最古でも、A の最新は残す。""" + entries = [] + for g in "abcd": + for i in (1, 2, 3): + entries.append((f"{g.upper()}{i}", f"g{g}", 0)) + write_state(tmp_path, entries) + + assert SnapshotManager(tmp_path).rotate() == 3 + remaining = names(tmp_path) + assert "A3" in remaining + assert not {"A1", "A2", "B1"} & set(remaining) + + +def test_total_limit_cannot_remove_series_latest(tmp_path, caplog): + """12: 10 系列 × 1 世代は消さず、警告を 1 行出す。""" + write_state(tmp_path, [(f"S{i}", f"g{i}x", 0) for i in range(10)]) + before = (tmp_path / "backups" / "snapshot.yml").read_bytes() + + with caplog.at_level(logging.INFO, logger="devbase"): + assert SnapshotManager(tmp_path).rotate(keep=3, max_total=9) == 0 + warnings = [r for r in caplog.records if r.levelno == logging.WARNING] + assert len(warnings) == 1 + assert "全体の上限 9 世代" in warnings[0].getMessage() + assert "10 世代を残します" in warnings[0].getMessage() + assert (tmp_path / "backups" / "snapshot.yml").read_bytes() == before + + +def test_legacy_series_is_counted_separately(tmp_path): + """13: 旧レイアウト 3 と default 3 は、どちらも消さない。""" + write_state(tmp_path, [("L1", None, 0), ("L2", None, 0), ("L3", None, 0), + ("D1", "default", 0), ("D2", "default", 0), ("D3", "default", 0)]) + assert SnapshotManager(tmp_path).rotate() == 0 + + +@pytest.mark.parametrize("kwargs", [{"keep": 0}, {"max_total": 0}, {"keep": -1}]) +def test_rotate_rejects_non_positive(tmp_path, kwargs): + """14: 0 以下は SnapshotError。何も消さない。""" + backups = write_state(tmp_path, [(f"D{i}", "default", 0) for i in range(5)]) + with pytest.raises(SnapshotError): + SnapshotManager(tmp_path).rotate(**kwargs) + assert all((backups / f"D{i}").exists() for i in range(5)) + + +@pytest.mark.parametrize("attrs", [{"keep": 0}, {"keep": 3, "max_total": 0}]) +def test_cli_rotate_rejects_non_positive(tmp_path, attrs): + write_state(tmp_path, [("D1", "default", 0)]) + ns = types.SimpleNamespace(subcommand="rotate", **attrs) + assert cmd_snapshot(tmp_path, ns) == 1 + + +def test_cli_rotate_passes_max_total(tmp_path): + """15: --max-total が manager へ渡る (既定の 6 なら 0 件になる)。""" + write_state(tmp_path, [("D1", "default", 0), ("W1", "with", 0), + ("D2", "default", 0), ("W2", "with", 0)]) + ns = types.SimpleNamespace(subcommand="rotate", keep=2, max_total=2) + assert cmd_snapshot(tmp_path, ns) == 0 + assert names(tmp_path) == ["D2", "W2"] + + +def test_cli_rotate_without_max_total_uses_keep_times_three(tmp_path, caplog): + """15・18: TUI と同じ keep だけの引数で動き、全体の上限は keep × 3。""" + write_state(tmp_path, [(f"S{i}", f"g{i}x", 0) for i in range(4)]) + ns = types.SimpleNamespace(subcommand="rotate", keep=1) + with caplog.at_level(logging.INFO, logger="devbase"): + assert cmd_snapshot(tmp_path, ns) == 0 + assert any("全体の上限 3 世代" in r.getMessage() for r in caplog.records + if r.levelno == logging.WARNING) + + +def test_existing_state_is_left_untouched(tmp_path): + """22: この端末と同じ 3 エントリで何も消さず、snapshot.yml を書かない。""" + backups = tmp_path / "backups" + backups.mkdir() + real = { + "max_generations": 3, + "snapshots": [ + {"name": "20260915-231738", "created_at": "2026-09-15T23:17:38.1", + "updated_at": "2026-09-22T10:00:00", "incremental_count": 9, + "volumes": vols("default")}, + {"name": "20260920-212546", "created_at": "2026-09-20T21:25:46.1", + "updated_at": "2026-09-20T21:25:46.1", "incremental_count": 0, + "volumes": vols("with")}, + {"name": "20260923-081407", "created_at": "2026-09-23T08:14:07.1", + "updated_at": "2026-09-23T08:14:07.1", "incremental_count": 0, + "volumes": vols("default")}, + ], + } + (backups / "snapshot.yml").write_text(yaml.safe_dump(real)) + before = (backups / "snapshot.yml").read_bytes() + + assert SnapshotManager(tmp_path).rotate() == 0 + assert (backups / "snapshot.yml").read_bytes() == before + + +# --------------------------------------------------------------------------- +# 世代の場所の検証 (受け入れ条件 25〜28、決定 7) +# --------------------------------------------------------------------------- + +def test_rotate_does_not_remove_outside_backups(tmp_path, caplog): + """25: ``../outside`` のエントリは一覧から外すだけで、外のディレクトリは残す。""" + outside = tmp_path / "outside" + outside.mkdir() + (outside / "keep.txt").write_text("keep") + write_state(tmp_path, [("../outside", "default", 0), ("D1", "default", 0), + ("D2", "default", 0), ("D3", "default", 0)]) + + with caplog.at_level(logging.INFO, logger="devbase"): + SnapshotManager(tmp_path).rotate() + assert (outside / "keep.txt").read_text() == "keep" + assert names(tmp_path) == ["D1", "D2", "D3"] + warnings = [r for r in caplog.records if r.levelno == logging.WARNING] + assert len(warnings) == 1 and "'../outside'" in warnings[0].getMessage() + + +def _link_outside(tmp_path: Path) -> Path: + outside = tmp_path / "backups-outside" + outside.mkdir() + (outside / "keep.txt").write_text("keep") + (tmp_path / "backups").mkdir(exist_ok=True) + (tmp_path / "backups" / "old").symlink_to(outside, target_is_directory=True) + return outside + + +def test_rotate_does_not_follow_symlink_outside(tmp_path, caplog): + """26: 兄弟の backups-outside/ を指すリンクの世代は、リンク先を消さない。""" + outside = _link_outside(tmp_path) + write_state(tmp_path, [("old", "default", 0), ("D1", "default", 0), + ("D2", "default", 0), ("D3", "default", 0)]) + + with caplog.at_level(logging.INFO, logger="devbase"): + SnapshotManager(tmp_path).rotate() + assert (outside / "keep.txt").read_text() == "keep" + assert names(tmp_path) == ["D1", "D2", "D3"] + warnings = [r for r in caplog.records if r.levelno == logging.WARNING] + assert len(warnings) == 1 and "'old'" in warnings[0].getMessage() + with pytest.raises(SnapshotError): + SnapshotManager(tmp_path)._safe_snap_dir("old") + + +def test_rotate_does_not_follow_symlink_inside(tmp_path, caplog): + """26: backups/ の中の系列の最新を指すリンクでも、その中身を消さない。""" + backups = write_state(tmp_path, [("D1", "default", 0), ("D2", "default", 0), + ("new", "default", 0)]) + (backups / "old").symlink_to(backups / "new", target_is_directory=True) + data = yaml.safe_load((backups / "snapshot.yml").read_text()) + data["snapshots"].insert(0, {"name": "old", "created_at": "2026-08-01T00:00:00", + "incremental_count": 0, "volumes": vols("default")}) + (backups / "snapshot.yml").write_text(yaml.safe_dump(data)) + + with caplog.at_level(logging.INFO, logger="devbase"): + SnapshotManager(tmp_path).rotate() + assert (backups / "new" / "full.tar.zst").read_text() == "archive" + assert names(tmp_path) == ["D1", "D2", "new"] + assert len([r for r in caplog.records if r.levelno == logging.WARNING]) == 1 + + +def test_safe_snap_dir_uses_path_components(tmp_path): + """決定 7: 兄弟の backups-outside/ は文字列の前方一致でも通さない。""" + _link_outside(tmp_path) + mgr = SnapshotManager(tmp_path) + with pytest.raises(SnapshotError): + mgr._safe_snap_dir("old") + + +def test_cli_delete_refuses_symlink(tmp_path): + """27: delete はリンクの世代を止め、リンク先を消さない。""" + outside = _link_outside(tmp_path) + write_state(tmp_path, [("old", "default", 0)]) + ns = types.SimpleNamespace(subcommand="delete", name="old") + assert cmd_snapshot(tmp_path, ns) == 1 + assert (outside / "keep.txt").read_text() == "keep" + + +@pytest.mark.parametrize("op", ["restore", "copy", "create"]) +def test_other_operations_refuse_symlink(tmp_path, op): + """28: restore / copy / create もリンクの世代で止まり、何も書かない。""" + outside = _link_outside(tmp_path) + write_state(tmp_path, [("old", "default", 0)]) + mgr = RecordingManager(tmp_path) + + with pytest.raises(SnapshotError): + if op == "restore": + mgr.restore("old") + elif op == "copy": + mgr.copy("old", "new") + else: + mgr.create(name="old") + assert mgr.calls == [] + assert not (tmp_path / "backups" / "new").exists() + assert sorted(p.name for p in outside.iterdir()) == ["keep.txt"] From 74c1ad850e664399051474b1db940cd3030d839d Mon Sep 17 00:00:00 2001 From: "takemi.ohama" Date: Thu, 24 Sep 2026 12:16:13 +0900 Subject: [PATCH 02/14] test: characterize snapshot command and auto-snapshot branches Add restore confirmation, rotate argument parsing, auto-snapshot failure, and missing or unknown subcommand characterization tests for apply round 1. Item-Id: R1-001 Round: 1 Impl-Runtime: codex Impl-Model: default --- tests/cli/test_snapshot_rotate_args.py | 31 +++++++++++++++ tests/commands/test_snapshot.py | 42 +++++++++++++++++++++ tests/commands/test_snapshot_dispatch.py | 14 +++++++ tests/snapshot/test_auto_snapshot_series.py | 12 ++++++ 4 files changed, 99 insertions(+) create mode 100644 tests/cli/test_snapshot_rotate_args.py create mode 100644 tests/commands/test_snapshot.py create mode 100644 tests/commands/test_snapshot_dispatch.py diff --git a/tests/cli/test_snapshot_rotate_args.py b/tests/cli/test_snapshot_rotate_args.py new file mode 100644 index 00000000..c8253a73 --- /dev/null +++ b/tests/cli/test_snapshot_rotate_args.py @@ -0,0 +1,31 @@ +"""snapshot rotate の引数解析の現状固定テスト。""" + +from devbase.cli import _create_parser + + +def test_rotate_explicit_limits(): + args = _create_parser().parse_args( + ["snapshot", "rotate", "--keep", "2", "--max-total", "5"] + ) + + assert args.subcommand == "rotate" + assert args.keep == 2 + assert type(args.keep) is int + assert args.max_total == 5 + assert type(args.max_total) is int + + +def test_rotate_default_limits(): + args = _create_parser().parse_args(["snapshot", "rotate"]) + + assert args.subcommand == "rotate" + assert args.keep == 3 + assert args.max_total is None + + +def test_rotate_alias_max_total(): + args = _create_parser().parse_args(["ss", "rotate", "--max-total", "4"]) + + assert args.subcommand == "rotate" + assert args.max_total == 4 + assert type(args.max_total) is int diff --git a/tests/commands/test_snapshot.py b/tests/commands/test_snapshot.py new file mode 100644 index 00000000..d4171ad2 --- /dev/null +++ b/tests/commands/test_snapshot.py @@ -0,0 +1,42 @@ +"""復元の確認分岐を、疑似ボリュームの最終状態で現状固定する。""" + +from types import SimpleNamespace + +import pytest + +from devbase.commands import snapshot + + +@pytest.mark.parametrize( + "is_tty,answer,expected_content", + [ + (True, "", "original"), + (True, "n", "original"), + (True, "y", "saved at point 2"), + (True, "YES", "saved at point 2"), + (False, None, "saved at point 2"), + ], +) +def test_restore_confirmation(tmp_path, monkeypatch, is_tty, answer, expected_content): + volume = {"content": "original"} + saved = {("daily", 2): "saved at point 2"} + + class FakeSnapshotManager: + def __init__(self, devbase_root): + pass + + def restore(self, name, point=None): + volume["content"] = saved[name, point] + + def respond(prompt): + if not is_tty: + pytest.fail("Non-TTY restore must not prompt for input") + return answer + + monkeypatch.setattr(snapshot, "SnapshotManager", FakeSnapshotManager) + monkeypatch.setattr(snapshot.sys, "stdin", SimpleNamespace(isatty=lambda: is_tty)) + monkeypatch.setattr("builtins.input", respond) + args = SimpleNamespace(subcommand="restore", name="daily", point=2) + + assert snapshot.cmd_snapshot(tmp_path, args) == 0 + assert volume["content"] == expected_content diff --git a/tests/commands/test_snapshot_dispatch.py b/tests/commands/test_snapshot_dispatch.py new file mode 100644 index 00000000..e4059eb9 --- /dev/null +++ b/tests/commands/test_snapshot_dispatch.py @@ -0,0 +1,14 @@ +"""未指定・未知の snapshot サブコマンドの終了コードを現状固定する。""" + +from types import SimpleNamespace + +import pytest + +from devbase.commands.snapshot import cmd_snapshot + + +@pytest.mark.parametrize("subcommand", [None, "bogus"]) +def test_missing_or_unknown_subcommand_returns_error(tmp_path, subcommand): + args = SimpleNamespace(subcommand=subcommand) + + assert cmd_snapshot(tmp_path, args) == 1 diff --git a/tests/snapshot/test_auto_snapshot_series.py b/tests/snapshot/test_auto_snapshot_series.py index 9b46441a..066bb415 100644 --- a/tests/snapshot/test_auto_snapshot_series.py +++ b/tests/snapshot/test_auto_snapshot_series.py @@ -99,3 +99,15 @@ def test_zero_interval_never_skips(root): container._auto_snapshot() assert (backups / "D" / "incr-001.tar.zst").exists() + + +def test_invalid_group_warns_without_creating_metadata(root, monkeypatch, caplog): + """現状固定: ボリューム解決の失敗は警告に落とし、呼び出し元へ戻る。""" + monkeypatch.setenv("DEVBASE_ACCOUNT_GROUP", "invalid/group") + + with caplog.at_level(logging.WARNING, logger="devbase"): + container._auto_snapshot() + + warnings = [record for record in caplog.records if record.levelno == logging.WARNING] + assert len(warnings) == 1 + assert not (root / "backups" / "snapshot.yml").exists() From 63f73318e8b47a4d809c8b0e87689c97d63f3c15 Mon Sep 17 00:00:00 2001 From: "takemi.ohama" Date: Thu, 24 Sep 2026 12:21:22 +0900 Subject: [PATCH 03/14] =?UTF-8?q?Test:=20characterization=20=E2=80=94=20li?= =?UTF-8?q?b/devbase/commands/container.py#=5Fauto=5Fsnapshot?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 自動スナップショットの「新世代の作成 → 既定 rotate()」のつなぎ目を現状固定する。 default 3 世代 (最新の差分数 10) と with 1 世代から default で呼ぶと、default は 新世代が積まれて 4 世代 → 最古が落ちて 3 世代に戻り、with の 1 世代は残る。 消えた最古のディレクトリが無く、with の世代のディレクトリが残ることも確認する。 対象コードは変更しない。 Item-Id: R1-004 Round: 1 Impl-Runtime: kiro Impl-Model: default --- tests/snapshot/test_auto_snapshot_series.py | 36 +++++++++++++++++++++ 1 file changed, 36 insertions(+) diff --git a/tests/snapshot/test_auto_snapshot_series.py b/tests/snapshot/test_auto_snapshot_series.py index 066bb415..1f9c2c2e 100644 --- a/tests/snapshot/test_auto_snapshot_series.py +++ b/tests/snapshot/test_auto_snapshot_series.py @@ -101,6 +101,42 @@ def test_zero_interval_never_skips(root): assert (backups / "D" / "incr-001.tar.zst").exists() +def test_new_generation_rotates_only_its_own_series(root, monkeypatch): + """現状固定: 作成の後の rotate() で、新世代を積んだ系列の最古だけが消える。 + + ``_auto_snapshot`` は「新世代の作成 → 既定の rotate()」を続けて呼ぶ。既定の + rotate() は系列ごとに ``max_generations`` (ここでは 3) 世代を残すため、default + 系列を 3 世代 (最新の差分数 10 で上限) と with 系列を 1 世代の状態から default + で呼ぶと、default は新世代が積まれて 4 世代 → 最古が 1 つ落ちて 3 世代に戻り、 + with の 1 世代はそのまま残る。作成とローテーションのつなぎ目を固定する。 + """ + backups = write_state(root, [ + ("D1", "default", 0), ("D2", "default", 0), ("D3", "default", 10), + ("W1", "with", 0)]) + monkeypatch.setenv("DEVBASE_ACCOUNT_GROUP", "default") + + container._auto_snapshot() + + entries = yaml.safe_load((backups / "snapshot.yml").read_text())["snapshots"] + # default 系列は 3 世代 (最古の D1 が落ち、新世代が 1 つ増えた)。 + default_series = [e["name"] for e in entries + if e.get("volumes", {}).get("group") == "devbase_home_default"] + with_series = [e["name"] for e in entries + if e.get("volumes", {}).get("group") == "devbase_home_with"] + assert len(default_series) == 3 + assert "D1" not in default_series + assert {"D2", "D3"} <= set(default_series) + new_names = set(default_series) - {"D2", "D3"} + assert len(new_names) == 1 # 新しく積まれた 1 世代 + assert with_series == ["W1"] # with の 1 世代はそのまま残る + + # 消えた最古 (D1) のディレクトリは無く、with (W1) のディレクトリは残る。 + assert not (backups / "D1").exists() + assert (backups / "W1" / "full.tar.zst").exists() + new_name = next(iter(new_names)) + assert (backups / new_name / "full.tar.zst").exists() + + def test_invalid_group_warns_without_creating_metadata(root, monkeypatch, caplog): """現状固定: ボリューム解決の失敗は警告に落とし、呼び出し元へ戻る。""" monkeypatch.setenv("DEVBASE_ACCOUNT_GROUP", "invalid/group") From 391a5265e7ccc7c00090ce4ce1d6633dbc7a2419 Mon Sep 17 00:00:00 2001 From: "takemi.ohama" Date: Thu, 24 Sep 2026 12:27:50 +0900 Subject: [PATCH 04/14] =?UTF-8?q?Test:=20characterization=20=E2=80=94=20sn?= =?UTF-8?q?apshot=20restore=20args,=20auto-snapshot=20future=20guard,=20do?= =?UTF-8?q?wn=20rotate=20failure,=20dispatch=20error,=20create=20increment?= =?UTF-8?q?als?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - cli: restore の name / --point (int, 既定 None) の解析を固定 - _auto_snapshot: 直近の取得時刻が未来なら間隔内でも差分を積む下限ガードを固定 - cmd_down: 停止後に rotate が例外を送出しても 0 を返し停止済みのままであることを固定 - cmd_snapshot: handler の SnapshotError を捕えて 1 を返す経路を固定 - SnapshotManager.create: incr-002 の番号付けと incremental_count の更新、 snapshot.snar 欠落時の full への切り替えを固定 Item-Id: R2-001 Round: 2 Impl-Runtime: claude Impl-Model: default Co-Authored-By: Claude Opus 5.5 (1M context) --- tests/cli/test_snapshot_rotate_args.py | 17 ++++++ .../commands/test_container_down_snapshot.py | 39 ++++++++++++++ tests/commands/test_snapshot_dispatch.py | 14 +++++ tests/snapshot/test_auto_snapshot_series.py | 10 ++++ tests/snapshot/test_manager_series.py | 54 +++++++++++++++++++ 5 files changed, 134 insertions(+) create mode 100644 tests/commands/test_container_down_snapshot.py diff --git a/tests/cli/test_snapshot_rotate_args.py b/tests/cli/test_snapshot_rotate_args.py index c8253a73..a89ce301 100644 --- a/tests/cli/test_snapshot_rotate_args.py +++ b/tests/cli/test_snapshot_rotate_args.py @@ -29,3 +29,20 @@ def test_rotate_alias_max_total(): assert args.subcommand == "rotate" assert args.max_total == 4 assert type(args.max_total) is int + + +def test_restore_name_and_point(): + args = _create_parser().parse_args(["snapshot", "restore", "daily", "--point", "2"]) + + assert args.subcommand == "restore" + assert args.name == "daily" + assert args.point == 2 + assert type(args.point) is int + + +def test_restore_default_point(): + args = _create_parser().parse_args(["snapshot", "restore", "daily"]) + + assert args.subcommand == "restore" + assert args.name == "daily" + assert args.point is None diff --git a/tests/commands/test_container_down_snapshot.py b/tests/commands/test_container_down_snapshot.py new file mode 100644 index 00000000..f78f51a0 --- /dev/null +++ b/tests/commands/test_container_down_snapshot.py @@ -0,0 +1,39 @@ +"""``cmd_down`` の停止後のローテーションが失敗したときの現状固定テスト。 + +停止は疑似コンテナの状態を変えるだけにし、Docker と実データの ``backups/`` に触らない。 +""" + +from __future__ import annotations + +import pytest + +from devbase.commands import container +from devbase.env import runtime as secret_runtime +from devbase.snapshot.manager import SnapshotManager + + +@pytest.fixture +def stopped(tmp_path, monkeypatch): + monkeypatch.chdir(tmp_path) + monkeypatch.setenv('DEVBASE_ROOT', str(tmp_path)) + monkeypatch.setattr(container, '_apply_context', lambda context: None) + monkeypatch.setattr(container, '_inject_secrets', + lambda *, required: secret_runtime.SecretEnv()) + state = {'dev-1': 'running'} + + def fake_down(compose_file=None): + state['dev-1'] = 'stopped' + + monkeypatch.setattr(container, 'docker_compose_down', fake_down) + return state + + +def test_rotate_failure_after_stop_returns_zero(stopped, monkeypatch): + """現状固定: ローテーションの例外は吸収し、停止済みのまま 0 を返す。""" + def boom(self, *a, **k): + raise RuntimeError("rotate failed") + + monkeypatch.setattr(SnapshotManager, 'rotate', boom) + + assert container.cmd_down() == 0 + assert stopped == {'dev-1': 'stopped'} diff --git a/tests/commands/test_snapshot_dispatch.py b/tests/commands/test_snapshot_dispatch.py index e4059eb9..106352fd 100644 --- a/tests/commands/test_snapshot_dispatch.py +++ b/tests/commands/test_snapshot_dispatch.py @@ -12,3 +12,17 @@ def test_missing_or_unknown_subcommand_returns_error(tmp_path, subcommand): args = SimpleNamespace(subcommand=subcommand) assert cmd_snapshot(tmp_path, args) == 1 + + +def test_handler_snapshot_error_returns_error(tmp_path, monkeypatch): + """handler が SnapshotError を送出すると、捕えて 1 を返す (未指定・未知とは別経路)。""" + from devbase.errors import SnapshotError + from devbase.snapshot.manager import SnapshotManager + + def boom(self, name, new_name): + raise SnapshotError("copy failed") + + monkeypatch.setattr(SnapshotManager, 'copy', boom) + args = SimpleNamespace(subcommand="copy", name="a", new_name="b") + + assert cmd_snapshot(tmp_path, args) == 1 diff --git a/tests/snapshot/test_auto_snapshot_series.py b/tests/snapshot/test_auto_snapshot_series.py index 1f9c2c2e..a1d2d182 100644 --- a/tests/snapshot/test_auto_snapshot_series.py +++ b/tests/snapshot/test_auto_snapshot_series.py @@ -101,6 +101,16 @@ def test_zero_interval_never_skips(root): assert (backups / "D" / "incr-001.tar.zst").exists() +def test_future_last_snapshot_is_not_skipped(root, monkeypatch): + """現状固定: 直近の取得時刻が未来 (時計のずれ) なら、間隔内でも飛ばさず積む。""" + backups = write_state(root, [("D", "default", 0)]) + monkeypatch.setenv("DEVBASE_SNAPSHOT_MIN_INTERVAL_MINUTES", "60") + _age(backups / "D" / "full.tar.zst", -600) + + container._auto_snapshot() + assert (backups / "D" / "incr-001.tar.zst").exists() + + def test_new_generation_rotates_only_its_own_series(root, monkeypatch): """現状固定: 作成の後の rotate() で、新世代を積んだ系列の最古だけが消える。 diff --git a/tests/snapshot/test_manager_series.py b/tests/snapshot/test_manager_series.py index 01d6f19b..221e4f16 100644 --- a/tests/snapshot/test_manager_series.py +++ b/tests/snapshot/test_manager_series.py @@ -9,6 +9,7 @@ import logging import os +import re import time import types from pathlib import Path @@ -160,6 +161,59 @@ def test_series_latest_uses_created_at(tmp_path): assert mgr.series_latest(vols("kkg")) is None +class ArchiveRecordingManager(RecordingManager): + """書き込むアーカイブ名を command から拾う (full / incr-NNN を区別する)。""" + + def _run_docker_tar(self, snap_dir, mode, command, volumes=None): + self.calls.append({"mode": mode, "command": command}) + if mode == "backup": + archive = re.search(r"/backup/(full\.tar\.zst|incr-\d+\.tar\.zst)", command) + (snap_dir / archive.group(1)).write_text("archive") + (snap_dir / "snapshot.snar").write_text("snar") + + +def _entry(root: Path, name: str) -> dict: + data = yaml.safe_load((root / "backups" / "snapshot.yml").read_text()) + return next(s for s in data["snapshots"] if s["name"] == name) + + +def test_create_numbers_incrementals_in_order(tmp_path): + """現状固定: 2 本目以降の差分は incr-002 と番号を進め、差分数を両方の台帳へ書く。""" + mgr = ArchiveRecordingManager(tmp_path, group="default") + mgr.create(name="g") + mgr.create(name="g", full=False) + mgr.create(name="g", full=False) + + snap_dir = tmp_path / "backups" / "g" + assert sorted(p.name for p in snap_dir.glob("*.tar.zst")) == [ + "full.tar.zst", "incr-001.tar.zst", "incr-002.tar.zst"] + meta = yaml.safe_load((snap_dir / "meta.yml").read_text()) + assert meta["type"] == "incremental" + assert meta["incremental_count"] == 2 + assert meta["files"] == ["full.tar.zst", "incr-001.tar.zst", "incr-002.tar.zst"] + assert _entry(tmp_path, "g")["incremental_count"] == 2 + + +def test_create_without_snar_falls_back_to_full(tmp_path): + """現状固定: 既存世代に snapshot.snar が無ければ差分でなく full を作り直す。""" + mgr = ArchiveRecordingManager(tmp_path, group="default") + mgr.create(name="h") + mgr.create(name="h", full=False) + snap_dir = tmp_path / "backups" / "h" + (snap_dir / "snapshot.snar").unlink() + + mgr.create(name="h", full=False) + + assert "incr-002.tar.zst" not in [p.name for p in snap_dir.iterdir()] + assert "incr-002" not in mgr.calls[-1]["command"] + assert "/backup/full.tar.zst" in mgr.calls[-1]["command"] + meta = yaml.safe_load((snap_dir / "meta.yml").read_text()) + assert meta["type"] == "full" + assert meta["incremental_count"] == 0 + assert meta["files"] == ["full.tar.zst"] + assert _entry(tmp_path, "h")["incremental_count"] == 0 + + # --------------------------------------------------------------------------- # 系列ごとの最小間隔 (受け入れ条件 6) # --------------------------------------------------------------------------- From 5fc82399d984c947837a1430a09fce57eb8e20a0 Mon Sep 17 00:00:00 2001 From: "takemi.ohama" Date: Thu, 24 Sep 2026 12:37:32 +0900 Subject: [PATCH 05/14] Refactor: extract snapshot interval and series rules Extract the automatic snapshot interval check and share series filtering and generation age ordering. Reuse HOME_UBUNTU_VOLUME for legacy snapshot listings without changing behavior. Item-Id: R3-001 Round: 3 Impl-Runtime: codex Impl-Model: default --- lib/devbase/commands/container.py | 37 +++++++++++++++++++------------ lib/devbase/commands/snapshot.py | 3 ++- lib/devbase/snapshot/manager.py | 31 ++++++++++++++++---------- 3 files changed, 44 insertions(+), 27 deletions(-) diff --git a/lib/devbase/commands/container.py b/lib/devbase/commands/container.py index d7bdc650..60fb58e6 100644 --- a/lib/devbase/commands/container.py +++ b/lib/devbase/commands/container.py @@ -803,6 +803,27 @@ def _snapshot_min_interval_minutes() -> int: ) +def _should_skip_by_interval( + label: str, min_interval: int, last: Optional[datetime], +) -> bool: + """系列の直近の取得から最小間隔内なら、理由を記録してスキップする。""" + from datetime import datetime, timedelta, timezone + + if min_interval > 0 and last is not None: + # 経過時間が負 (last が未来) の場合はスキップしない。システム時計の + # ズレや他環境からのリストアで last が未来になると delta が負になり、 + # 常に閾値未満と判定されて無期限にスキップされてしまうため、 + # timedelta(0) <= delta の下限ガードを設ける。 + delta = datetime.now(timezone.utc) - last + if timedelta(0) <= delta < timedelta(minutes=min_interval): + logger.info( + "[0/6] %s の直近のスナップショット (%s) から%d分以内のためスキップします", + label, last.astimezone().strftime('%Y-%m-%d %H:%M:%S'), min_interval, + ) + return True + return False + + def _auto_snapshot(remote: bool = False) -> None: """デプロイ前の自動スナップショット (系列ごとの差分世代数ベース世代管理。PLAN68)。 @@ -820,8 +841,6 @@ def _auto_snapshot(remote: bool = False) -> None: if not devbase_root: return try: - from datetime import datetime, timedelta, timezone - from devbase.snapshot.manager import SnapshotManager mgr = SnapshotManager(Path(devbase_root)) # 最小間隔・積み先はどちらも系列 (起動するグループのボリュームの組) の @@ -829,18 +848,8 @@ def _auto_snapshot(remote: bool = False) -> None: label = mgr.series_label(mgr.volumes) min_interval = _snapshot_min_interval_minutes() last = mgr.last_snapshot_time(mgr.volumes) - if min_interval > 0 and last is not None: - # 経過時間が負 (last が未来) の場合はスキップしない。システム時計の - # ズレや他環境からのリストアで last が未来になると delta が負になり、 - # 常に閾値未満と判定されて無期限にスキップされてしまうため、 - # timedelta(0) <= delta の下限ガードを設ける。 - delta = datetime.now(timezone.utc) - last - if timedelta(0) <= delta < timedelta(minutes=min_interval): - logger.info( - "[0/6] %s の直近のスナップショット (%s) から%d分以内のためスキップします", - label, last.astimezone().strftime('%Y-%m-%d %H:%M:%S'), min_interval, - ) - return + if _should_skip_by_interval(label, min_interval, last): + return target = mgr.auto_snapshot_target() if target is None: logger.info("[0/6] 新しいスナップショット世代を作成中 (%s)...", label) diff --git a/lib/devbase/commands/snapshot.py b/lib/devbase/commands/snapshot.py index b0ce330e..a00172d3 100644 --- a/lib/devbase/commands/snapshot.py +++ b/lib/devbase/commands/snapshot.py @@ -6,6 +6,7 @@ from devbase.errors import SnapshotError from devbase.log import get_logger from devbase.snapshot.manager import SnapshotManager +from devbase.volume.manager import HOME_UBUNTU_VOLUME logger = get_logger(__name__) @@ -68,7 +69,7 @@ def _snapshot_list(mgr) -> int: print("-" * 90) for s in snapshots: # 対象ボリュームは PLAN39 以降に記録される。旧世代は共通ボリュームのみ。 - volumes = ', '.join((s.get('volumes') or {}).values()) or 'devbase_home_ubuntu' + volumes = ', '.join((s.get('volumes') or {}).values()) or HOME_UBUNTU_VOLUME print( f"{s['name']:<24} " f"{s.get('created_at', 'N/A')[:19]:<24} " diff --git a/lib/devbase/snapshot/manager.py b/lib/devbase/snapshot/manager.py index 4e5b4746..33826d56 100644 --- a/lib/devbase/snapshot/manager.py +++ b/lib/devbase/snapshot/manager.py @@ -248,11 +248,9 @@ def last_snapshot_time(self, volumes: Optional[dict] = None) -> Optional[datetim if volumes is None: snap_dirs = list(self.backups_dir.iterdir()) else: - key = self.series_key(volumes) snap_dirs = [ - self.backups_dir / s['name'] for s in self._entries() - if self.series_key(self._entry_volumes(s)) == key - and is_single_segment_name(s['name']) + self.backups_dir / s['name'] for _, s in self._series_entries(volumes) + if is_single_segment_name(s['name']) ] latest: Optional[float] = None for snap_dir in snap_dirs: @@ -542,7 +540,7 @@ def rotate(self, keep: int = DEFAULT_MAX_GENERATIONS, remaining = [s for i, s in enumerate(snapshots) if i not in removed_ids] order = {id(s): i for i, s in enumerate(snapshots)} - remaining.sort(key=lambda s: (s.get('created_at', '') or '', order[id(s)])) + remaining.sort(key=lambda s: self._entry_age(s, order[id(s)])) meta['snapshots'] = remaining meta['max_generations'] = keep self._save_metadata(meta) @@ -561,8 +559,7 @@ def _rotation_plan(self, snapshots: list, keep: int, max_total: int) -> list: groups.setdefault(key, []).append(index) def age(index: int) -> tuple: - # created_at が同じなら snapshot.yml で前にあるものを古いとみなす - return (snapshots[index].get('created_at', '') or '', index) + return self._entry_age(snapshots[index], index) plan: list = [] kept: dict = {} @@ -589,6 +586,11 @@ def age(index: int) -> tuple: # 系列 (PLAN68) # ------------------------------------------------------------------ + @staticmethod + def _entry_age(entry: dict, index: int) -> tuple: + # created_at が同じなら snapshot.yml で前にあるものを古いとみなす + return (entry.get('created_at', '') or '', index) + @staticmethod def _entry_volumes(entry: dict) -> dict: """``snapshot.yml`` のエントリの対象ボリュームの組。 @@ -621,18 +623,23 @@ def _entries(self) -> list: return [s for s in (self._load_metadata().get('snapshots') or []) if isinstance(s, dict) and 'name' in s] + def _series_entries(self, volumes: dict) -> "list[tuple[int, dict]]": + """対象ボリュームの系列に属するエントリを元の添字とともに返す。""" + key = self.series_key(volumes) + return [ + (index, snap) for index, snap in enumerate(self._entries()) + if self.series_key(self._entry_volumes(snap)) == key + ] + def series_latest(self, volumes: Optional[dict] = None) -> Optional[dict]: """系列の最新の世代のエントリ (``created_at`` が最大)。無ければ ``None``。 ``created_at`` が同じなら ``snapshot.yml`` で後ろのものを新しいとみなす。 """ - key = self.series_key(self.volumes if volumes is None else volumes) latest = None latest_age = None - for index, snap in enumerate(self._entries()): - if self.series_key(self._entry_volumes(snap)) != key: - continue - age = (snap.get('created_at', '') or '', index) + for index, snap in self._series_entries(self.volumes if volumes is None else volumes): + age = self._entry_age(snap, index) if latest_age is None or age > latest_age: latest, latest_age = snap, age return latest From c2d75e009be80baa1d930c02c55590122eb83843 Mon Sep 17 00:00:00 2001 From: "takemi.ohama" Date: Thu, 24 Sep 2026 12:44:12 +0900 Subject: [PATCH 06/14] =?UTF-8?q?Refactor:=20consolidate=5Fduplication=20/?= =?UTF-8?q?=20extract=5Fmethod=20=E2=80=94=20snapshot=20archive=20names=20?= =?UTF-8?q?&=20=5Fensure=5Fenv=5Ffiles?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit R3-003: 差分アーカイブの命名規約 (incr- 接頭辞・.tar.zst 接尾辞・incr-{n:03d} 書式・incr-*.tar.zst グロブ・^incr-(\d+)\.tar\.zst$ 正規表現) が _create_incremental・restore・last_snapshot_time に散らばっていたのを、 モジュール先頭の定数 (INCR_PREFIX / ARCHIVE_SUFFIX / FULL_ARCHIVE / INCR_GLOB / _INCR_ARCHIVE_RE) と helper (incr_archive_name / is_incr_archive) へ寄せた。 R3-004: _ensure_env_files から、子プロセスの env init 実行・ログ・例外処理・ finally の release_store を _initialize_global_env(devbase_root, group) へ、 project .env の touch・ログ・例外処理を _create_project_env(project_env) へ 抽出した。呼び出し元には存在判定と順序制御・結果の集約を残す。抽出経路 (非ゼロ終了・起動例外・touch 失敗) の現状固定テストを追加した。 振る舞いは不変。tests/snapshot tests/cli tests/commands は 1561 passed / 1 skipped。 Item-Id: R3-003 Round: 3 Impl-Runtime: kiro Impl-Model: default --- lib/devbase/commands/container.py | 91 +++++++++++++++++++----------- lib/devbase/snapshot/manager.py | 43 +++++++++----- tests/cli/test_up_roundtrips.py | 94 +++++++++++++++++++++++++++++++ 3 files changed, 183 insertions(+), 45 deletions(-) diff --git a/lib/devbase/commands/container.py b/lib/devbase/commands/container.py index 60fb58e6..de3bfd96 100644 --- a/lib/devbase/commands/container.py +++ b/lib/devbase/commands/container.py @@ -1949,48 +1949,75 @@ def _ensure_env_files() -> bool: logger.info("Running 'devbase env init' to create them...") success = True - child_env = {**os.environ, 'PYTHONPATH': str(devbase_root / 'lib')} - # 子プロセスは cwd=$DEVBASE_ROOT で起動し、実行時のプロジェクトを持たない。グループを - # 渡さないと $DEVBASE_ROOT/env のグループの共通の参照へ書き、親が読み直す参照と - # 揃わない (PLAN56 決定 10)。layout: group でないときは渡さない - init_argv = [sys.executable, '-m', 'devbase.cli', 'env', 'init'] - if group is not None: - init_argv += ['--group', group] if not has_global: - logger.info("Creating devbase root .env...") - try: - result = subprocess.run( - init_argv, - env=child_env, - cwd=str(devbase_root), - check=False - ) - if result.returncode != 0: - success = False - logger.error("Failed to create devbase root .env") - except Exception as e: - logger.error("Running env init for devbase root: %s", e) + if not _initialize_global_env(devbase_root, group): success = False - finally: - # 書いたのは子プロセスで、持ち回っている SecretStore の控えは更新されない。 - # サーバ backend では最初の 404 が空として残り、そのまま起動すると env init が - # 書いた共通機密が渡らない。終了コードによらず捨て、以後は現物を読み直す - # (PLAN55 決定 5)。 - _runtime.release_store() if not has_project: - logger.info("Creating project .env...") - try: - project_env.touch() - logger.info("Created empty project .env: %s", project_env) - except Exception as e: - logger.error("Failed to create project .env: %s", e) + if not _create_project_env(project_env): success = False return success +def _initialize_global_env(devbase_root: Path, group: Optional[str]) -> bool: + """子プロセスの ``devbase env init`` で共通の ``.env`` を作る。 + + 子プロセスは cwd=$DEVBASE_ROOT で起動し、実行時のプロジェクトを持たない。グループを + 渡さないと $DEVBASE_ROOT/env のグループの共通の参照へ書き、親が読み直す参照と + 揃わない (PLAN56 決定 10)。layout: group でないときは渡さない。 + + 終了コードによらず、持ち回っている SecretStore の控えは解放する。書いたのは子プロセス + で控えは更新されないため、サーバ backend では最初の 404 が空として残り、そのまま起動 + すると env init が書いた共通機密が渡らない。以後は現物を読み直す (PLAN55 決定 5)。 + + Returns: + 子プロセスが正常終了 (終了コード 0) したら True。非ゼロ終了・起動時の例外は False。 + """ + from devbase.env import runtime as _runtime + + logger.info("Creating devbase root .env...") + child_env = {**os.environ, 'PYTHONPATH': str(devbase_root / 'lib')} + init_argv = [sys.executable, '-m', 'devbase.cli', 'env', 'init'] + if group is not None: + init_argv += ['--group', group] + + success = True + try: + result = subprocess.run( + init_argv, + env=child_env, + cwd=str(devbase_root), + check=False + ) + if result.returncode != 0: + success = False + logger.error("Failed to create devbase root .env") + except Exception as e: + logger.error("Running env init for devbase root: %s", e) + success = False + finally: + _runtime.release_store() + return success + + +def _create_project_env(project_env: Path) -> bool: + """空のプロジェクト ``.env`` を作る。 + + Returns: + 作成できたら True。作成に失敗したら False。 + """ + logger.info("Creating project .env...") + try: + project_env.touch() + logger.info("Created empty project .env: %s", project_env) + except Exception as e: + logger.error("Failed to create project .env: %s", e) + return False + return True + + _IMAGE_MAX_AGE_DAYS_DEFAULT = 7 diff --git a/lib/devbase/snapshot/manager.py b/lib/devbase/snapshot/manager.py index 33826d56..ad903a31 100644 --- a/lib/devbase/snapshot/manager.py +++ b/lib/devbase/snapshot/manager.py @@ -37,6 +37,26 @@ DEFAULT_MAX_INCREMENTALS = 10 METADATA_FILE = 'snapshot.yml' +# 差分アーカイブの命名規約 (接頭辞・接尾辞・連番書式・グロブ・正規表現) を 1 箇所へ +# 寄せる。片方だけ書式や桁を変えると作成・列挙・復元の照合が崩れるため、同じ規約に +# 由来する重複としてここでまとめて持つ。 +INCR_PREFIX = 'incr-' +ARCHIVE_SUFFIX = '.tar.zst' +FULL_ARCHIVE = f'full{ARCHIVE_SUFFIX}' +INCR_GLOB = f'{INCR_PREFIX}*{ARCHIVE_SUFFIX}' +_INCR_ARCHIVE_RE = re.compile( + r'^' + re.escape(INCR_PREFIX) + r'(\d+)' + re.escape(ARCHIVE_SUFFIX) + r'$') + + +def incr_archive_name(n: int) -> str: + """差分番号から差分アーカイブのファイル名を組み立てる (``incr-003.tar.zst``)。""" + return f'{INCR_PREFIX}{n:03d}{ARCHIVE_SUFFIX}' + + +def is_incr_archive(name: str) -> bool: + """名前が差分アーカイブの実体かどうか。""" + return bool(_INCR_ARCHIVE_RE.match(name)) + # GNU tar の incremental はディレクトリを (dev, ino) で追跡して rename を検出する。 # ディレクトリが削除され作り直されると **inode 番号が再利用される**ため、tar は無関係な # ディレクトリを rename されたものと誤判定し、dumpdir に偽の R/T レコードを書く。 @@ -261,9 +281,7 @@ def last_snapshot_time(self, volumes: Optional[dict] = None) -> Optional[datetim continue # アーカイブ実体 (full.tar.zst / incr-NNN.tar.zst) のみを対象とし、 # meta.yml / snapshot.snar / *.bak 等は除外する。 - if f.name != 'full.tar.zst' and not ( - f.name.startswith('incr-') and f.name.endswith('.tar.zst') - ): + if f.name != FULL_ARCHIVE and not is_incr_archive(f.name): continue mtime = f.stat().st_mtime if latest is None or mtime > latest: @@ -286,7 +304,7 @@ def restore(self, name: str, point: int | None = None) -> None: if not snap_dir.exists(): raise SnapshotError(f"スナップショット '{name}' が見つかりません") - full_archive = snap_dir / 'full.tar.zst' + full_archive = snap_dir / FULL_ARCHIVE if not full_archive.exists(): raise SnapshotError(f"フルバックアップが見つかりません: {full_archive}") @@ -310,19 +328,18 @@ def restore(self, name: str, point: int | None = None) -> None: # フルバックアップの復元 logger.info("フルバックアップを復元中...") self._extract_archive( - snap_dir, 'full.tar.zst', + snap_dir, FULL_ARCHIVE, self.clear_command(volumes) + - "zstd -d /backup/full.tar.zst -c | " + f"zstd -d /backup/{FULL_ARCHIVE} -c | " "tar --listed-incremental=/dev/null -xf - -C /target", volumes, pre_restore_name, skipped_renames, ) # 差分バックアップを順番に適用（pointが指定されていればそこまで） - incr_re = re.compile(r'^incr-(\d+)\.tar\.zst$') - incr_files = sorted(snap_dir.glob('incr-*.tar.zst')) + incr_files = sorted(snap_dir.glob(INCR_GLOB)) for incr in incr_files: if point is not None: - m = incr_re.match(incr.name) + m = _INCR_ARCHIVE_RE.match(incr.name) if not m: continue if int(m.group(1)) > point: @@ -804,7 +821,7 @@ def _create_full(self, name: str, snap_dir: Path) -> None: self._run_docker_tar( snap_dir, 'backup', "tar --listed-incremental=/backup/snapshot.snar " - "-cf - -C /source . | zstd -1 -T0 -o /backup/full.tar.zst" + f"-cf - -C /source . | zstd -1 -T0 -o /backup/{FULL_ARCHIVE}" ) # meta.yml を作成 @@ -813,7 +830,7 @@ def _create_full(self, name: str, snap_dir: Path) -> None: 'created_at': datetime.now().isoformat(), 'type': 'full', 'volumes': dict(self.volumes), - 'files': ['full.tar.zst'], + 'files': [FULL_ARCHIVE], 'incremental_count': 0, } self._save_snap_meta(snap_dir, meta) @@ -840,9 +857,9 @@ def _create_incremental(self, name: str, snap_dir: Path) -> None: return # 差分番号を決定 - existing = sorted(snap_dir.glob('incr-*.tar.zst')) + existing = sorted(snap_dir.glob(INCR_GLOB)) next_num = len(existing) + 1 - incr_name = f'incr-{next_num:03d}.tar.zst' + incr_name = incr_archive_name(next_num) logger.info("差分バックアップを作成中: %s/%s", name, incr_name) diff --git a/tests/cli/test_up_roundtrips.py b/tests/cli/test_up_roundtrips.py index 8f293553..6ac82ee2 100644 --- a/tests/cli/test_up_roundtrips.py +++ b/tests/cli/test_up_roundtrips.py @@ -405,3 +405,97 @@ def fake_run(argv, **kwargs): container._ensure_env_files() assert seen['argv'][-2:] == ['env', 'init'] + + +# --------------------------------------------------------------------------- +# R3-004 現状固定: `_ensure_env_files` の異常系 (extract_method の前に固定) +# --------------------------------------------------------------------------- + +def _pop_global_and_project(openbao): + """`web` の共通機密とプロジェクト機密を落とし、env init と project .env 作成を通す。""" + for path in (TEAM_GLOBAL, TEAM_WEB): + openbao.secrets.pop(path, None) + openbao.versions.pop(path, None) + + +def test_ensure_env_files_nonzero_exit_returns_false_but_creates_project_env( + up_root, openbao, monkeypatch): + """現状固定: env init が非ゼロ終了しても project .env は作られ、戻り値は False。 + + さらに終了コードによらずストアのキャッシュは解放され、次の `runtime.resolve` は + サーバから読み直す。 + """ + root = up_root['root'] + _pop_global_and_project(openbao) + project_dir = root / 'projects' / 'web' + monkeypatch.chdir(project_dir) + monkeypatch.setenv('PWD', str(project_dir)) + project_env = project_dir / '.env' + if project_env.exists(): + project_env.unlink() + + def fake_run(argv, **kwargs): + return subprocess.CompletedProcess(argv, 1) + + monkeypatch.setattr(container.subprocess, 'run', fake_run) + # 事前にストアを載せておき、解放されたことをログイン回数の増分で見る + runtime.resolve(root) + logins_before = openbao.logins + + assert container._ensure_env_files() is False + # 子プロセスが失敗しても project .env の作成は試みる + assert project_env.exists() + # 失敗後もキャッシュは解放され、resolve が新たに認証する + runtime.resolve(root) + assert openbao.logins > logins_before + + +def test_ensure_env_files_env_init_launch_exception_returns_false( + up_root, openbao, monkeypatch): + """現状固定: env init の起動そのものが例外でも、握りつぶして False を返す。""" + root = up_root['root'] + _pop_global_and_project(openbao) + project_dir = root / 'projects' / 'web' + monkeypatch.chdir(project_dir) + monkeypatch.setenv('PWD', str(project_dir)) + project_env = project_dir / '.env' + if project_env.exists(): + project_env.unlink() + + def boom(argv, **kwargs): + raise OSError('cannot launch') + + monkeypatch.setattr(container.subprocess, 'run', boom) + + assert container._ensure_env_files() is False + # 例外でも後続の project .env 作成は続く + assert project_env.exists() + + +def test_ensure_env_files_project_env_touch_failure_returns_false( + up_root, openbao, monkeypatch): + """現状固定: project .env の作成 (touch) が失敗すると False を返す。""" + root = up_root['root'] + _pop_global_and_project(openbao) + project_dir = root / 'projects' / 'web' + monkeypatch.chdir(project_dir) + monkeypatch.setenv('PWD', str(project_dir)) + project_env = project_dir / '.env' + if project_env.exists(): + project_env.unlink() + + def ok_run(argv, **kwargs): + return subprocess.CompletedProcess(argv, 0) + + monkeypatch.setattr(container.subprocess, 'run', ok_run) + + real_touch = Path.touch + + def failing_touch(self, *a, **k): + if self.name == '.env': + raise PermissionError('read-only') + return real_touch(self, *a, **k) + + monkeypatch.setattr(Path, 'touch', failing_touch) + + assert container._ensure_env_files() is False From 5ed1d6157a9bb25e6edb0e602a76148d3f4ffc5b Mon Sep 17 00:00:00 2001 From: "takemi.ohama" Date: Thu, 24 Sep 2026 12:50:16 +0900 Subject: [PATCH 07/14] =?UTF-8?q?Refactor:=20consolidate=5Fduplication=20?= =?UTF-8?q?=E2=80=94=20lib/devbase/snapshot/manager.py#SnapshotManager.res?= =?UTF-8?q?tore?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit full と各 incr の展開で重複していた zstd | tar --listed-incremental=/dev/null のコマンド組み立てを _incremental_extract_command(archive) に集約した。 生成されるコマンド文字列は変わらない。 Item-Id: R4-001 Round: 4 Impl-Runtime: claude Impl-Model: default Co-Authored-By: Claude Opus 5.5 (1M context) --- lib/devbase/snapshot/manager.py | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/lib/devbase/snapshot/manager.py b/lib/devbase/snapshot/manager.py index ad903a31..6b496a27 100644 --- a/lib/devbase/snapshot/manager.py +++ b/lib/devbase/snapshot/manager.py @@ -330,8 +330,7 @@ def restore(self, name: str, point: int | None = None) -> None: self._extract_archive( snap_dir, FULL_ARCHIVE, self.clear_command(volumes) + - f"zstd -d /backup/{FULL_ARCHIVE} -c | " - "tar --listed-incremental=/dev/null -xf - -C /target", + self._incremental_extract_command(FULL_ARCHIVE), volumes, pre_restore_name, skipped_renames, ) @@ -347,8 +346,7 @@ def restore(self, name: str, point: int | None = None) -> None: logger.info("差分バックアップを適用中: %s", incr.name) self._extract_archive( snap_dir, incr.name, - f"zstd -d /backup/{incr.name} -c | " - f"tar --listed-incremental=/dev/null -xf - -C /target", + self._incremental_extract_command(incr.name), volumes, pre_restore_name, skipped_renames, ) @@ -777,6 +775,12 @@ def clear_command(volumes: dict) -> str: 'done; ' ) + @staticmethod + def _incremental_extract_command(archive: str) -> str: + """``/backup/`` を ``/target`` へ展開するコマンドを組み立てる。""" + return (f"zstd -d /backup/{archive} -c | " + "tar --listed-incremental=/dev/null -xf - -C /target") + def _run_docker_tar(self, snap_dir: Path, mode: str, command: str, volumes: Optional[dict] = None ) -> subprocess.CompletedProcess: From 346f49ca840971270eba4642f2fc310b3e9db0bf Mon Sep 17 00:00:00 2001 From: "takemi.ohama" Date: Thu, 24 Sep 2026 12:57:18 +0900 Subject: [PATCH 08/14] =?UTF-8?q?Refactor:=20introduce=5Fnamed=5Fconstant?= =?UTF-8?q?=20=E2=80=94=20lib/devbase/snapshot/manager.py#SnapshotManager.?= =?UTF-8?q?=5Frotation=5Fplan?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ローテーションの削除理由を None / 'total' からモジュール定数 _REASON_PER_SERIES / _REASON_TOTAL に置き換え、_rotation_plan の生成側と rotate の読み分けを名前で対応付ける。値は rotate の外へ出ないため振る舞いは不変。 Item-Id: R4-003 Round: 4 Impl-Runtime: claude Impl-Model: default Co-Authored-By: Claude Opus 5.5 (1M context) --- lib/devbase/snapshot/manager.py | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/lib/devbase/snapshot/manager.py b/lib/devbase/snapshot/manager.py index 6b496a27..0cda91c1 100644 --- a/lib/devbase/snapshot/manager.py +++ b/lib/devbase/snapshot/manager.py @@ -89,6 +89,10 @@ def rename_only_failure(stderr: str) -> Optional[list]: # 失敗した rename が大量に出うるので、余裕をもって分割する。 _CHECK_COMMAND_BUDGET = 60_000 +# ローテーションで世代を消す理由。系列ごとの保持数を超えた分か、全体の上限を超えた分か。 +_REASON_PER_SERIES = 'series' +_REASON_TOTAL = 'total' + def chunk_paths(paths: list, budget: int = _CHECK_COMMAND_BUDGET) -> list: """引用済みのパスを、1 コマンドの長さが budget を超えないように分ける。 @@ -538,14 +542,14 @@ def rotate(self, keep: int = DEFAULT_MAX_GENERATIONS, if snap_dir.exists(): shutil.rmtree(snap_dir) deleted_ids.add(index) - if reason is not None: + if reason == _REASON_TOTAL: logger.info( "ローテーション: 全体の上限 %d 世代を超えたため、%s の %s を削除しました", max_total, self.series_label(self._entry_volumes(snap)), name) per_series: dict = {} for index, reason in plan: - if reason is None and index in deleted_ids: + if reason == _REASON_PER_SERIES and index in deleted_ids: label = self.series_label(self._entry_volumes(snapshots[index])) per_series[label] = per_series.get(label, 0) + 1 for label, count in per_series.items(): @@ -565,8 +569,8 @@ def _rotation_plan(self, snapshots: list, keep: int, max_total: int) -> list: """ローテーションで消すエントリを決める (副作用なし)。 Returns: - ``(snapshots の添字, 理由)`` の並び。理由は系列ごとの保持なら ``None``、 - 全体の上限なら ``'total'``。 + ``(snapshots の添字, 理由)`` の並び。理由は系列ごとの保持なら + ``_REASON_PER_SERIES``、全体の上限なら ``_REASON_TOTAL``。 """ groups: dict = {} for index, snap in enumerate(snapshots): @@ -581,7 +585,7 @@ def age(index: int) -> tuple: for key, indexes in groups.items(): indexes.sort(key=age) excess = max(0, len(indexes) - keep) - plan.extend((i, None) for i in indexes[:excess]) + plan.extend((i, _REASON_PER_SERIES) for i in indexes[:excess]) kept[key] = indexes[excess:] total = sum(len(v) for v in kept.values()) @@ -593,7 +597,7 @@ def age(index: int) -> tuple: "消さないため %d 世代を残します", max_total, total) break oldest = min(candidates, key=lambda v: age(v[0])) - plan.append((oldest.pop(0), 'total')) + plan.append((oldest.pop(0), _REASON_TOTAL)) total -= 1 return plan From 2b63b89c5632a2eaa4d833eb5287a58b378df7ed Mon Sep 17 00:00:00 2001 From: "takemi.ohama" Date: Thu, 24 Sep 2026 13:00:24 +0900 Subject: [PATCH 09/14] =?UTF-8?q?Refactor:=20consolidate=5Fduplication=20?= =?UTF-8?q?=E2=80=94=20lib/devbase/snapshot/manager.py#SnapshotManager.cle?= =?UTF-8?q?ar=5Fcommand?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit サブディレクトリ名をコンテナ内パスへ写す規約（sub があれば {root}/{sub}、空文字なら root）が volume_mount_args と clear_command に別々に書かれていたため、静的補助 _container_path へ集約した。 Item-Id: R4-004 Round: 4 Impl-Runtime: claude Impl-Model: default Co-Authored-By: Claude Opus 5.5 (1M context) --- lib/devbase/snapshot/manager.py | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/lib/devbase/snapshot/manager.py b/lib/devbase/snapshot/manager.py index 0cda91c1..359dcaa8 100644 --- a/lib/devbase/snapshot/manager.py +++ b/lib/devbase/snapshot/manager.py @@ -749,6 +749,14 @@ def _ensure_snapshot_image(self) -> str: logger.info("devbase-snapshotイメージのビルド完了") return SNAPSHOT_IMAGE + @staticmethod + def _container_path(root: str, sub: str) -> str: + """サブディレクトリ名 ``sub`` をコンテナ内のパスへ写す。 + + 空文字は旧レイアウトで、``root`` そのものを指す。 + """ + return f'{root}/{sub}' if sub else root + @staticmethod def volume_mount_args(volumes: dict, mode: str) -> list: """対象ボリュームの ``docker run -v`` 引数を組み立てる。 @@ -760,7 +768,7 @@ def volume_mount_args(volumes: dict, mode: str) -> list: suffix = ':ro' if mode == 'backup' else '' args = [] for sub, name in volumes.items(): - target = f'{root}/{sub}' if sub else root + target = SnapshotManager._container_path(root, sub) args.extend(['-v', f'{name}:{target}{suffix}']) return args @@ -772,7 +780,7 @@ def clear_command(volumes: dict) -> str: 旧レイアウトも同じ形で扱える。 """ roots = ' '.join( - f'/target/{sub}' if sub else '/target' for sub in volumes) + SnapshotManager._container_path('/target', sub) for sub in volumes) return ( 'for d in ' + roots + '; do ' 'find "$d" -mindepth 1 -maxdepth 1 -exec rm -rf -- {} + 2>/dev/null; ' From ae15233b36882f02db578ed02a10ea0dad99bb90 Mon Sep 17 00:00:00 2001 From: "takemi.ohama" Date: Thu, 24 Sep 2026 13:04:07 +0900 Subject: [PATCH 10/14] =?UTF-8?q?Refactor:=20remove=5Fdead=5Fcode=20?= =?UTF-8?q?=E2=80=94=20SnapshotManager.should=5Fstart=5Fnew=5Fgeneration?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Remove the unused generation predicate wrapper and preserve its boolean assertions through auto_snapshot_target. Item-Id: R4-005 Round: 4 Impl-Runtime: codex Impl-Model: default --- lib/devbase/snapshot/manager.py | 10 ---------- tests/snapshot/test_manager_series.py | 2 +- tests/snapshot/test_manager_volumes.py | 6 +++--- 3 files changed, 4 insertions(+), 14 deletions(-) diff --git a/lib/devbase/snapshot/manager.py b/lib/devbase/snapshot/manager.py index 359dcaa8..4de13efa 100644 --- a/lib/devbase/snapshot/manager.py +++ b/lib/devbase/snapshot/manager.py @@ -696,16 +696,6 @@ def auto_snapshot_target( return None return name - def should_start_new_generation( - self, max_incrementals: int = DEFAULT_MAX_INCREMENTALS, - ) -> bool: - """新しい世代を作るべきか (``auto_snapshot_target(...) is None``)。 - - 系列 (対象ボリュームの組) の最新の世代に世代が無い、または差分数が上限以上 - なら True。グループを切り替えただけでは True にならない (PLAN68)。 - """ - return self.auto_snapshot_target(max_incrementals) is None - # ------------------------------------------------------------------ # Private helpers # ------------------------------------------------------------------ diff --git a/tests/snapshot/test_manager_series.py b/tests/snapshot/test_manager_series.py index 221e4f16..b13ff13a 100644 --- a/tests/snapshot/test_manager_series.py +++ b/tests/snapshot/test_manager_series.py @@ -110,7 +110,7 @@ def test_returning_group_appends_to_its_latest_generation(tmp_path): mgr = RecordingManager(tmp_path, group="default") assert mgr.auto_snapshot_target() == "A" - assert mgr.should_start_new_generation() is False + assert (mgr.auto_snapshot_target() is None) is False def test_group_without_generation_starts_a_new_one(tmp_path, caplog): diff --git a/tests/snapshot/test_manager_volumes.py b/tests/snapshot/test_manager_volumes.py index f5699a6a..5b69dc80 100644 --- a/tests/snapshot/test_manager_volumes.py +++ b/tests/snapshot/test_manager_volumes.py @@ -200,7 +200,7 @@ def test_layout_change_starts_a_new_generation(root): _write_legacy_snapshot(root) mgr = RecordingManager(root) - assert mgr.should_start_new_generation() is True + assert (mgr.auto_snapshot_target() is None) is True def test_group_change_starts_a_new_generation(root, monkeypatch): @@ -210,14 +210,14 @@ def test_group_change_starts_a_new_generation(root, monkeypatch): monkeypatch.setenv("DEVBASE_ACCOUNT_GROUP", "kkg") other = RecordingManager(root) - assert other.should_start_new_generation() is True + assert (other.auto_snapshot_target() is None) is True def test_same_layout_keeps_appending_increments(root): mgr = RecordingManager(root) mgr.create(name="snap1") - assert mgr.should_start_new_generation() is False + assert (mgr.auto_snapshot_target() is None) is False def test_incremental_on_a_different_layout_is_refused(root): From a81af47d6053e9838d6847e0618d01c2e5d3e656 Mon Sep 17 00:00:00 2001 From: "takemi.ohama" Date: Thu, 24 Sep 2026 13:07:49 +0900 Subject: [PATCH 11/14] =?UTF-8?q?Revert:=20=E7=AF=84=E5=9B=B2=E5=A4=96?= =?UTF-8?q?=E3=81=AE=E6=A7=8B=E9=80=A0=E6=94=B9=E5=96=84=E3=82=92=20PR=20?= =?UTF-8?q?=E3=81=8B=E3=82=89=E5=A4=96=E3=81=99=20(PLAN68)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit cross-refactoring の提案のうち、この PR が変えていない既存の関数に向いたものを戻す。 - R3-003/R3-004 (c2d75e0): restore のアーカイブ名の集約と _ensure_env_files の分割、その現状固定テスト - R3-005 (5fc8239 の commands/snapshot.py 分): _snapshot_list の定数化 - R4-001 (5ed1d61): restore の展開コマンドの集約 - R4-004 (2b63b89): clear_command の重複の集約 範囲外の提案は別の課題として起票する。 Co-Authored-By: Claude Opus 5.5 (1M context) --- lib/devbase/commands/container.py | 91 +++++++++++------------------- lib/devbase/commands/snapshot.py | 3 +- lib/devbase/snapshot/manager.py | 65 ++++++--------------- tests/cli/test_up_roundtrips.py | 94 ------------------------------- 4 files changed, 51 insertions(+), 202 deletions(-) diff --git a/lib/devbase/commands/container.py b/lib/devbase/commands/container.py index de3bfd96..60fb58e6 100644 --- a/lib/devbase/commands/container.py +++ b/lib/devbase/commands/container.py @@ -1949,73 +1949,46 @@ def _ensure_env_files() -> bool: logger.info("Running 'devbase env init' to create them...") success = True - - if not has_global: - if not _initialize_global_env(devbase_root, group): - success = False - - if not has_project: - if not _create_project_env(project_env): - success = False - - return success - - -def _initialize_global_env(devbase_root: Path, group: Optional[str]) -> bool: - """子プロセスの ``devbase env init`` で共通の ``.env`` を作る。 - - 子プロセスは cwd=$DEVBASE_ROOT で起動し、実行時のプロジェクトを持たない。グループを - 渡さないと $DEVBASE_ROOT/env のグループの共通の参照へ書き、親が読み直す参照と - 揃わない (PLAN56 決定 10)。layout: group でないときは渡さない。 - - 終了コードによらず、持ち回っている SecretStore の控えは解放する。書いたのは子プロセス - で控えは更新されないため、サーバ backend では最初の 404 が空として残り、そのまま起動 - すると env init が書いた共通機密が渡らない。以後は現物を読み直す (PLAN55 決定 5)。 - - Returns: - 子プロセスが正常終了 (終了コード 0) したら True。非ゼロ終了・起動時の例外は False。 - """ - from devbase.env import runtime as _runtime - - logger.info("Creating devbase root .env...") child_env = {**os.environ, 'PYTHONPATH': str(devbase_root / 'lib')} + # 子プロセスは cwd=$DEVBASE_ROOT で起動し、実行時のプロジェクトを持たない。グループを + # 渡さないと $DEVBASE_ROOT/env のグループの共通の参照へ書き、親が読み直す参照と + # 揃わない (PLAN56 決定 10)。layout: group でないときは渡さない init_argv = [sys.executable, '-m', 'devbase.cli', 'env', 'init'] if group is not None: init_argv += ['--group', group] - success = True - try: - result = subprocess.run( - init_argv, - env=child_env, - cwd=str(devbase_root), - check=False - ) - if result.returncode != 0: + if not has_global: + logger.info("Creating devbase root .env...") + try: + result = subprocess.run( + init_argv, + env=child_env, + cwd=str(devbase_root), + check=False + ) + if result.returncode != 0: + success = False + logger.error("Failed to create devbase root .env") + except Exception as e: + logger.error("Running env init for devbase root: %s", e) success = False - logger.error("Failed to create devbase root .env") - except Exception as e: - logger.error("Running env init for devbase root: %s", e) - success = False - finally: - _runtime.release_store() - return success + finally: + # 書いたのは子プロセスで、持ち回っている SecretStore の控えは更新されない。 + # サーバ backend では最初の 404 が空として残り、そのまま起動すると env init が + # 書いた共通機密が渡らない。終了コードによらず捨て、以後は現物を読み直す + # (PLAN55 決定 5)。 + _runtime.release_store() + if not has_project: + logger.info("Creating project .env...") + try: + project_env.touch() + logger.info("Created empty project .env: %s", project_env) + except Exception as e: + logger.error("Failed to create project .env: %s", e) + success = False -def _create_project_env(project_env: Path) -> bool: - """空のプロジェクト ``.env`` を作る。 - - Returns: - 作成できたら True。作成に失敗したら False。 - """ - logger.info("Creating project .env...") - try: - project_env.touch() - logger.info("Created empty project .env: %s", project_env) - except Exception as e: - logger.error("Failed to create project .env: %s", e) - return False - return True + return success _IMAGE_MAX_AGE_DAYS_DEFAULT = 7 diff --git a/lib/devbase/commands/snapshot.py b/lib/devbase/commands/snapshot.py index a00172d3..b0ce330e 100644 --- a/lib/devbase/commands/snapshot.py +++ b/lib/devbase/commands/snapshot.py @@ -6,7 +6,6 @@ from devbase.errors import SnapshotError from devbase.log import get_logger from devbase.snapshot.manager import SnapshotManager -from devbase.volume.manager import HOME_UBUNTU_VOLUME logger = get_logger(__name__) @@ -69,7 +68,7 @@ def _snapshot_list(mgr) -> int: print("-" * 90) for s in snapshots: # 対象ボリュームは PLAN39 以降に記録される。旧世代は共通ボリュームのみ。 - volumes = ', '.join((s.get('volumes') or {}).values()) or HOME_UBUNTU_VOLUME + volumes = ', '.join((s.get('volumes') or {}).values()) or 'devbase_home_ubuntu' print( f"{s['name']:<24} " f"{s.get('created_at', 'N/A')[:19]:<24} " diff --git a/lib/devbase/snapshot/manager.py b/lib/devbase/snapshot/manager.py index 4de13efa..1252942d 100644 --- a/lib/devbase/snapshot/manager.py +++ b/lib/devbase/snapshot/manager.py @@ -37,26 +37,6 @@ DEFAULT_MAX_INCREMENTALS = 10 METADATA_FILE = 'snapshot.yml' -# 差分アーカイブの命名規約 (接頭辞・接尾辞・連番書式・グロブ・正規表現) を 1 箇所へ -# 寄せる。片方だけ書式や桁を変えると作成・列挙・復元の照合が崩れるため、同じ規約に -# 由来する重複としてここでまとめて持つ。 -INCR_PREFIX = 'incr-' -ARCHIVE_SUFFIX = '.tar.zst' -FULL_ARCHIVE = f'full{ARCHIVE_SUFFIX}' -INCR_GLOB = f'{INCR_PREFIX}*{ARCHIVE_SUFFIX}' -_INCR_ARCHIVE_RE = re.compile( - r'^' + re.escape(INCR_PREFIX) + r'(\d+)' + re.escape(ARCHIVE_SUFFIX) + r'$') - - -def incr_archive_name(n: int) -> str: - """差分番号から差分アーカイブのファイル名を組み立てる (``incr-003.tar.zst``)。""" - return f'{INCR_PREFIX}{n:03d}{ARCHIVE_SUFFIX}' - - -def is_incr_archive(name: str) -> bool: - """名前が差分アーカイブの実体かどうか。""" - return bool(_INCR_ARCHIVE_RE.match(name)) - # GNU tar の incremental はディレクトリを (dev, ino) で追跡して rename を検出する。 # ディレクトリが削除され作り直されると **inode 番号が再利用される**ため、tar は無関係な # ディレクトリを rename されたものと誤判定し、dumpdir に偽の R/T レコードを書く。 @@ -285,7 +265,9 @@ def last_snapshot_time(self, volumes: Optional[dict] = None) -> Optional[datetim continue # アーカイブ実体 (full.tar.zst / incr-NNN.tar.zst) のみを対象とし、 # meta.yml / snapshot.snar / *.bak 等は除外する。 - if f.name != FULL_ARCHIVE and not is_incr_archive(f.name): + if f.name != 'full.tar.zst' and not ( + f.name.startswith('incr-') and f.name.endswith('.tar.zst') + ): continue mtime = f.stat().st_mtime if latest is None or mtime > latest: @@ -308,7 +290,7 @@ def restore(self, name: str, point: int | None = None) -> None: if not snap_dir.exists(): raise SnapshotError(f"スナップショット '{name}' が見つかりません") - full_archive = snap_dir / FULL_ARCHIVE + full_archive = snap_dir / 'full.tar.zst' if not full_archive.exists(): raise SnapshotError(f"フルバックアップが見つかりません: {full_archive}") @@ -332,17 +314,19 @@ def restore(self, name: str, point: int | None = None) -> None: # フルバックアップの復元 logger.info("フルバックアップを復元中...") self._extract_archive( - snap_dir, FULL_ARCHIVE, + snap_dir, 'full.tar.zst', self.clear_command(volumes) + - self._incremental_extract_command(FULL_ARCHIVE), + "zstd -d /backup/full.tar.zst -c | " + "tar --listed-incremental=/dev/null -xf - -C /target", volumes, pre_restore_name, skipped_renames, ) # 差分バックアップを順番に適用（pointが指定されていればそこまで） - incr_files = sorted(snap_dir.glob(INCR_GLOB)) + incr_re = re.compile(r'^incr-(\d+)\.tar\.zst$') + incr_files = sorted(snap_dir.glob('incr-*.tar.zst')) for incr in incr_files: if point is not None: - m = _INCR_ARCHIVE_RE.match(incr.name) + m = incr_re.match(incr.name) if not m: continue if int(m.group(1)) > point: @@ -350,7 +334,8 @@ def restore(self, name: str, point: int | None = None) -> None: logger.info("差分バックアップを適用中: %s", incr.name) self._extract_archive( snap_dir, incr.name, - self._incremental_extract_command(incr.name), + f"zstd -d /backup/{incr.name} -c | " + f"tar --listed-incremental=/dev/null -xf - -C /target", volumes, pre_restore_name, skipped_renames, ) @@ -739,14 +724,6 @@ def _ensure_snapshot_image(self) -> str: logger.info("devbase-snapshotイメージのビルド完了") return SNAPSHOT_IMAGE - @staticmethod - def _container_path(root: str, sub: str) -> str: - """サブディレクトリ名 ``sub`` をコンテナ内のパスへ写す。 - - 空文字は旧レイアウトで、``root`` そのものを指す。 - """ - return f'{root}/{sub}' if sub else root - @staticmethod def volume_mount_args(volumes: dict, mode: str) -> list: """対象ボリュームの ``docker run -v`` 引数を組み立てる。 @@ -758,7 +735,7 @@ def volume_mount_args(volumes: dict, mode: str) -> list: suffix = ':ro' if mode == 'backup' else '' args = [] for sub, name in volumes.items(): - target = SnapshotManager._container_path(root, sub) + target = f'{root}/{sub}' if sub else root args.extend(['-v', f'{name}:{target}{suffix}']) return args @@ -770,19 +747,13 @@ def clear_command(volumes: dict) -> str: 旧レイアウトも同じ形で扱える。 """ roots = ' '.join( - SnapshotManager._container_path('/target', sub) for sub in volumes) + f'/target/{sub}' if sub else '/target' for sub in volumes) return ( 'for d in ' + roots + '; do ' 'find "$d" -mindepth 1 -maxdepth 1 -exec rm -rf -- {} + 2>/dev/null; ' 'done; ' ) - @staticmethod - def _incremental_extract_command(archive: str) -> str: - """``/backup/`` を ``/target`` へ展開するコマンドを組み立てる。""" - return (f"zstd -d /backup/{archive} -c | " - "tar --listed-incremental=/dev/null -xf - -C /target") - def _run_docker_tar(self, snap_dir: Path, mode: str, command: str, volumes: Optional[dict] = None ) -> subprocess.CompletedProcess: @@ -827,7 +798,7 @@ def _create_full(self, name: str, snap_dir: Path) -> None: self._run_docker_tar( snap_dir, 'backup', "tar --listed-incremental=/backup/snapshot.snar " - f"-cf - -C /source . | zstd -1 -T0 -o /backup/{FULL_ARCHIVE}" + "-cf - -C /source . | zstd -1 -T0 -o /backup/full.tar.zst" ) # meta.yml を作成 @@ -836,7 +807,7 @@ def _create_full(self, name: str, snap_dir: Path) -> None: 'created_at': datetime.now().isoformat(), 'type': 'full', 'volumes': dict(self.volumes), - 'files': [FULL_ARCHIVE], + 'files': ['full.tar.zst'], 'incremental_count': 0, } self._save_snap_meta(snap_dir, meta) @@ -863,9 +834,9 @@ def _create_incremental(self, name: str, snap_dir: Path) -> None: return # 差分番号を決定 - existing = sorted(snap_dir.glob(INCR_GLOB)) + existing = sorted(snap_dir.glob('incr-*.tar.zst')) next_num = len(existing) + 1 - incr_name = incr_archive_name(next_num) + incr_name = f'incr-{next_num:03d}.tar.zst' logger.info("差分バックアップを作成中: %s/%s", name, incr_name) diff --git a/tests/cli/test_up_roundtrips.py b/tests/cli/test_up_roundtrips.py index 6ac82ee2..8f293553 100644 --- a/tests/cli/test_up_roundtrips.py +++ b/tests/cli/test_up_roundtrips.py @@ -405,97 +405,3 @@ def fake_run(argv, **kwargs): container._ensure_env_files() assert seen['argv'][-2:] == ['env', 'init'] - - -# --------------------------------------------------------------------------- -# R3-004 現状固定: `_ensure_env_files` の異常系 (extract_method の前に固定) -# --------------------------------------------------------------------------- - -def _pop_global_and_project(openbao): - """`web` の共通機密とプロジェクト機密を落とし、env init と project .env 作成を通す。""" - for path in (TEAM_GLOBAL, TEAM_WEB): - openbao.secrets.pop(path, None) - openbao.versions.pop(path, None) - - -def test_ensure_env_files_nonzero_exit_returns_false_but_creates_project_env( - up_root, openbao, monkeypatch): - """現状固定: env init が非ゼロ終了しても project .env は作られ、戻り値は False。 - - さらに終了コードによらずストアのキャッシュは解放され、次の `runtime.resolve` は - サーバから読み直す。 - """ - root = up_root['root'] - _pop_global_and_project(openbao) - project_dir = root / 'projects' / 'web' - monkeypatch.chdir(project_dir) - monkeypatch.setenv('PWD', str(project_dir)) - project_env = project_dir / '.env' - if project_env.exists(): - project_env.unlink() - - def fake_run(argv, **kwargs): - return subprocess.CompletedProcess(argv, 1) - - monkeypatch.setattr(container.subprocess, 'run', fake_run) - # 事前にストアを載せておき、解放されたことをログイン回数の増分で見る - runtime.resolve(root) - logins_before = openbao.logins - - assert container._ensure_env_files() is False - # 子プロセスが失敗しても project .env の作成は試みる - assert project_env.exists() - # 失敗後もキャッシュは解放され、resolve が新たに認証する - runtime.resolve(root) - assert openbao.logins > logins_before - - -def test_ensure_env_files_env_init_launch_exception_returns_false( - up_root, openbao, monkeypatch): - """現状固定: env init の起動そのものが例外でも、握りつぶして False を返す。""" - root = up_root['root'] - _pop_global_and_project(openbao) - project_dir = root / 'projects' / 'web' - monkeypatch.chdir(project_dir) - monkeypatch.setenv('PWD', str(project_dir)) - project_env = project_dir / '.env' - if project_env.exists(): - project_env.unlink() - - def boom(argv, **kwargs): - raise OSError('cannot launch') - - monkeypatch.setattr(container.subprocess, 'run', boom) - - assert container._ensure_env_files() is False - # 例外でも後続の project .env 作成は続く - assert project_env.exists() - - -def test_ensure_env_files_project_env_touch_failure_returns_false( - up_root, openbao, monkeypatch): - """現状固定: project .env の作成 (touch) が失敗すると False を返す。""" - root = up_root['root'] - _pop_global_and_project(openbao) - project_dir = root / 'projects' / 'web' - monkeypatch.chdir(project_dir) - monkeypatch.setenv('PWD', str(project_dir)) - project_env = project_dir / '.env' - if project_env.exists(): - project_env.unlink() - - def ok_run(argv, **kwargs): - return subprocess.CompletedProcess(argv, 0) - - monkeypatch.setattr(container.subprocess, 'run', ok_run) - - real_touch = Path.touch - - def failing_touch(self, *a, **k): - if self.name == '.env': - raise PermissionError('read-only') - return real_touch(self, *a, **k) - - monkeypatch.setattr(Path, 'touch', failing_touch) - - assert container._ensure_env_files() is False From b58f9ee0ab996658d51c456f599e8b721e9523e7 Mon Sep 17 00:00:00 2001 From: "takemi.ohama" Date: Thu, 24 Sep 2026 13:34:18 +0900 Subject: [PATCH 12/14] =?UTF-8?q?fix(snapshot):=20=E5=BC=95=E7=94=A8?= =?UTF-8?q?=E7=AC=A6=E3=81=AA=E3=81=97=E3=81=AE=20created=5Fat=20=E3=82=92?= =?UTF-8?q?=E6=96=87=E5=AD=97=E5=88=97=E3=81=AB=E6=8F=83=E3=81=88=E3=81=A6?= =?UTF-8?q?=E6=AF=94=E3=81=B9=E3=82=8B=20(PLAN68)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit snapshot.yml を手で書いて created_at に引用符が無いと YAML が datetime で返し、 文字列のエントリと比べて series_latest / rotate が TypeError で落ちていた。 _entry_age で isoformat の文字列に揃える。 Co-Authored-By: Claude Opus 5.5 (1M context) --- lib/devbase/snapshot/manager.py | 10 +++++++--- tests/snapshot/test_manager_series.py | 27 +++++++++++++++++++++++++++ 2 files changed, 34 insertions(+), 3 deletions(-) diff --git a/lib/devbase/snapshot/manager.py b/lib/devbase/snapshot/manager.py index 1252942d..cefc276f 100644 --- a/lib/devbase/snapshot/manager.py +++ b/lib/devbase/snapshot/manager.py @@ -4,7 +4,7 @@ import shlex import shutil import subprocess -from datetime import datetime, timezone +from datetime import date, datetime, timezone from pathlib import Path from typing import Optional @@ -592,8 +592,12 @@ def age(index: int) -> tuple: @staticmethod def _entry_age(entry: dict, index: int) -> tuple: - # created_at が同じなら snapshot.yml で前にあるものを古いとみなす - return (entry.get('created_at', '') or '', index) + # created_at が同じなら snapshot.yml で前にあるものを古いとみなす。 + # 引用符なしの日時は YAML が datetime で返すため、文字列に揃えて比べる + created = entry.get('created_at') or '' + if isinstance(created, date): # datetime も date の派生 + created = created.isoformat() + return (str(created), index) @staticmethod def _entry_volumes(entry: dict) -> dict: diff --git a/tests/snapshot/test_manager_series.py b/tests/snapshot/test_manager_series.py index b13ff13a..c7c5b37d 100644 --- a/tests/snapshot/test_manager_series.py +++ b/tests/snapshot/test_manager_series.py @@ -12,6 +12,7 @@ import re import time import types +from datetime import datetime from pathlib import Path import pytest @@ -161,6 +162,32 @@ def test_series_latest_uses_created_at(tmp_path): assert mgr.series_latest(vols("kkg")) is None +def _unquote_created_at(backups: Path, *targets: str) -> None: + """指定した世代の ``created_at`` を引用符なしの YAML timestamp で書き直す。""" + path = backups / "snapshot.yml" + data = yaml.safe_load(path.read_text()) + for snap in data["snapshots"]: + if snap["name"] in targets: + snap["created_at"] = datetime.fromisoformat(snap["created_at"]) + path.write_text(yaml.safe_dump(data)) + + +def test_series_latest_accepts_yaml_timestamp(tmp_path): + """手で書いた snapshot.yml の引用符なしの日時が混ざっても比べられる。""" + backups = write_state(tmp_path, [("D1", "default", 0), ("D2", "default", 0)]) + _unquote_created_at(backups, "D1") + assert RecordingManager(tmp_path, group="default").series_latest()["name"] == "D2" + + +def test_rotate_accepts_yaml_timestamp(tmp_path): + backups = write_state(tmp_path, [ + ("D1", "default", 0), ("D2", "default", 0), ("D3", "default", 0), + ("D4", "default", 0)]) + _unquote_created_at(backups, "D2", "D4") + assert SnapshotManager(tmp_path).rotate() == 1 + assert names(tmp_path) == ["D2", "D3", "D4"] + + class ArchiveRecordingManager(RecordingManager): """書き込むアーカイブ名を command から拾う (full / incr-NNN を区別する)。""" From f557461c878cd29987b1ff65bb881a9f710e5fac Mon Sep 17 00:00:00 2001 From: "takemi.ohama" Date: Thu, 24 Sep 2026 13:45:38 +0900 Subject: [PATCH 13/14] =?UTF-8?q?fix(snapshot):=20=E7=B3=BB=E5=88=97?= =?UTF-8?q?=E3=81=AE=E6=9C=80=E6=96=B0=E3=81=8C=E6=89=B1=E3=81=88=E3=81=AA?= =?UTF-8?q?=E3=81=84=E4=B8=96=E4=BB=A3=E3=81=AA=E3=82=89=E6=96=B0=E3=81=97?= =?UTF-8?q?=E3=81=84=E4=B8=96=E4=BB=A3=E3=82=92=E4=BD=9C=E3=82=8B=20(PLAN6?= =?UTF-8?q?8)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 系列の最新のエントリがシンボリックリンクか不正な名前 (../outside など) のとき、 auto_snapshot_target がその名前を返し create が _safe_snap_dir で落ちていた。 rotate は系列の最新を消さないため、devbase up のたびに同じ失敗を繰り返す。 積み先を返す前に _safe_snap_dir を通し、拒否されたら WARNING を出して None を返す。 Co-Authored-By: Claude Opus 5.5 (1M context) --- lib/devbase/snapshot/manager.py | 16 ++++++++--- tests/snapshot/test_manager_series.py | 40 +++++++++++++++++++++++++++ 2 files changed, 52 insertions(+), 4 deletions(-) diff --git a/lib/devbase/snapshot/manager.py b/lib/devbase/snapshot/manager.py index cefc276f..a2da809e 100644 --- a/lib/devbase/snapshot/manager.py +++ b/lib/devbase/snapshot/manager.py @@ -659,7 +659,8 @@ def auto_snapshot_target( Returns: 系列の最新の世代の名前。新しい世代を作るべきなら ``None`` - (その理由を INFO で 1 行出す)。 + (その理由を INFO で 1 行出す。最新の世代がシンボリックリンクか + 不正な名前なら WARNING)。 """ label = self.series_label(self.volumes) latest = self.series_latest() @@ -668,9 +669,16 @@ def auto_snapshot_target( return None name = latest['name'] - snap_dir = self.backups_dir / name - if (is_single_segment_name(name) and not snap_dir.is_symlink() - and snap_dir.is_dir()): + try: + # create と同じ検証を先に通す。通らない世代へ積もうとすると、rotate が + # 系列の最新を消さないため、起動のたびに同じ失敗を繰り返す (決定 7) + snap_dir = self._safe_snap_dir(name) + except SnapshotError as e: + logger.warning( + "%s の最新の世代 '%s' は扱えないため、新しい世代を作成します: %s", + label, name, e) + return None + if snap_dir.is_dir(): recorded = self.snapshot_volumes(snap_dir) if recorded != self.volumes: logger.info( diff --git a/tests/snapshot/test_manager_series.py b/tests/snapshot/test_manager_series.py index c7c5b37d..c7e0fcb7 100644 --- a/tests/snapshot/test_manager_series.py +++ b/tests/snapshot/test_manager_series.py @@ -499,3 +499,43 @@ def test_other_operations_refuse_symlink(tmp_path, op): assert mgr.calls == [] assert not (tmp_path / "backups" / "new").exists() assert sorted(p.name for p in outside.iterdir()) == ["keep.txt"] + + +def _bad_latest(tmp_path: Path, kind: str) -> tuple: + """系列 default の最新がリンクか ``../outside`` の状態を作る。(名前, 外の場所) を返す。""" + if kind == "symlink": + outside = _link_outside(tmp_path) + name = "old" + else: + outside = tmp_path / "outside" + outside.mkdir() + (outside / "keep.txt").write_text("keep") + name = "../outside" + write_state(tmp_path, [("D1", "default", 0), (name, "default", 0)]) + return name, outside + + +@pytest.mark.parametrize("kind", ["symlink", "traversal"]) +def test_bad_latest_starts_a_new_generation(tmp_path, caplog, kind): + """決定 7: 系列の最新が扱えない世代なら積まず、理由を WARNING で 1 行出す。""" + name, _ = _bad_latest(tmp_path, kind) + mgr = RecordingManager(tmp_path, group="default") + + with caplog.at_level(logging.INFO, logger="devbase"): + assert mgr.auto_snapshot_target() is None + warnings = [r for r in caplog.records if r.levelno == logging.WARNING] + assert len(warnings) == 1 and f"'{name}'" in warnings[0].getMessage() + + +@pytest.mark.parametrize("kind", ["symlink", "traversal"]) +def test_bad_latest_is_left_behind_and_rotated_out(tmp_path, kind): + """新しい世代を作れば、扱えない世代は最新でなくなり、rotate が一覧から外す。""" + name, outside = _bad_latest(tmp_path, kind) + mgr = RecordingManager(tmp_path, group="default") + + created = mgr.create(name=mgr.auto_snapshot_target()) + assert mgr.auto_snapshot_target() == created + + mgr.rotate(keep=1) + assert names(tmp_path) == [created] + assert sorted(p.name for p in outside.iterdir()) == ["keep.txt"] From 2b0685615d8666a1e9175f32cdc51ca065871afc Mon Sep 17 00:00:00 2001 From: "takemi.ohama" Date: Thu, 24 Sep 2026 18:57:49 +0900 Subject: [PATCH 14/14] =?UTF-8?q?docs(PLAN68):=20=E3=82=B9=E3=83=8A?= =?UTF-8?q?=E3=83=83=E3=83=97=E3=82=B7=E3=83=A7=E3=83=83=E3=83=88=E3=81=AE?= =?UTF-8?q?=E7=B3=BB=E5=88=97=E3=81=AE=E7=A2=BA=E5=AE=9A=E4=BB=95=E6=A7=98?= =?UTF-8?q?=E3=82=92=20docs/specifications=20=E3=81=B8=E7=A7=BB=E3=81=99?= =?UTF-8?q?=20(#248)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit PLAN68 の要求と設計を issues/old/ へ移し、系列の識別・差分の積み先・ 系列ごとの最小間隔・保持と全体の上限・世代の場所の検証・ログの文言・ テスト観点を snapshot-series.md として確定仕様にした。クラス図は実装に 合わせて直した。スナップショットガイドの「系列」から新しい仕様へリンクする。 Co-Authored-By: Claude Opus 5.5 (1M context) --- docs/specifications/snapshot-series.md | 346 ++++++++++++++++++ docs/user/snapshot-guide.md | 3 + .../PLAN68_snapshot-series-design.md | 0 issues/{ => old}/PLAN68_snapshot-series.md | 0 4 files changed, 349 insertions(+) create mode 100644 docs/specifications/snapshot-series.md rename issues/{ => old}/PLAN68_snapshot-series-design.md (100%) rename issues/{ => old}/PLAN68_snapshot-series.md (100%) diff --git a/docs/specifications/snapshot-series.md b/docs/specifications/snapshot-series.md new file mode 100644 index 00000000..cdfd6381 --- /dev/null +++ b/docs/specifications/snapshot-series.md @@ -0,0 +1,346 @@ +# スナップショットの世代の系列（アカウントグループごとの保持と差分の積み先） + +## 概要 + +スナップショットの世代は、控える**対象ボリュームの組**ごとの**系列**に分かれる。組は共通 +ボリューム `devbase_home_ubuntu` とグループのボリューム `devbase_home_` からなり、 +共通ボリュームは固定のため、系列はアカウントグループと 1 対 1 に対応する。 + +- `devbase up` の自動スナップショットは、起動するグループの系列の**最新の世代**へ差分を積む。 + 別のグループを起動してから戻っても、`full.tar.zst` を取り直さない +- 自動スナップショットの最小間隔（`DEVBASE_SNAPSHOT_MIN_INTERVAL_MINUTES`）は系列ごとに判定する +- ローテーションは系列ごとに `keep` 世代を残し、全体の上限（既定 `keep × 3`）を超えた分を系列を + またいで古い順に消す。**各系列の最新の世代は消さない** +- ローテーションは消す前に世代の場所を検証し、`backups/` の外やシンボリックリンクの先を消さない + +系列はどこにも保存しない。`snapshot.yml` のエントリの `volumes` からその場で導く。 + +利用者向けの読み方は [スナップショットガイド: 系列](../user/snapshot-guide.md#系列) と +[CLI リファレンス: `devbase snapshot rotate`](../user/cli-reference/05-snapshot.md#devbase-snapshot-rotate) +にある。 + +## 対象範囲 + +- 自動スナップショットの積み先・新しい世代を作る条件・最小間隔の判定 +- `devbase up` / `devbase down` / `devbase snapshot rotate` / TUI のローテーションの規則 +- 世代の場所の検証（`_safe_snap_dir`）。`create` / `restore` / `copy` / `delete` / `rotate` が共有する +- ログの文言 + +含まない: + +- 世代を名前で区別すること。`--name` の世代・`copy` の世代・`pre-restore-*` も対象ボリュームの組で + 系列に入り、同じ規則で消える。名前付きの世代の保護は #256 で扱う +- 復元の対象・順序・失敗時の案内（変えていない）。復元前の自動バックアップが控えるボリュームは #255 +- `devbase snapshot list` / `devbase status` の表示。`status` の「最新」は `snapshot.yml` の最後の + エントリ（最も新しく**作られた**世代）で、直前に差分を積んだ世代とは一致しないことがある +- 保持をバイト数で制限すること +- リモート扱いの `up`（自動スナップショットを作らない。[別ホストの Docker への dev コンテナ起動](remote-docker-context.md#自動スナップショット)） + +## 用語 + +| 用語 | 意味 | +| --- | --- | +| 世代 | `backups/<名前>/` の 1 つ。`full.tar.zst` 1 つと、0 個以上の `incr-NNN.tar.zst` からなる | +| 対象ボリュームの組 | `snapshot.yml` のエントリの `volumes`（マウント名 → ボリューム名）。現行は `{ai: devbase_home_ubuntu, group: devbase_home_}` | +| 系列 | 対象ボリュームの組が同じ世代の集まり。`volumes` が無い・空・dict でないエントリ（PLAN39 より前の旧レイアウト）は `{'': devbase_home_ubuntu}` の組として 1 つの系列になる | +| 系列の最新の世代 | 系列の中で `created_at` が最も新しい世代。同じ値なら `snapshot.yml` で後ろのもの | +| 全体の上限 | 系列をまたいで数えた世代の数の上限（`max_total`） | + +## 構成要素 + +| 要素 | 置き場所 | 責務 | +| --- | --- | --- | +| 系列の解決 | `lib/devbase/snapshot/manager.py` の `SnapshotManager.series_key` / `series_label` / `_entry_volumes` / `_series_entries` / `series_latest` | エントリを系列に分け、系列の最新の世代を返す。系列の表示名を作る | +| 積み先の判定 | 同 `auto_snapshot_target` | 積み先の世代の名前か、新しい世代を作るべきこと（`None`）を返し、理由をログへ出す | +| 最小間隔の元 | 同 `last_snapshot_time(volumes=None)` | 渡された組の系列の世代のアーカイブの mtime の最大を返す | +| 保持 | 同 `rotate(keep, max_total)` と、副作用の無い `_rotation_plan` | 削除候補を決め、場所を検証してから消す | +| 世代の場所の検証 | 同 `_safe_snap_dir` | 名前・シンボリックリンク・`backups/` の外を `SnapshotError` で止める | +| 自動スナップショット | `lib/devbase/commands/container.py` の `_auto_snapshot` / `_should_skip_by_interval` | 最小間隔を系列で判定し、判定の結果に従って `create` と `rotate()` を呼ぶ | +| 自動ローテーション | 同 `cmd_down` | `mgr.rotate()` を既定の引数で呼ぶ | +| 手動ローテーション | `lib/devbase/cli.py`（`--keep` / `--max-total`）、`lib/devbase/commands/snapshot.py` の `cmd_snapshot` / `_snapshot_rotate` | 引数を `rotate` へ渡す。`max_total` は `getattr(args, 'max_total', None)` で受ける | +| TUI | `lib/devbase/tui/actions_snapshot.py` の `_op_rotate` | 「グループごとに保持する世代数 (--keep)」だけを問い、`max_total` は渡さない | + +世代の規則は `manager.py` に置き、`container.py` は判定の結果に従って作成を呼ぶだけにする。 + +```mermaid +classDiagram + class SnapshotManager { + +volumes: dict + +series_key(volumes)$ tuple + +series_label(volumes)$ str + +series_latest(volumes) dict|None + +auto_snapshot_target(max_incrementals) str|None + +last_snapshot_time(volumes) datetime|None + +rotate(keep, max_total) int + -_rotation_plan(snapshots, keep, max_total) list + -_safe_snap_dir(name) Path + -_series_entries(volumes) list + -_entry_volumes(entry)$ dict + -_entry_age(entry, index)$ tuple + } + class 世代のエントリ { + +name + +created_at + +updated_at + +incremental_count + +volumes + } + SnapshotManager ..> 世代のエントリ : snapshot.yml から読む +``` + +`系列` は型として持たない。`series_key(volumes)` が返すタプルを辞書のキーにして、その場で +エントリを束ねる。 + +系列の判定とローテーションはホストの Python の中だけで完結し、Docker を呼ばない。Docker を +呼ぶのは既存の `create` のアーカイブ作成だけで、この規則は呼ぶ回数（full か差分か）を決める。 + +## 仕様 + +### 系列の識別 + +- `series_key(volumes)` は `tuple(sorted((str(k), str(v)) ...))`。値は検証しない。キーとして比べる + だけで、マウントには使わないためである。マウントに使う値は従来どおり `snapshot_volumes` が + `meta.yml` から検証して返す +- 判定には `meta.yml` ではなく `snapshot.yml` を使う。ローテーションが全世代の `meta.yml` を読まずに + 済み、壊れた `meta.yml` の検証エラーでローテーション全体が止まらない +- `series_label(volumes)` は `group` があれば `グループ <名前>`（`devbase_home_` を外した名前）、 + 無ければ `旧レイアウト（共通ボリュームのみ）` +- 世代の新旧は `(created_at, snapshot.yml での位置)` で比べる（`_entry_age`）。`created_at` が無い + エントリは空文字として最も古い。引用符なしの日時は YAML が `datetime` で返すため、`isoformat()` の + 文字列に揃えて比べる + +**系列を保存しない理由**: エントリは PLAN39 以降すべて `volumes` を持ち、系列はそこから一意に決まる。 +別に保存すると `volumes` と食い違ったときの規則と既存の `snapshot.yml` の移行が要る。導けば移行が +無く、変更前の devbase へ戻しても同じ `snapshot.yml` を読める。 + +### 差分の積み先 + +`auto_snapshot_target(max_incrementals=10)` は次の順に判定する。対象は `self.volumes`（起動する +グループの組）の系列である。 + +| 順 | 条件 | 結果 | ログ | +| --- | --- | --- | --- | +| 1 | 系列に世代が無い | `None` | INFO `{系列} の世代がまだ無いため、新しい世代を作成します` | +| 2 | 系列の最新の世代が `_safe_snap_dir` を通らない（不正な名前・シンボリックリンク・`backups/` の外） | `None` | WARNING `{系列} の最新の世代 '{名前}' は扱えないため、新しい世代を作成します: {理由}` | +| 3 | 最新の世代のディレクトリがあり、その `meta.yml` の組が `self.volumes` と違う | `None` | INFO `世代 {名前} の meta.yml の対象ボリューム ({組}) が {系列} と一致しないため、新しい世代を作成します` | +| 4 | 最新の世代の `incremental_count` が上限以上 | `None` | INFO `世代 {名前}（{系列}）の差分が上限 ({上限}) に達したため、新しい世代を作成します` | +| 5 | それ以外 | 最新の世代の名前 | 出さない | + +- 順 2 は、通らない世代へ積もうとすると `create` が止まり、`rotate` は系列の最新を消さないため、 + 起動のたびに同じ失敗を繰り返すことを防ぐ。新しい世代を作れば、扱えない世代は最新でなくなり、 + 次のローテーションで一覧から外れる +- ディレクトリが無い世代の名前を返したときは、`create(name=...)` が新しいディレクトリとして + full を作る +- 名前を明示して組の違う世代へ差分を作ろうとすると、`_create_incremental` が理由を示して + `SnapshotError` で止める（旧レイアウトの世代へは差分を積まない） + +**間に別グループの起動を挟んでも、系列の最新の世代へ積む。** `snapshot.snar` は世代ごとにあり、 +系列が同じならアーカイブのレイアウト（`/source/ai` と `/source/group`）もマウントするボリュームも +同じで、snar の記録と今のツリーのパスが対応する。間に変わった共通ボリュームの中身は「前回の差分 +からの変更」として次の差分に入るだけである。代償として、共通ボリュームの変化は系列ごとに別々に +控えられる。 + +### 最小間隔 + +`_auto_snapshot` は `last_snapshot_time(mgr.volumes)` で、起動するグループの系列の世代だけを見る。 + +- 系列の世代（`snapshot.yml` のエントリ）のうち、名前が単一の要素で、シンボリックリンクでない + 実ディレクトリだけを走査する。`volumes` を省くと `backups/` の全ディレクトリを走査する +- 数えるのはアーカイブ（`full.tar.zst` / `incr-*.tar.zst`）の mtime だけで、`meta.yml` / + `snapshot.snar` / `*.bak` は数えない。作成に失敗しても残りうるためである +- `間隔 > 0` かつ `0 ≤ 経過 < 間隔` なら飛ばす。経過が負（mtime が未来）なら飛ばさない。 + `DEVBASE_SNAPSHOT_MIN_INTERVAL_MINUTES=0` はどの系列でも飛ばさない + +**系列ごとにする理由**: 全体で判定すると、default を控えた直後に with を起動したとき、with の +系列は何時間も控えていなくても飛ばされる。系列ごとにして控える回数が増えても、2 回目以降は差分になる。 + +### 保持 + +`rotate(keep=3, max_total=None)`: + +1. `max_total` を省けば `keep × 3`。`keep < 1` か `max_total < 1` なら `SnapshotError`（何も消さない） +2. エントリを `series_key` で束ね、各系列を古い順に並べ、古い側から `len - keep` 件を削除候補にする + （理由: 系列ごとの保持） +3. 残りの総数が `max_total` を超える間、残りが 2 件以上の系列の最古の世代のうち最も古い 1 件を + 削除候補にする（理由: 全体の上限）。そうした系列が無ければ警告して打ち切る +4. 候補が無ければ 0 を返し、`snapshot.yml` を書かない +5. 候補ごとに `_safe_snap_dir` で検証し、通ればディレクトリを消す。通らなければディレクトリを + 消さず、一覧からだけ外して警告する +6. 残りを古い順に並べて `snapshot.yml` へ保存し、`max_generations` に `keep` を書く。 + 検証を通って消した（ディレクトリが既に無かったものを含む）数を返す。一覧から外しただけのエントリは数えない + +常に成り立つこと: + +- **各系列の最新の世代は `rotate` では消えない。** 次の差分の積み先であり、消すとそのグループの次の + 起動で full を取り直すことになる。使わなくなったグループや旧レイアウトの系列の最新の世代も残り、 + 不要なら `devbase snapshot delete` で消す +- ディスクに載る世代の数は、`rotate` の後に `max(max_total, 系列の数)` を超えない +- 旧レイアウトの系列は 1 つの系列として数え、他の系列の保持に影響しない + +**全体の上限を持つ理由**: 系列ごとの保持だけでは、使わなくなった系列が 3 世代ずつ残り続け、 +グループの数だけディスクの使用量が増える。既定を `keep` と独立した定数にしないのは、`--keep` を +増やした利用者の上限が黙って指定より少なくなるためである。バイト数で持たないのは、1 世代の大きさが +差分の数で 1 桁以上振れ、何世代残るかを予測できないためである。 + +`devbase up`（作成の後）と `devbase down` は `rotate()` を既定の引数（系列ごと 3・全体 9）で呼ぶ。 +手動の `--keep` / `--max-total` は保存せず、その 1 回だけに効く。4 グループ以上の端末では、 +自動のローテーションで各系列の世代が 3 未満になりうる。自動のローテーションの上限を変える手段は +持たない。 + +### `devbase snapshot rotate` + +| 項目 | 内容 | +| --- | --- | +| 形 | `devbase snapshot rotate [--keep N] [--max-total M]` | +| `--keep N` | 系列（グループ）ごとに残す数。既定 3 | +| `--max-total M` | 全体の上限。既定 `N × 3`。各グループの最新の世代は上限を超えても残す | +| 失敗 | `N < 1` / `M < 1` は `SnapshotError` → `cmd_snapshot` がエラーを出して終了コード 1 | +| TUI | `keep` だけを問い、`max_total` を渡さない。`cmd_snapshot` は `getattr` で受け、既定の `keep × 3` で動く | + +**`--keep` の意味を「全体で残す数」から「系列ごとに残す数」へ変えた。** 自動のローテーション +（系列ごと）と同じ語を同じ意味にするためである。同じ値で残る世代の**数**は変更前より減らないが、 +**どの世代が残るか**は変わりうる（全体の上限は、世代の多い系列の古い世代を、他の系列の最新の世代より +先に消す）。 + +### 世代の場所の検証 + +`_safe_snap_dir(name)` は次の順に判定し、通らなければ `SnapshotError` にする。 + +1. 名前が `is_single_segment_name` を満たさない（`../outside` など） +2. `backups/<名前>` がシンボリックリンク（リンク先が `backups/` の外・兄弟の `backups-outside/`・ + `backups/` の中の別の世代のどれでも） +3. 解決後のパスが `Path.is_relative_to(backups_dir.resolve())` でない + +- 包含は文字列の前方一致ではなく**パスの要素の単位**で比べる。前方一致では兄弟の + `backups-outside/` が `backups` で始まるため通ってしまう +- devbase はリンクの世代を作らない。リンクを通すと、リンク先がどこでも `delete` が実体を消しうる +- `backups/` 自体をリンクにした構成は、解決後の `backups/` と比べるため使える +- `create` / `restore` / `copy` / `delete` はこの検証で止まり、ボリュームへの書き込みもディレクトリの + 作成・削除も起こさない。`rotate` だけは止まらず、そのエントリを一覧から外して警告する。不正な + エントリ 1 つで `devbase down` のたびにローテーション全体が止まるのを避けるためである + +## データ・設定 + +**形は変えていない。** 移行は無く、変更前の devbase も同じファイルを読める。 + +| 保存先 | 系列の規則での扱い | +| --- | --- | +| `backups/snapshot.yml` の `snapshots[].volumes` | 系列の識別子の元 | +| 同 `snapshots[].created_at` | 世代の新旧。差分を積んでも変わらない（差分は `updated_at` と `incremental_count` を更新する） | +| 同 `snapshots[].incremental_count` | 差分の上限の判定 | +| 同 `max_generations` | `rotate` が消したときに `keep` を書く。読む側は無い | +| 各世代の `meta.yml` の `volumes` | 系列の判定には使わない。積む直前の検証（`auto_snapshot_target` の順 3 と `_create_incremental`）でだけ読む | +| アーカイブの mtime | 最小間隔の判定 | + +| 設定 | 値 | +| --- | --- | +| `DEFAULT_MAX_GENERATIONS` | 3（系列ごとの保持数の既定） | +| 全体の上限の既定 | `keep × 3`（既定 9） | +| `DEFAULT_MAX_INCREMENTALS` | 10（1 世代の差分の上限） | +| `DEVBASE_SNAPSHOT_MIN_INTERVAL_MINUTES` | 既定 60。0 で無効。不正な値は既定へ戻す | + +### ログの文言 + +| 場面 | 水準 | 文言 | +| --- | --- | --- | +| 最小間隔で飛ばす | INFO | `[0/6] {系列} の直近のスナップショット ({時刻}) から{分}分以内のためスキップします` | +| 新しい世代を作る | INFO | `[0/6] 新しいスナップショット世代を作成中 ({系列})...`（直前に積み先の判定の理由の 1 行） | +| 差分を積む | INFO | `[0/6] スナップショットを差分更新中: {名前} ({系列})` | +| 系列ごとの保持で消した | INFO | `ローテーション: {系列} の {数} 世代を削除しました（グループごとに {keep} 世代保持）` | +| 全体の上限で消した | INFO | `ローテーション: 全体の上限 {max_total} 世代を超えたため、{系列} の {名前} を削除しました` | +| 全体の上限を満たせない | WARNING | `全体の上限 {max_total} 世代を超えていますが、各グループの最新の世代は消さないため {残り} 世代を残します` | +| 場所が不正なエントリ | WARNING | `snapshot.yml の世代 '{名前}' は場所が不正なため、ディレクトリを消さずに一覧からだけ外します: {理由}` | + +`{系列}` は `series_label` の値（例: `グループ default`）。グループの切替は新しい世代を作る理由では +ないため、切替を理由とする行は無い。文言に直前の世代を作ったプロジェクトは添えない。世代は +プロジェクトではなくボリュームの組に属し、同じグループの複数のプロジェクトが 1 つの世代へ積む。 + +## エラー処理 + +| 場面 | 扱い | +| --- | --- | +| `_auto_snapshot` の中の例外（グループ名の不正による `DevbaseError`、`SnapshotError` など） | 警告 1 行に変えて起動を続ける。`snapshot.yml` を作らない | +| `cmd_down` の `rotate()` の失敗 | 警告 1 行。コンテナの停止は済んでおり、終了コード 0 | +| `rotate` の `keep` / `max_total` が 1 未満 | `SnapshotError`。何も消さず `snapshot.yml` も書かない | +| `rotate` の候補が `_safe_snap_dir` を通らない | ディレクトリを消さず、一覧から外して警告。他の候補の削除は続ける | +| `create` / `restore` / `copy` / `delete` の世代が `_safe_snap_dir` を通らない | `SnapshotError` → CLI は終了コード 1 | + +## 運用 + +- 次の `devbase up` から新しい規則で動く。利用者の操作は要らない +- 使わなくなったグループや旧レイアウトの系列の最新の世代は自動では消えない。 + `devbase snapshot list` の「対象ボリューム」とサイズの列で見て、`devbase snapshot delete` で消す +- 長く残したい世代は `backups/` の外へ複製する。`copy` の世代も系列に入り、ローテーションの対象になる +- 切り戻し（この変更の前の devbase へ戻す）では、最初の `devbase down` で全体 3 世代の旧規則の + ローテーションが走り、系列ごとに残っていた世代が消える。`snapshot copy` の退避では守れないため、 + `backups/` の外へ複製してから戻す +- 別グループの起動を挟んで積んだ差分を実際の `devbase-snapshot` で復元し、最後の差分の時点の + 共通ボリュームに戻ることは、自動テストでは確かめていない（tar は差し替えている） + +## テスト観点 + +自動テストは `DEVBASE_ROOT` を `tmp_path` へ向け、`SnapshotManager._run_docker_tar` を差し替えて +Docker を起動しない。`snapshot.yml` を直接書くテストは、世代のディレクトリと `meta.yml` も書く。 + +積み先（`tests/snapshot/test_manager_series.py`、`tests/snapshot/test_manager_volumes.py`）: + +- default の世代の後に with の世代を作っても、default の積み先は default の最新の世代であること +- 系列に世代が無いグループは `None` と理由の行。差分の上限は系列ごとに数え、他の系列の差分数を + 使わないこと +- 旧レイアウトの世代だけなら `None`。組の違う世代を名前で指定した差分は `SnapshotError` +- `meta.yml` の組が食い違う最新の世代では `None`。`created_at` の新旧（引用符なしの YAML の日時を含む） +- 系列の最新がシンボリックリンクか `../outside` のとき、`None` と WARNING 1 行。新しい世代を作った後は + それが積み先になり、`rotate` で扱えないエントリが一覧から外れ、外の中身が残ること + +最小間隔と `_auto_snapshot` の流れ（`tests/snapshot/test_auto_snapshot_series.py`）: + +- グループを行き来した後の起動で `create(name=..., full=False)` が呼ばれ、世代の数が変わらないこと。 + 出力に切替を理由とする行が無く、`グループ default` を含むこと +- 10 分前の系列は飛ばし、2 時間前の系列は積むこと。間隔 0 なら飛ばさないこと。mtime が未来なら + 飛ばさないこと +- 新しい世代の作成の後の `rotate()` が、その系列の最古だけを消し、他の系列を残すこと +- 不正なグループ名では警告 1 行で、`snapshot.yml` を作らないこと + +保持（`tests/snapshot/test_manager_series.py`）: + +- default 4・with 1 で default の最古 1 世代だけが消え、戻り値が 1 +- default と with を交互に 4 つずつで、各 3 世代が残ること +- 4 系列 × 3 世代（A1〜D3 の順）で A1・B1・C1 が消えること。A の 3 世代が最古のときは A の古い 2 と + B の最古が消え、A の最新が残ること +- 10 系列 × 1 世代で `rotate(keep=3, max_total=9)` が何も消さず、WARNING が 1 件 +- 旧レイアウト 3 世代と default 3 世代で何も消さないこと +- `keep=0` / `max_total=0` が `SnapshotError` で何も消さないこと +- 既存と同じ 3 エントリ（default 2・with 1）の `snapshot.yml` で 0 を返し、ファイルのバイト列が + 変わらないこと +- 系列ごとの削除と全体の上限の削除のログにグループ名があること + +場所の検証（`tests/snapshot/test_manager_series.py`）: + +- `../outside` のエントリが削除の対象でも `backups/` の外が残り、エントリが外れ、WARNING 1 件 +- 兄弟の `backups-outside/` を指すリンク、`backups/` の中の最新の世代を指すリンクのどちらでも、 + リンク先の中身が残り、エントリが外れ、WARNING 1 件。`_safe_snap_dir('old')` が `SnapshotError` +- 同じリンクで `cmd_snapshot` の `delete` が終了コード 1 でリンク先が残ること。`restore` / `copy` / + `create(name=...)` が `SnapshotError` で、tar を呼ばず `backups/new` も作らないこと + +CLI と TUI: + +- `--keep` / `--max-total` / 別名の parse（`tests/cli/test_snapshot_rotate_args.py`） +- `cmd_snapshot` に `keep=2, max_total=2` を渡すと上限が効くこと。`max_total` を省くと上限が + `keep × 3` になること。0 以下で終了コード 1（`tests/snapshot/test_manager_series.py`） +- TUI の問いの文言と、`max_total` を渡さないこと（`tests/cli/tui/test_actions_snapshot.py`） +- `cmd_down` の `rotate()` の失敗で終了コード 0（`tests/commands/test_container_down_snapshot.py`） + +復元のテスト（`tests/snapshot/test_restore_incremental.py`、`test_manager_volumes.py` の復元）は +変えずに通ること。 + +実機で確かめる観点: 2 つのグループのプロジェクトを `DEVBASE_SNAPSHOT_MIN_INTERVAL_MINUTES=0` で +交互に `devbase up` し、それぞれの系列の最新の世代に `incr-001` が積まれ、世代の数が変わらないことを +`devbase snapshot list` で見る。 + +## 関連リンク + +- [スナップショットガイド](../user/snapshot-guide.md) +- [CLI リファレンス: snapshot (ss) グループ](../user/cli-reference/05-snapshot.md) +- [コンテナ運用ガイド](../user/container-operations.md) +- [別ホストの Docker への dev コンテナ起動（docker context）](remote-docker-context.md) +- [位置引数の解決（プロジェクト名・イメージ名）](cli-argument-resolution.md)（スナップショットの名前の規則） diff --git a/docs/user/snapshot-guide.md b/docs/user/snapshot-guide.md index be4944e5..e0a507e1 100644 --- a/docs/user/snapshot-guide.md +++ b/docs/user/snapshot-guide.md @@ -78,6 +78,9 @@ graph LR `devbase snapshot delete <名前>` で削除してください。系列の数が全体の上限を超えているときは、 上限を超えたまま残し、警告を出します +積み先・最小間隔・ローテーションの規則と世代の場所の検証の仕様は +[スナップショットの世代の系列](../specifications/snapshot-series.md) にあります。 + ### 世代の概念 ```mermaid diff --git a/issues/PLAN68_snapshot-series-design.md b/issues/old/PLAN68_snapshot-series-design.md similarity index 100% rename from issues/PLAN68_snapshot-series-design.md rename to issues/old/PLAN68_snapshot-series-design.md diff --git a/issues/PLAN68_snapshot-series.md b/issues/old/PLAN68_snapshot-series.md similarity index 100% rename from issues/PLAN68_snapshot-series.md rename to issues/old/PLAN68_snapshot-series.md