fix forgotten verticalpodautoscaler.Config (#1506)

* fix forgotten verticalpodautoscaler.Config * move a const to a more suitable place --------- Co-authored-by: Herve Nicol <[email protected]>
giantswarm · Feb 5, 2024 · 8d9b502 · 8d9b502
1 parent aad9fa7
commit 8d9b502
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/service/controller/resource/monitoring/verticalpodautoscaler/resource.go b/service/controller/resource/monitoring/verticalpodautoscaler/resource.go
@@ -24,7 +24,8 @@ import (
 )
 
 const (
-	Name = "verticalpodautoscaler"
+	Name                              = "verticalpodautoscaler"
+	unknownObservabilityBundleVersion = "0.0.0"
 )
 
 type Config struct {
@@ -173,8 +174,6 @@ func (r *Resource) getObject(ctx context.Context, v interface{}) (*vpa_types.Ver
 	return vpa, nil
 }
 
-const unknownObservabilityBundleVersion = "0.0.0"
-
 // We get the management cluster observability bundle app version for the management cluster to know if we need to configure the VPA CR to target the StatefulSet or the Prometheus CR.
 // We need to do this because VPA uses the scale subresource to scale the target object, and the Prometheus CR has a scale subresource since the observability-bundle 1.2.0.
 func (r *Resource) getManagementClusterObservabilityBundleAppVersion(ctx context.Context, cluster metav1.Object) (string, error) {

diff --git a/service/controller/resource/resource.go b/service/controller/resource/resource.go
@@ -267,9 +267,11 @@ func New(config Config) ([]resource.Interface, error) {
 	var verticalPodAutoScalerResource resource.Interface
 	{
 		c := verticalpodautoscaler.Config{
-			Logger:    config.Logger,
-			K8sClient: config.K8sClient,
-			VpaClient: config.VpaClient,
+			Logger:       config.Logger,
+			K8sClient:    config.K8sClient,
+			VpaClient:    config.VpaClient,
+			Installation: config.Installation,
+			Provider:     config.Provider,
 		}
 
 		verticalPodAutoScalerResource, err = verticalpodautoscaler.New(c)