Skip to content

OpenBao backend: 管理者のいない個人やチームは、サーバの立て方も devbase が前提にする権限も分からず使えない → 公開の文書とポリシーの雛形だけで自分のサーバを立てて接続でき、手元のサーバでもコンテナの中の bao が届く(#295) - #468

Merged
takemi-ohama merged 4 commits into
mainfrom
design/issue-295
Oct 12, 2026

Conversation

@takemi-ohama

@takemi-ohama takemi-ohama commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

#295 の要求と受け入れ条件の写し、および設計文書(スプリント m8b)を置き、用語集に語を足しました。文書だけの変更で、コードは変えていません。

利用者向けの変化

  • 無し

移行の手順

  • 無し

変更の要点

テスト

supervise.py の run のステップの結果です。

ステップ exit 結果
glossary 0 ok。宣言と用語集が揃っている
requirements-check 2 stopped。issues/issue-295-requirements.md を読めない([Errno 2] No such file or ...)

requirements-check は止まっており、受け入れ条件の写しの確認は通っていません。このファイルはこの PR で追加しています。それでもステップから読めていないため、マージ前に原因を確かめる必要があります。

モード: standard / 通した工程: 設計 → 要求と受け入れ条件 → ドキュメントレビュー

🤖 Generated with Claude Code

決めたこと

issues/issue-295-design.md

  • 決定 1: 雛形を文書から相対リンクで開けるようにするため、ポリシーの雛形と試用の構成を docs/user/openbao/ に置く
  • 決定 2: 接続する側とサーバを用意する側の読み手を分けるため、サーバの用意を新しい文書 1 本にまとめ、利用の形の 3 つを見出しで分ける
  • 決定 3: 公開インタフェースを増やさないため、雛形を書き出す副命令を足さず、文書と雛形のファイルだけにする
  • 決定 4: 手元のサーバでもコンテナの中の bao を使えるようにするため、http の loopback の openbao.url はローカル扱いのときだけ、コンテナへ渡す BAO_ADDR のホストを host.docker.internal に置き換え、host-gateway の行を足す
  • 決定 5: ホストに bao を入れずに試せるようにするため、試用の構成を compose の 1 ファイルにし、管理の操作をサーバのコンテナの中の bao で打つ
  • 決定 6: 利用者ごとの権限を他の利用者の設定に触れずに変えられるようにするため、雛形のポリシーを利用者の entity へ直に付け、identity group を使わない
  • 決定 7: version: 2 の雛形をグループ名の書き換えだけで使えるようにするため、グループごとのポリシーを置き換え語 __GROUP__ から作る
  • 決定 8: 試用で既存の接続設定を失わないため、試用の前に secrets/backend.yml と secrets/bootstrap.env.age を退避し、片付けで戻す
  • 決定 9: 失効させる端末を secret_id の値を持たずに選べるようにするため、端末の名前を secret_id の metadata に入れて発行し、失効は accessor で行う

takemi-ohama and others added 2 commits October 12, 2026 22:30
利用者が個人・チームで自分の OpenBao サーバを立てる前提の設計。雛形と試用の構成を
docs/user/openbao/ に置き、サーバの用意を新しい文書 1 本にまとめ、loopback の
openbao.url ではコンテナへ渡す BAO_ADDR を host.docker.internal に置き換える。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 cross-review | round 1 | codex | REQUEST_CHANGES

修正が必要な指摘は major 1 件です。

Comment thread issues/issue-295-design.md Outdated

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 cross-review | round 1 | kiro | APPROVE

ドメインモデルの節(コンテキスト・集約・不変条件・ドメインイベント・用語)と用語集の差分を確認した。コンテキストは 2 つで、順応者(conformist)の関係が宣言されている。集約 3 つはそれぞれ持ち主が 1 つに決まる。不変条件 I1〜I10 は 1 行 1 条件で矛盾は無い。ドメインイベント E1〜E13 はすべて受け手を持ち、要求(issue-295-requirements.md)の E1〜E13 と番号・内容が一致する。用語の表の 6 語はいずれも用語集に反映されている。criticalまたはmajorに当たる修正アクションは、この節の範囲では見つからなかった。

https の loopback は証明書の名前が host.docker.internal と合わないため置き換えず、
受け入れ条件 16 のテスト設計と I7・用語集・流れ図・入出力の契約を揃える(#468 レビュー指摘)。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@takemi-ohama

Copy link
Copy Markdown
Contributor Author

🔧 /ndf:fix サマリ | round 1 | commit d0a8d5d

対応件数: critical=0 / major=1 / minor=0(合計 1 件)
決着: 1 件 / 見送り: 0 件 / 却下: 0 件
CI: SUCCESS

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 cross-review | round 2 | claude | APPROVE

round 1 の指摘(I7 の変換条件を http の loopback に限る)への対応差分を確認した。I7・用語の表・用語集(json/md)・入出力の契約の表・流れ図・決定 4・テスト設計(受け入れ条件 8・16 の行)で変換条件が http の loopback に揃っており、critical / major に当たる修正アクションは無い。

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 cross-review | round 2 | kiro | APPROVE

round 2 の差分(issue-295-design.md・glossary.md・glossary.json のみ、コード変更なし)を確認した。round 1 の major(I7 と用語集『コンテナから見た接続先』の変換条件を http の loopback に限定)に対応する変更で、I7(59 行)・用語の表(90 行)・用語集・入出力の契約の表(229-230 行、https の loopback 行を明示追加)・処理の流れの図(281 行)・決定 4(332・334 行)がすべて『http の loopback かつローカル扱いのときだけ host.docker.internal へ置き換え、https を含むそれ以外は openbao.url のまま』で一貫し、実装する人が同じ振る舞いを組める。取り残した『loopback』記述の食い違いは無い。critical / major に当たる修正アクションは、この差分の範囲で見つからなかった。

@takemi-ohama

Copy link
Copy Markdown
Contributor Author

🔧 /ndf:fix サマリ | commit (なし)

対応件数: critical=0 / major=0 / minor=0(合計 0 件)
決着: 0 件 / 見送り: 0 件 / 却下: 0 件
CI: NONE

@takemi-ohama
takemi-ohama marked this pull request as ready for review October 12, 2026 13:48
@takemi-ohama
takemi-ohama merged commit 23f1312 into main Oct 12, 2026
10 checks passed
@takemi-ohama takemi-ohama mentioned this pull request Oct 12, 2026
2 tasks done
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant