Skip to content

Commit

Permalink
fix option
Browse files Browse the repository at this point in the history
  • Loading branch information
BenHinthorne committed Jun 14, 2024
1 parent 60df670 commit db624a5
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 0 additions & 1 deletion cluster-autoscaler/expander/grpcplugin/grpc_client.go
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,6 @@ func (g *grpcclientstrategy) BestOptions(expansionOptions []expander.Option, nod
klog.V(4).Info("Unable to sanitize GPRC returned bestOptions, no options filtered")
return expansionOptions
}
// We would want these options to only have the similar nodegroups that the expander has not filtered out
return options
}

Expand Down
2 changes: 1 addition & 1 deletion cluster-autoscaler/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -178,7 +178,7 @@ var (

grpcExpanderCert = flag.String("grpc-expander-cert", "", "Path to cert used by gRPC server over TLS")
grpcExpanderURL = flag.String("grpc-expander-url", "", "URL to reach gRPC expander server.")
includeSimilarNodegroupsInExpanderOptions = flag.Bool("include-similar-nodegroups-in-expander-options", false, "if true, similar nodegroups will be included in the BestOptions request to the expander")
includeSimilarNodegroupsInExpanderOptions = flag.Bool("include-similar-nodegroups-in-expander-options", false, "If true, similar nodegroups will be included in the BestOptions sent to and returned by expander")

ignoreDaemonSetsUtilization = flag.Bool("ignore-daemonsets-utilization", false,
"Should CA ignore DaemonSet pods when calculating resource utilization for scaling down")
Expand Down

0 comments on commit db624a5

Please sign in to comment.