chore(meshexternalservice): prevent targeting mesh circuit breaker by…

… validation Signed-off-by: slonka <[email protected]>
kumahq · Sep 19, 2024 · fa4aa70 · fa4aa70
1 parent 1a81ca1
commit fa4aa70
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 7 deletions.
diff --git a/pkg/plugins/policies/core/matchers/egress_test.go b/pkg/plugins/policies/core/matchers/egress_test.go
@@ -128,7 +128,7 @@ var _ = Describe("EgressMatchedPolicies", func() {
 					Expect(err).ToNot(HaveOccurred())
 					Expect(bytes).To(test_matchers.MatchGoldenYAML(strings.Replace(inputFile, ".input.", ".golden.", 1)))
 				},
-				Entry("should generate to resource rules for egress and mesh externalservice"),
+				XEntry("should generate to resource rules for egress and mesh externalservice"),
 			)
 		},
 		test.EntriesForFolder(filepath.Join("egressmatchedpolicies", "meshexternalservice", "torules")),
@@ -151,7 +151,7 @@ var _ = Describe("EgressMatchedPolicies", func() {
 					Expect(err).ToNot(HaveOccurred())
 					Expect(bytes).To(test_matchers.MatchGoldenYAML(strings.Replace(inputFile, ".input.", ".golden.", 1)))
 				},
-				Entry("should generate to resource rules for egress and mesh externalservice"),
+				XEntry("should generate to resource rules for egress and mesh externalservice"),
 			)
 		},
 		test.EntriesForFolder(filepath.Join("egressmatchedpolicies", "meshexternalservice", "fromtorules")),

diff --git a/pkg/plugins/policies/meshcircuitbreaker/api/v1alpha1/validator.go b/pkg/plugins/policies/meshcircuitbreaker/api/v1alpha1/validator.go
@@ -62,7 +62,6 @@ func validateTo(topTargetRef common_api.TargetRef, to []To) validators.Validatio
 			SupportedKinds: []common_api.TargetRefKind{
 				common_api.Mesh,
 				common_api.MeshService,
-				common_api.MeshExternalService,
 				common_api.MeshMultiZoneService,
 			},
 		}))

diff --git a/pkg/plugins/policies/meshcircuitbreaker/api/v1alpha1/validator_test.go b/pkg/plugins/policies/meshcircuitbreaker/api/v1alpha1/validator_test.go
@@ -228,7 +228,7 @@ to:
       name: web-backend
     default:
       connectionLimits: { }`),
-			Entry("with MeshExternalService example", `
+			XEntry("with MeshExternalService example", `
 targetRef:
   kind: Mesh
 to:
@@ -490,7 +490,7 @@ violations:
   - field: spec.to[0].default.outlierDetection.detectors.successRate.standardDeviationFactor
     message: 'invalid number'`,
 			}),
-			Entry("status codes out of range in expectedStatuses", testCase{
+			XEntry("status codes out of range in expectedStatuses", testCase{
 				inputYaml: `
 targetRef:
   kind: MeshService

diff --git a/pkg/plugins/policies/meshcircuitbreaker/plugin/v1alpha1/plugin_test.go b/pkg/plugins/policies/meshcircuitbreaker/plugin/v1alpha1/plugin_test.go
@@ -463,7 +463,7 @@ var _ = Describe("MeshCircuitBreaker", func() {
 		}),
 	)
 
-	It("should generate correct configuration for MeshExternalService with ZoneEgress", func() {
+	XIt("should generate correct configuration for MeshExternalService with ZoneEgress", func() {
 		// given
 		rs := core_xds.NewResourceSet()
 		rs.Add(&core_xds.Resource{

diff --git a/test/e2e_env/universal/meshexternalservice/meshexternalservice.go b/test/e2e_env/universal/meshexternalservice/meshexternalservice.go
@@ -675,7 +675,7 @@ spec:
 		})
 	})
 
-	Context("MeshExternalService with MeshCircuitBreaker", func() {
+	XContext("MeshExternalService with MeshCircuitBreaker", func() {
 		E2EAfterEach(func() {
 			Expect(DeleteMeshResources(universal.Cluster, meshNameNoDefaults,
 				meshcircuitbreaker_api.MeshCircuitBreakerResourceTypeDescriptor,