helmChartValues.settings overwritten

[ApocDev]

When trying to apply a feature gate value, eg; settings.featureGates.spotToSpotConsolidation, the construct currently overwrites any values because it is not doing a deep merge on objects.

https://github.com/aws-samples/cdk-eks-karpenter/blob/main/src/index.ts#L156-L162

and

https://github.com/aws-samples/cdk-eks-karpenter/blob/main/src/index.ts#L197-L199

[andskli]

Hi @ApocDev, thanks for reporting this. It seems the behavior is undesired. I think what should happen here is that the inverse precedence should happen, i.e. allow helmExtraValues to overwrite what's in helmChartValues. Could you have a look at #168?