Fix check eip in dual stack

spidernet-io · Dec 6, 2023 · 7833c50 · 7833c50
1 parent 467deb2
commit 7833c50
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/pkg/controller/webhook/validate.go b/pkg/controller/webhook/validate.go
@@ -258,7 +258,7 @@ func checkEGWIppools(client client.Client, cfg *config.Config, ctx context.Conte
 	return nil
 }
 
-func checkEIP(client client.Client, ctx context.Context, ipv4, ipv6, egwName string) (bool, error) {
+func checkEIP(client client.Client, ctx context.Context, ipv4, ipv6, egwName string, cfg *config.Config) (bool, error) {
 
 	eipIPV4 := ipv4
 	eipIPV6 := ipv6
@@ -276,9 +276,11 @@ func checkEIP(client client.Client, ctx context.Context, ipv4, ipv6, egwName str
 		return false, err
 	}
 
-	if eipIPV4 == egw.Spec.Ippools.Ipv4DefaultEIP || eipIPV6 == egw.Spec.Ippools.Ipv6DefaultEIP {
-		if eipIPV4 != egw.Spec.Ippools.Ipv4DefaultEIP || eipIPV6 != egw.Spec.Ippools.Ipv6DefaultEIP {
-			return false, fmt.Errorf("%v egw Ipv4DefaultEIP=%v Ipv6DefaultEIP=%v, they can only be used together", egwName, egw.Spec.Ippools.Ipv4DefaultEIP, egw.Spec.Ippools.Ipv6DefaultEIP)
+	if cfg.FileConfig.EnableIPv4 && cfg.FileConfig.EnableIPv6 {
+		if eipIPV4 == egw.Spec.Ippools.Ipv4DefaultEIP || eipIPV6 == egw.Spec.Ippools.Ipv6DefaultEIP {
+			if eipIPV4 != egw.Spec.Ippools.Ipv4DefaultEIP || eipIPV6 != egw.Spec.Ippools.Ipv6DefaultEIP {
+				return false, fmt.Errorf("%v egw Ipv4DefaultEIP=%v Ipv6DefaultEIP=%v, they can only be used together", egwName, egw.Spec.Ippools.Ipv4DefaultEIP, egw.Spec.Ippools.Ipv6DefaultEIP)
+			}
 		}
 	}