Merge pull request #113 from qinguoyi/master

add set controller reference err log
seaweedfs · Aug 2, 2024 · e75acf8 · e75acf8
2 parents 32927c9 + 8fa781f
commit e75acf8
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/internal/controller/controller_filer_ingress.go b/internal/controller/controller_filer_ingress.go
@@ -11,6 +11,7 @@ import (
 )
 
 func (r *SeaweedReconciler) createAllIngress(m *seaweedv1.Seaweed) *networkingv1.Ingress {
+	log := r.Log.WithValues("sw-create-ingress", m.Name)
 	labels := labelsForIngress(m.Name)
 	pathType := networkingv1.PathTypePrefix
 
@@ -95,6 +96,8 @@ func (r *SeaweedReconciler) createAllIngress(m *seaweedv1.Seaweed) *networkingv1
 	}
 
 	// Set master instance as the owner and controller
-	ctrl.SetControllerReference(m, dep, r.Scheme)
+	if err := ctrl.SetControllerReference(m, dep, r.Scheme); err != nil {
+		log.Error(err, "set controller reference for Ingress failed")
+	}
 	return dep
 }