Skip to content

Commit 144e96f

Browse files
authored
fix(k8s): remove oldbinpacking from autoscaler estimator (#887)
1 parent 7bd219a commit 144e96f

3 files changed

Lines changed: 4 additions & 4 deletions

File tree

cmd/scw/testdata/test-all-usage-k8s-cluster-create-usage.stderr.golden

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ ARGS:
3030
[pools.{index}.tags.{index}] The tags associated with the pool
3131
[autoscaler-config.scale-down-disabled] Disable the cluster autoscaler
3232
[autoscaler-config.scale-down-delay-after-add] How long after scale up that scale down evaluation resumes
33-
[autoscaler-config.estimator] Type of resource estimator to be used in scale up (unknown_estimator | binpacking | oldbinpacking)
33+
[autoscaler-config.estimator] Type of resource estimator to be used in scale up (unknown_estimator | binpacking)
3434
[autoscaler-config.expander] Type of node group expander to be used in scale up (unknown_expander | random | most_pods | least_waste | priority)
3535
[autoscaler-config.ignore-daemonsets-utilization] Ignore DaemonSet pods when calculating resource utilization for scaling down
3636
[autoscaler-config.balance-similar-node-groups] Detect similar node groups and balance the number of nodes between them

cmd/scw/testdata/test-all-usage-k8s-cluster-update-usage.stderr.golden

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ ARGS:
1717
[tags.{index}] The new tags associated with the cluster
1818
[autoscaler-config.scale-down-disabled] Disable the cluster autoscaler
1919
[autoscaler-config.scale-down-delay-after-add] How long after scale up that scale down evaluation resumes
20-
[autoscaler-config.estimator] Type of resource estimator to be used in scale up (unknown_estimator | binpacking | oldbinpacking)
20+
[autoscaler-config.estimator] Type of resource estimator to be used in scale up (unknown_estimator | binpacking)
2121
[autoscaler-config.expander] Type of node group expander to be used in scale up (unknown_expander | random | most_pods | least_waste | priority)
2222
[autoscaler-config.ignore-daemonsets-utilization] Ignore DaemonSet pods when calculating resource utilization for scaling down
2323
[autoscaler-config.balance-similar-node-groups] Detect similar node groups and balance the number of nodes between them

internal/namespaces/k8s/v1/k8s_cli.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -338,7 +338,7 @@ func k8sClusterCreate() *core.Command {
338338
Short: `Type of resource estimator to be used in scale up`,
339339
Required: false,
340340
Positional: false,
341-
EnumValues: []string{"unknown_estimator", "binpacking", "oldbinpacking"},
341+
EnumValues: []string{"unknown_estimator", "binpacking"},
342342
},
343343
{
344344
Name: "autoscaler-config.expander",
@@ -510,7 +510,7 @@ func k8sClusterUpdate() *core.Command {
510510
Short: `Type of resource estimator to be used in scale up`,
511511
Required: false,
512512
Positional: false,
513-
EnumValues: []string{"unknown_estimator", "binpacking", "oldbinpacking"},
513+
EnumValues: []string{"unknown_estimator", "binpacking"},
514514
},
515515
{
516516
Name: "autoscaler-config.expander",

0 commit comments

Comments
 (0)