505: fail with message when webhook not enabled and minReplicas is nil

streamnative · Dec 5, 2022 · 0442021 · 0442021
1 parent 698546e
commit 0442021
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 3 deletions.
diff --git a/controllers/spec/common.go b/controllers/spec/common.go
@@ -1671,3 +1671,9 @@ func getFilenameOfComponentPackage(componentPackage string) string {
 	}
 	return componentPackage
 }
+
+func panicIfNil(value any, message string) {
+	if value == nil {
+		panic(message)
+	}
+}
diff --git a/controllers/spec/hpa.go b/controllers/spec/hpa.go
@@ -154,11 +154,16 @@ func makeBuiltinHPA(objectMeta *metav1.ObjectMeta, minReplicas, maxReplicas int3
 	}
 }
 
-func makeHPA(objectMeta *metav1.ObjectMeta, minReplicas, maxReplicas int32, podPolicy v1alpha1.PodPolicy, targetRef autov2beta2.CrossVersionObjectReference) *autov2beta2.HorizontalPodAutoscaler {
+func makeHPA(objectMeta *metav1.ObjectMeta, minReplicas, maxReplicas *int32, podPolicy v1alpha1.PodPolicy, targetRef autov2beta2.CrossVersionObjectReference) *autov2beta2.HorizontalPodAutoscaler {
+
+	panicIfNil(minReplicas, "MinReplicas should not be nil but is nil; This is likely because the webhook is not installed properly so it does not have a default value")
+	panicIfNil(maxReplicas, "MaxReplicas should not be nil but is nil; This is likely because the webhook is not installed properly so it does not have a default value")
+	min := *minReplicas
+	max := *maxReplicas
 	spec := autov2beta2.HorizontalPodAutoscalerSpec{
 		ScaleTargetRef: targetRef,
-		MinReplicas:    &minReplicas,
-		MaxReplicas:    maxReplicas,
+		MinReplicas:    &min,
+		MaxReplicas:    max,
 		Metrics:        podPolicy.AutoScalingMetrics,
 		Behavior:       podPolicy.AutoScalingBehavior,
 	}