Deprecate eks.Cluster defaultAddonsToRemoves property

[t0yv0]

Hello!

• Vote on this issue by adding a 👍 reaction
• If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details

In a recent investigation of the patch conflict (#2732 and #2998) it came up that we use upstream patching to implement default_addon_to_remove property (which pluralizes in Pulumi as defaultAddonToRemoves). This property seems to implement create hooks to remove eks.Cluster addons.

When trying to write a test for this functionality it appears that it is now redundant:

• provision simple eks.Cluster
• run aws eks list-addons --cluster-name cluster-b83e45e
• # { "addons": [] "

The cluster does not provision any addons by default. Instead, eks.Addon is a resource that users can opt into to provision the addons that are needed.

Suggest making this property a no-op, removing upstream patches, pertaining to it, and making it as deprecated to remove in a future version.

Affected area/feature

[mikhailshilkov]

Sounds like a good plan!

[SharpEdgeMarshall]

Sorry @t0yv0, I just checked and the creation of a new EKS cluster provision the 3 default addons (vpc-cni, core-dns, kubeproxy) even if the command aws eks list-addons --cluster-name <my-cluster> returns an empty list.
If the feature will be removed, how we are supposed to remove default addons?

[scottslowe]

My experience mirrors that of @SharpEdgeMarshall; even when I don't explicitly specify add-ons in my Pulumi code I'll still get VPC CNI, kube-proxy, and CoreDNS. If these add-ons are going to be installed by default, then how will users opt-out without this property?

[t0yv0]

Thanks @scottslowe and @SharpEdgeMarshall for our feedback! It appears that I incorrectly interpreted the aws eks list-addons --cluster-name cluster-b83e45e results as authoritative. If this is being relied on we will keep the option. Thank you!

[pulumi-bot]

Cannot close issue:

• does not have required labels: resolution/

Please fix these problems and try again.