Skip to content

Commit 8b31986

Browse files
authored
fix: use correct names for autoscaler policies configuration (#1167)
## Description PascalCase was used to create the `[[runners.autoscaler.policy]]` which is wrong according to the documentation. This PR fixes the names in the configuration file. `IdleCount` --> `idle_count` The worker instances were used once and then terminated. This resulted in a high load of some AWS autoscaling APIs and we got rate limiting. In the end most pipelines failed. ## Verification Deployed the changes into my environment. The worker machines are reused now.
1 parent e113eeb commit 8b31986

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

main.tf

+1-1
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,7 @@ locals {
130130
runners_autoscaling = [for config in var.runner_worker_docker_autoscaler_autoscaling_options : {
131131
for key, value in config :
132132
# Convert key from snake_case to PascalCase which is the casing for this section.
133-
join("", [for subkey in split("_", key) : title(subkey)]) => jsonencode(value) if value != null
133+
key => jsonencode(value) if value != null
134134
}]
135135
})
136136

0 commit comments

Comments
 (0)