Fix horizontal scaling for ReplicationGroups (#136)

gfrey · web-flow · commit a63890eeb966 · 2024-07-01T06:55:55.000Z
This will set the value for the number of node groups of a ReplicationGroup to the value given in the Spec, thereby enabling horizontal scaling. Fixes aws-controllers-k8s/community#2080. By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
diff --git a/pkg/resource/replication_group/post_set_output.go b/pkg/resource/replication_group/post_set_output.go
@@ -89,6 +89,10 @@ func setNodeGroupConfiguration(
 
 		ko.Spec.NodeGroupConfiguration = nodeGroupConfigurations
 	}
+
+	if respRG.NodeGroups != nil && ko.Spec.NumNodeGroups != nil {
+		*ko.Spec.NumNodeGroups = int64(len(respRG.NodeGroups))
+	}
 }
 
 //TODO: for all the fields here, reevaluate if the latest observed state should always be populated,

Original file line number	Diff line number	Diff line change
`@@ -89,6 +89,10 @@ func setNodeGroupConfiguration(`
`89`	`89`
`90`	`90`	`ko.Spec.NodeGroupConfiguration = nodeGroupConfigurations`
`91`	`91`	`}`
	`92`	`+`
	`93`	`+ if respRG.NodeGroups != nil && ko.Spec.NumNodeGroups != nil {`
	`94`	`+ *ko.Spec.NumNodeGroups = int64(len(respRG.NodeGroups))`
	`95`	`+ }`
`92`	`96`	`}`
`93`	`97`
`94`	`98`	`//TODO: for all the fields here, reevaluate if the latest observed state should always be populated,`