operator; fix max-targets validation for Streams

Nordix · Feb 6, 2023 · 6538230 · 6538230
1 parent a76d340
commit 6538230
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/api/v1/stream_webhook.go b/api/v1/stream_webhook.go
@@ -110,7 +110,8 @@ func (r *Stream) validateUpdate(oldObj runtime.Object) error {
 		return apierrors.NewForbidden(r.GroupResource(),
 			r.Name, field.Forbidden(field.NewPath("metadata", "labels", "trench"), "update on trench label is forbidden"))
 	}
-	if r.Spec.MaxTargets != old.Spec.MaxTargets {
+	if r.Spec.MaxTargets != old.Spec.MaxTargets &&
+		(r.Spec.MaxTargets == nil || old.Spec.MaxTargets == nil || *r.Spec.MaxTargets != *old.Spec.MaxTargets) {
 		return apierrors.NewForbidden(r.GroupResource(),
 			r.Name, field.Forbidden(field.NewPath("metadata", "spec", "max-targets"), "update on max-targets is forbidden"))
 	}