Skip to content

Conversation

@maxsxu
Copy link
Member

@maxsxu maxsxu commented Mar 4, 2025

Motivation

There're two kinds of add-ons for EKS:

  • Self-managed add-ons: it is installed by default when the cluster is created, but is unmanaged. including VPC CNI, CoreDNS, and kube-proxy. (This what we use for now)
  • Amazon EKS add-ons: install add-on by explicit declaration

We should use Amazon EKS add-ons for our new AWS CloudEnvironment

Modifications

  • Take over coredns, kube-proxy, vpc-cni
  • Enable prefix delegation for vpc-cni, which just affect the aws-node DaemonSet and new Nodes

Verifying this change

  • Make sure that the change passes the CI checks.

(Please pick either of the following options)

This change is a trivial rework / code cleanup without any test coverage.

(or)

This change is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(example:)

  • Added integration tests for end-to-end deployment with large payloads (10MB)
  • Extended integration test for recovery after broker failure

Documentation

  • no-need-doc

@maxsxu maxsxu requested review from ciiiii and dpappa March 4, 2025 02:49
@maxsxu maxsxu requested a review from a team as a code owner March 4, 2025 02:49
@github-actions github-actions bot added the no-need-doc This pr does not need any document label Mar 4, 2025
@ignacioli
Copy link

Does it mean we will get rid of executing eksctl utils update-.* when upgrading EKS?

@maxsxu maxsxu merged commit 60a8b1d into master Mar 4, 2025
4 checks passed
@maxsxu maxsxu deleted the max/feat branch March 4, 2025 08:18
maxsxu pushed a commit that referenced this pull request Mar 4, 2025
🤖 I have created a release *beep* *boop*
---


##
[4.2.0](v4.1.2...v4.2.0)
(2025-03-04)


### Features

* take over self-managed add-ons with Amazon EKS add-ons
([#153](#153))
([60a8b1d](60a8b1d))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-need-doc This pr does not need any document

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants