apis: conformance: add more conflict condition on asymetrical traffic

Signed-off-by: Arthur Outhenin-Chalandre <[email protected]>

Author: MrFreezeex

conformance/service_import.go

@@ -214,9 +214,12 @@ func testClusterIPServiceImport() {
 				}
 			})
 
-			Specify("should expose the union of the constituent service ports", Label(RequiredLabel), func() {
+			Specify("should expose the union of the constituent service ports and raise a conflict", Label(RequiredLabel), func() {
 				AddReportEntry(SpecRefReportEntry, "https://github.com/kubernetes/enhancements/tree/master/keps/sig-multicluster/1645-multi-cluster-services-api#service-port")
 
+				t.awaitServiceExportCondition(&clients[0], v1alpha1.ServiceExportConditionConflict, metav1.ConditionTrue)
+				t.awaitServiceExportCondition(&clients[1], v1alpha1.ServiceExportConditionConflict, metav1.ConditionTrue)
+
 				t.awaitServiceImport(&clients[0], t.helloService.Name, false,
 					func(g Gomega, serviceImport *v1alpha1.ServiceImport) {
 						g.Expect(sortMCSPorts(serviceImport.Spec.Ports)).To(Equal(toMCSPorts(

pkg/apis/v1alpha1/serviceexport.go

@@ -235,10 +235,8 @@ const (
 	ServiceExportConditionConflict ServiceExportConditionType = "Conflict"
 
 	// ServiceExportReasonPortConflict is used with the "Conflict" condition
-	// when the exported service has a conflict related to port configuration.
-	// This includes when ports on resulting imported services would have
-	// duplicated names (including unnamed/empty name) or duplicated
-	// port/protocol pairs.
+	// when the exported service has a conflict related to port configuration
+	// if the ports are not identical in all the constituent Services.
 	ServiceExportReasonPortConflict ServiceExportConditionReason = "PortConflict"
 
 	// ServiceExportReasonTypeConflict is used with the "Conflict" condition
@@ -265,6 +263,13 @@ const (
 	// annotations.
 	ServiceExportReasonAnnotationsConflict ServiceExportConditionReason = "AnnotationsConflict"
 
+	// ServiceExportReasonIPFamilyConflict is used with the "Conflict" condition
+	// when the exported service has a conflict related to IPFamilies conflicting.
+	// IPFamilies handling is implemnetation specific but implementations may
+	// used when conflicting IPFamily may result in network traffic reaching
+	// only a subset of the backends depending on the IP protocol used.
+	ServiceExportReasonIPFamilyConflict ServiceExportConditionReason = "IPFamilyConflict"
+
 	// ServiceExportReasonNoConflicts is used with the "Conflict" condition
 	// when the condition is False.
 	ServiceExportReasonNoConflicts ServiceExportConditionReason = "NoConflicts"