search

iits-consulting / terraform-opentelekomcloud-project-factory

This repository helps to create an OTC-based cloud-native infrastructure landscape with Kubernetes, load balancers, VPCs, etc. With these modules, we provide you a rocket start while you can still deep-dive into detailed configuration later.
GNU General Public License v3.0
83 stars 20 forks source link

cce: fix logical flaw in local var autoscaler_node_min #94

Closed k11h-de closed 10 months ago

k11h-de commented 10 months ago

problem

setting variable autoscaler_node_min = 3 always sets min_node_count to 1 in the created node pool(s) so having more than 1 node at minimum per node pool is not possible

cause

I think there is a minor typo in the logic in https://github.com/iits-consulting/terraform-opentelekomcloud-project-factory/blob/master/modules/cce/variables.tf#L213

test

I tested this locally and changing the operand fixed the problem

canaykin commented 10 months ago

Hi Karsten,

You are correct that the operand was wrong.

Thank you both for pointing it out and fixing it.

Best, Can.