fix: status ServiceSyncError caused by ClusterIp (#79)

Is it necessary to update the ClusterIP from the cep?
labring · Apr 2, 2022 · 43602ed · 43602ed
1 parent 995e634
commit 43602ed
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/controllers/sync.go b/controllers/sync.go
@@ -19,11 +19,12 @@ package controllers
 import (
 	"context"
 	"fmt"
-	"github.com/sealyun/endpoints-operator/metrics"
-	"k8s.io/klog"
 	"strconv"
 	"sync"
 
+	"github.com/sealyun/endpoints-operator/metrics"
+	"k8s.io/klog"
+
 	"github.com/sealyun/endpoints-operator/api/network/v1beta1"
 	libv1 "github.com/sealyun/endpoints-operator/library/api/core/v1"
 	corev1 "k8s.io/api/core/v1"
@@ -52,7 +53,7 @@ func (c *Reconciler) syncService(ctx context.Context, cep *v1beta1.ClusterEndpoi
 			if err := controllerutil.SetControllerReference(cep, svc, c.scheme); err != nil {
 				return err
 			}
-			svc.Spec.ClusterIP = cep.Spec.ClusterIP
+			// svc.Spec.ClusterIP = cep.Spec.ClusterIP
 			svc.Spec.Type = corev1.ServiceTypeClusterIP
 			svc.Spec.SessionAffinity = corev1.ServiceAffinityNone
 			svc.Spec.Ports = convertServicePorts(cep.Spec.Ports)