feat: handle affinity, anti-affinity, and topologySpreadConstraints

GSA-TTS / datagov-brokerpak-eks

Broker AWS EKS instances using the OSBAPI (eg from cloud.gov)

Other

9 stars 7 forks source link

feat: handle affinity, anti-affinity, and topologySpreadConstraints #96

Open mogul opened 2 years ago

mogul commented 2 years ago

...when auto-scaling.

mogul commented 2 years ago

I don't think so because the API hasn't changed.

That said, I don't understand how it's working now with these placeholders in there... I believe I copied them from wherever you originally had the YAML when I came up with this Helm solution. Do you know what they should be?

nickumia-reisys commented 2 years ago

Yes, I don't know how they got messed up, but,

The launch-template-name is supposed to be the same as https://github.com/GSA/datagov-brokerpak-eks/blob/52648eda4890aab6a2768bb35ca0d8e4af92f01f/terraform/modules/provision-aws/eks.tf#L109
The cluster-name is supposed to be the same name as the eks cluster, https://github.com/GSA/datagov-brokerpak-eks/blob/52648eda4890aab6a2768bb35ca0d8e4af92f01f/terraform/modules/provision-aws/eks.tf#L2

The original solution was doing a terraform replace on those strings with correct variables.