Skip to content

sub-backend の EndpointSlice に既定値を書く - #431

Merged
5ym merged 1 commit into
mainfrom
fix-endpointslice-diff
Oct 17, 2026
Merged

5ym merged 1 commit into
mainfrom
fix-endpointslice-diff

Conversation

@5ym

@5ym 5ym commented Oct 17, 2026

Copy link
Copy Markdown
Collaborator

#430 で bootstrap を ArgoCD に移したら、EndpointSlice/kube-system/sub-backend-manual だけ OutOfSync のまま残った。ports と endpoints は丸ごと比べるリストで、API サーバが足す protocol: TCP と conditions: {} が差分になる。git 側に書いて揃える(live の値と同じなので挙動は変わらない)。

🤖 Generated with Claude Code

https://claude.ai/code/session_01JbmWyUB9jcBcYCnNyiGBSV

@claude

claude Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

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


レビュー結果

問題なしです。

  • 不具合: ports[].protocol: TCP は API サーバの既定値と同じです。endpoints[].conditions: {} も live と同じ空オブジェクトなので、挙動は変わりません。ArgoCD の差分だけが消えるはずです。
  • セキュリティ: シークレットの混入はありません。宛先 10.10.0.2 は変更前から同じです。
  • 設計: 既定値を git 側に書いて揃える方法は、ArgoCD の標準的な対処です。コメントに理由が書いてあり、あとから読めます。ignoreDifferences を足すよりも影響範囲が小さいです。
  • テスト: マニフェストだけの変更で、自動テストの対象にはなりません。

要確認(任意): マージ後に ArgoCD で EndpointSlice/kube-system/sub-backend-manual が Synced になることを確認してください。ならなければ、live に他の既定値(例: conditions.ready など)が残っていないか argocd app diff で見てください。ここは diff を実行していないので、私は確認できていません。

@5ym
5ym merged commit 70f983f into main Oct 17, 2026
1 check passed
@5ym
5ym deleted the fix-endpointslice-diff branch October 17, 2026 14:47
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