Rename reserved namespace to gatekeeper_internal_

Signed-off-by: Max Smythe <[email protected]>
open-policy-agent · Jan 5, 2024 · b8a5935 · b8a5935
1 parent 70504d1
commit b8a5935
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 13 deletions.
diff --git a/constraint/pkg/client/drivers/k8scel/schema/schema.go b/constraint/pkg/client/drivers/k8scel/schema/schema.go
@@ -19,7 +19,7 @@ const (
 	Name = "K8sNativeValidation"
 	// ReservedPrefix signifies a prefix that no user-defined value (variable, matcher, etc.) is allowed to have.
 	// This gives us the ability to add new variables in the future without worrying about breaking pre-existing templates.
-	ReservedPrefix = "g8r_internal_"
+	ReservedPrefix = "gatekeeper_internal_"
 	// ParamsName is the VAP variable constraint parameters will be bound to.
 	ParamsName = "params"
 )

diff --git a/constraint/pkg/client/drivers/k8scel/schema/schema_test.go b/constraint/pkg/client/drivers/k8scel/schema/schema_test.go
@@ -58,7 +58,7 @@ func TestValidationErrors(t *testing.T) {
 				FailurePolicy: ptr.To[string]("Fail"),
 				MatchConditions: []MatchCondition{
 					{
-						Name:       "g8r_internal_must_match_something",
+						Name:       "gatekeeper_internal_must_match_something",
 						Expression: "true == true",
 					},
 				},
@@ -83,7 +83,7 @@ func TestValidationErrors(t *testing.T) {
 				},
 				Variables: []Variable{
 					{
-						Name:       "g8r_internal_my_variable",
+						Name:       "gatekeeper_internal_my_variable",
 						Expression: "true",
 					},
 				},

diff --git a/constraint/pkg/client/drivers/k8scel/transform/cel_snippets.go b/constraint/pkg/client/drivers/k8scel/transform/cel_snippets.go
@@ -72,7 +72,7 @@ const (
 
 func MatchExcludedNamespacesGlobV1Alpha1() admissionregistrationv1alpha1.MatchCondition {
 	return admissionregistrationv1alpha1.MatchCondition{
-		Name:       "g8r_internal_match_excluded_namespaces",
+		Name:       "gatekeeper_internal_match_excluded_namespaces",
 		Expression: matchExcludedNamespacesGlob,
 	}
 }
@@ -89,7 +89,7 @@ func MatchExcludedNamespacesGlobCEL() []cel.ExpressionAccessor {
 
 func MatchNamespacesGlobV1Alpha1() admissionregistrationv1alpha1.MatchCondition {
 	return admissionregistrationv1alpha1.MatchCondition{
-		Name:       "g8r_internal_match_namespaces",
+		Name:       "gatekeeper_internal_match_namespaces",
 		Expression: matchNamespacesGlob,
 	}
 }
@@ -106,7 +106,7 @@ func MatchNamespacesGlobCEL() []cel.ExpressionAccessor {
 
 func MatchNameGlobV1Alpha1() admissionregistrationv1alpha1.MatchCondition {
 	return admissionregistrationv1alpha1.MatchCondition{
-		Name:       "g8r_internal_match_name",
+		Name:       "gatekeeper_internal_match_name",
 		Expression: matchNameGlob,
 	}
 }
@@ -123,7 +123,7 @@ func MatchNameGlobCEL() []cel.ExpressionAccessor {
 
 func MatchKindsV1Alpha1() admissionregistrationv1alpha1.MatchCondition {
 	return admissionregistrationv1alpha1.MatchCondition{
-		Name:       "g8r_internal_match_kinds",
+		Name:       "gatekeeper_internal_match_kinds",
 		Expression: matchKinds,
 	}
 }

diff --git a/constraint/pkg/client/drivers/k8scel/transform/make_vap_objects_test.go b/constraint/pkg/client/drivers/k8scel/transform/make_vap_objects_test.go
@@ -61,19 +61,19 @@ func TestTemplateToPolicyDefinition(t *testing.T) {
 							Expression: "true == true",
 						},
 						{
-							Name:       "g8r_internal_match_excluded_namespaces",
+							Name:       "gatekeeper_internal_match_excluded_namespaces",
 							Expression: matchExcludedNamespacesGlob,
 						},
 						{
-							Name:       "g8r_internal_match_namespaces",
+							Name:       "gatekeeper_internal_match_namespaces",
 							Expression: matchNamespacesGlob,
 						},
 						{
-							Name:       "g8r_internal_match_name",
+							Name:       "gatekeeper_internal_match_name",
 							Expression: matchNameGlob,
 						},
 						{
-							Name:       "g8r_internal_match_kinds",
+							Name:       "gatekeeper_internal_match_kinds",
 							Expression: matchKinds,
 						},
 					},
@@ -105,7 +105,7 @@ func TestTemplateToPolicyDefinition(t *testing.T) {
 				FailurePolicy: ptr.To[string]("Fail"),
 				MatchConditions: []schema.MatchCondition{
 					{
-						Name:       "g8r_internal_match_something",
+						Name:       "gatekeeper_internal_match_something",
 						Expression: "true == true",
 					},
 				},
@@ -138,7 +138,7 @@ func TestTemplateToPolicyDefinition(t *testing.T) {
 				},
 				Variables: []schema.Variable{
 					{
-						Name:       "g8r_internal_my_variable",
+						Name:       "gatekeeper_internal_my_variable",
 						Expression: "true",
 					},
 				},