cluster-api-provider-aws icon indicating copy to clipboard operation
cluster-api-provider-aws copied to clipboard

Consistent EKS addon management for vpc-cni and kube-proxy

Open MinhNguyen-at opened this issue 5 months ago • 1 comments
trafficstars

/kind feature Thanks to https://github.com/kubernetes-sigs/cluster-api-provider-aws/pull/5520 CAPA is on its way to provide some resemblance of the ability to migrate to self-managed versions of eks addons.

Currently, vpc-cni and kube-proxy are explicitly reconciled separately regardless of what is listed as an addon. vpc-cni reconciliation has the label "aws.cluster.x-k8s.io/prevent-deletion" to allow short-circuiting the ReconcileCNI() while ReconcileKubeProxy() does not.

It would be nice to have a consistent story for addons and migrating to self-managed versions of addons.

Proposed solution(s): a) add annotation/label check for deletion prevention to short circuit ReconcileKubeProxy, similar to ReconcileCNI. b) remove ReconcileKubeProxy and ReconcileCNI and have them as managed addons.

B is cleaner schema wise but would make it more difficult for existing users to upgrade CAPA versions.

  • Cluster-api-provider-aws version: v2.8.2 (big thanks to https://github.com/kubernetes-sigs/cluster-api-provider-aws/pull/5458 for fixing infinite reconciliation loop with addons)
  • Kubernetes version: 1.30
  • OS debian

MinhNguyen-at avatar Jun 13 '25 02:06 MinhNguyen-at