[5ym]

CRD がまだ無いときの dry-run は danything/gitops の ApplicationSet の syncOptions がまとめて飛ばす(danything/gitops#682)ので、リソースごとの SkipDryRunOnMissingResource=true を外す。InfisicalSecret は Prune=false だけ残す。

namespace.yaml がある場合は、次の PR でファイルごとやめる(privileged-movers は ApplicationSet の managedNamespaceMetadata が付ける)。消すときに Namespace が prune されないよう、ここで Prune=false を先に当てる。

danything/gitops#682 が当たってからマージする。

🤖 Generated with Claude Code

https://claude.ai/code/session_01T8TqVdo6LUrWKdVQgbScVU

[claude]

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

---

レビュー結果

deploy/volsync.yaml のみの変更(SkipDryRunOnMissingResource=true を 3 箇所から削除、InfisicalSecret は Prune=false を維持)を確認しました。問題はありません。

• 不具合: 注釈の削除のみで、Prune=false は残っており、YAML も妥当です。
• セキュリティ: シークレットの混入や権限まわりの変更はありません。
• 設計: リソースごとの注釈をやめて ApplicationSet の syncOptions に集約する方針で、重複が減ります。
• テスト: マニフェストの変更なので該当なしです。

マージ順序について(要確認)

• PR 本文のとおり、chore(argocd): InfisicalSecret・VolSync の sync-options の注釈をやめる gitops#682 が先に反映されていないと、ReplicationSource / ReplicationDestination の CRD が無い状態で dry-run が失敗し、初回 sync が落ちます。#682 のマージ後に、このリポジトリの Application に SkipDryRunOnMissingResource が実際に効いていることを確認してからマージしてください。
• このリポジトリには deploy/namespace.yaml が無いため、本文にある「Namespace 用の Prune=false を先に当てる」処理はこの PR の対象外と理解しています。他リポジトリ向けの記述なら問題ありません。
  | branch: chore/skip-dry-run

[claude]

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

---

レビュー結果

変更は deploy/volsync.yaml の注釈削除のみ(3 箇所)です。YAML としては妥当で、不具合やセキュリティ上の問題はありません。テストは、マニフェストのみの変更なので対象外です。

指摘(要確認)

• PR 本文と差分がずれています。 本文には「InfisicalSecret は Prune=false だけ残す」とあります。しかし最新コミット 14383d9 で Prune=false も外れていて、InfisicalSecret の annotations ブロックごと消えています(deploy/volsync.yaml 9–10 行目付近)。
  • Prune=false を外すのが意図どおりなら、本文を直してください。InfisicalSecret がリポジトリから消えたときに、生成された Secret が prune される挙動で問題ないか確認してください。
  • 残すつもりだったなら、注釈を戻す必要があります。
  • なお、先に出ていたレビューコメントは「Prune=false を維持」としていましたが、これは現在の差分と合っていません。訂正します。
• namespace.yaml はこのリポジトリにありません。 deploy/ には argocd.yaml、deployment.yaml、httproute.yaml、pvc.yaml、secret.yaml、service.yaml、volsync.yaml しかありません。本文の namespace.yaml と「Prune=false を先に当てる」の記述は、この PR では実体がありません。本文の整理をお勧めします。
• マージ順序は chore(argocd): InfisicalSecret・VolSync の sync-options の注釈をやめる gitops#682 が先です。 #682 が反映される前にマージすると、ReplicationSource / ReplicationDestination の CRD が無い状態で dry-run が失敗し、初回 sync が落ちる恐れがあります。
  | branch: chore/skip-dry-run

[claude]

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

---

レビュー結果

deploy/secret.yaml と deploy/volsync.yaml の削除のみの変更を、3 コミット分まとめて確認しました。大きな問題は見つかっていません。ただし、PR 本文が実際の差分と合っておらず、マージ前提が 1 つ増えています。

要確認・指摘

• PR 本文が古い: 本文は「InfisicalSecret は Prune=false だけ残す」とあります。しかし 14383d9 でその Prune=false も外れており、deploy/volsync.yaml の InfisicalSecret には注釈がなくなりました。あわせて creationPolicy: Orphan と recursive: false も消えています。本文を現状に合わせて直してください。
• マージ前提が増えている(要確認): 14383d9 のメッセージによると、Argo CD は Secret を resource.exclusions で見ないため、operator が tracking-id を Secret に写しても prune されません。この前提も chore(argocd): InfisicalSecret・VolSync の sync-options の注釈をやめる gitops#682 に依存します。#682 が当たる前にマージすると、volsync-xool-db や xool-secrets が prune される恐れがあります。#682 の反映後にマージしてください。
• 既定値の削除(要確認): creationPolicy: Orphan と recursive: false は、使っている Infisical operator の CRD の既定値と同じという理解です。バージョンによって既定値が違わないか、CRD で確認してください。違っていれば、xool-secrets が CR 削除時に消える可能性があります。

その他の観点

• 不具合: 上記以外にありません。YAML も妥当です。
• セキュリティ: シークレットの混入や権限の変更はありません。
• 設計: リソースごとの注釈を ApplicationSet の syncOptions に集約する方針で、重複が減ります。
• テスト: マニフェストの変更なので該当なしです。
  | branch: chore/skip-dry-run