古いコメントと重複の掃除(releaseName・pulumi の準備・やめたアプリへの言及)

[5ym]

古いコメントと重複の掃除。動作は変わらない見込み。

• releaseName を 6 つの Application から外す(Application 名と同じ値。ArgoCD の既定)
• pulumi.yml の preview / up の共通の準備を .github/actions/pulumi-setup に
• 消えたファイルややめたアプリ(Mattermost・ERPNext・GitLab・Traefik・wg-easy)を指すコメントを直す
• nodePort の経緯を短く
• SOPS 2 ファイルを今の規則で暗号化し直す(復号した値は前後で同じことを sha256 で確認)。machine config の差分は infisical の Secret の先頭のコメント 4 行だけなので、マージ後に apply-config で揃える

🤖 Generated with Claude Code

https://claude.ai/code/session_01JbmWyUB9jcBcYCnNyiGBSV

[claude]

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

---

レビュー結果

問題は見つかりませんでした。動作が変わらないという PR の説明どおりに読めます。

確認したこと

• releaseName の削除: 外した 6 つのうち、Application 名と同じ値だったものだけです。違う値の apps/infisical-operator/application.yaml (infisical-secrets-operator) は残っています。
• .github/actions/pulumi-setup:
  • preview と up のどちらも actions/checkout の後に呼んでいます。ローカルの composite action は checkout が先に要るので、順序は正しいです。
  • composite の各 run に shell: bash があり、working-directory: pulumi も元のままです。
  • paths: &paths / *paths の YAML アンカーは、GitHub Actions のワークフローで使えます。
  • paths に .github/actions/pulumi-setup/** を足したので、action だけの変更でも preview が走ります。
• apps/infisical-operator への参照の直し: bootstrap/infisical-operator は存在せず、apps/infisical-operator が実在します。
• docs/router.md にある DMZ の記述 (31 行目) と、matrix の README / livekit.yaml の「転送は要らない」が合っています。
• SOPS:
  • .sops.yaml の encrypted_regex と 2 ファイルの encrypted_regex が一致しています。
  • 外れた githubAppPrivateKey・service.webhook.mattermost・password は、この 2 ファイルに残っていません。
  • 差分はコメント、age の暗号文、lastmodified だけです。
  • 復号値の sha256 が前後で同じかどうかは、ここでは検証できません (age の秘密鍵が無いため)。PR の記載を信用しています。

要確認 (任意)

• bootstrap/argocd/values.yaml の 73 行目付近と apps/k8up の mattermostWebhook のように、Mattermost という名前のキーが意図して残っています。名前はテンプレートや subscriptions が参照しているためで、コメントにも理由が書いてあり、今回の掃除の対象外で問題ありません。
• マージ後の apply-config で、infisical の Secret の先頭コメント 4 行を揃える手順は PR 本文のとおりです。

[github-actions]

🍹 preview on doany-cloudflare/prod

Pulumi report

Previewing update (prod):

@ previewing update......
pulumi:pulumi:Stack doany-cloudflare-prod  
Resources:
54 unchanged