Skip to content

docs(bootstrap): machine config に入るコメントを直す - #640

Merged
5ym merged 1 commit into
mainfrom
inline-comments
Oct 30, 2026
Merged

5ym merged 1 commit into
mainfrom
inline-comments

Conversation

@5ym

@5ym 5ym commented Oct 30, 2026

Copy link
Copy Markdown
Collaborator

#637 の見直しで残した、machine config (inlineManifests) にそのまま入るコメントの直し。

  • infisical-values.sops.yaml の 2 つ目 (infisical-secrets): 英語のコメントを日本語に、../../.sops.yaml を ../.sops.yaml に。SOPS は暗号化した値だけを MAC で見るので、コメントを直しても復号できる (手元で sops -d を確かめた)
  • apiserver-rbac.yaml: もう無い bootstrap-apply.yml と Application bootstrap の経緯を消す
  • ROADMAP の該当項目を消す

オブジェクトは変わらず、コメントだけ。 ただし描いたものは変わるので、マージ後に apply-config と upgrade-k8s で当て直す (コマンドは会話で渡す)。

🤖 Generated with Claude Code

https://claude.ai/code/session_01T9PbUfqG4pPxus99UeLiJs

infisical の Secret の英語のコメントを日本語にし、.sops.yaml への相対パスを直す (../../ → ../)。
apiserver-rbac の古い bootstrap-apply.yml と Application bootstrap の話を消す。ROADMAP の項目も消す。
コメントも inlineManifests に入るので、描いたものが変わる (当て直しが要る)。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01T9PbUfqG4pPxus99UeLiJs
@claude

claude Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

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


レビュー結果

問題は見つかりませんでした。

  • infisical-values.sops.yaml: 2 つ目のドキュメント(平文のコメントだけ)の変更です。../.sops.yaml は、bootstrap/ から見たリポジトリ直下の .sops.yaml に当たります(存在を確認しました)。1 つ目のドキュメントの sops: ブロックには触れていないので、MAC への影響はありません。日本語訳も元の意味(ENCRYPTION_KEY は失くせない、AUTH_SECRET は入れ替え可)と合っています。
  • apiserver-rbac.yaml: 消した記述は過去の経緯だけで、オブジェクトは変わりません。残したコメントに矛盾はありません。
  • ROADMAP.md: 消した項目は 3 つを挙げていました。残る local-path.yaml は、いま見るとコメントがすでに日本語で、古い参照もありません。項目を消してよい状態です。
  • マージ後に apply-config と upgrade-k8s を当て直す手順は PR 本文のとおりで、了解です。
    | branch: inline-comments

@5ym
5ym merged commit 9f08280 into main Oct 30, 2026
5 checks passed
@5ym
5ym deleted the inline-comments branch October 30, 2026 16:25
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