add core api sort priority for NAmespace kind in gvkLessThan

kubernetes-sigs · Nov 15, 2023 · ad5ca54 · ad5ca54
1 parent e219b88
commit ad5ca54
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 0 deletions.
diff --git a/api/types/kustomization.go b/api/types/kustomization.go
@@ -22,6 +22,7 @@ const (
 	MetadataNamespacePath       = "metadata/namespace"
 	MetadataNamespaceApiVersion = "v1"
 	MetadataNamePath            = "metadata/name"
+	NamespaceKind               = "Namespace"
 
 	OriginAnnotations      = "originAnnotations"
 	TransformerAnnotations = "transformerAnnotations"

diff --git a/plugin/builtin/sortordertransformer/SortOrderTransformer.go b/plugin/builtin/sortordertransformer/SortOrderTransformer.go
@@ -163,6 +163,9 @@ func gvkLessThan(gvk1, gvk2 resid.Gvk, typeOrders map[string]int) bool {
 	if index1 != index2 {
 		return index1 < index2
 	}
+	if (index1 == types.NamespaceKind && index2 == types.NamespaceKind) && (gvk1.Group == "" || gvk2.Group == "") {
+		return legacyGVKSortString(gvk1) > legacyGVKSortString(gvk2)
+	}
 	return legacyGVKSortString(gvk1) < legacyGVKSortString(gvk2)
 }
 

diff --git a/plugin/builtin/sortordertransformer/SortOrderTransformer_test.go b/plugin/builtin/sortordertransformer/SortOrderTransformer_test.go
@@ -69,6 +69,12 @@ kind: Namespace
 metadata:
   name: apple
 ---
+apiVersion: servicebus.azure.com/v1api20210101preview
+kind: Namespace
+metadata:
+  name: aso-namespace
+  namespace: default
+---
 apiVersion: v1
 kind: Secret
 metadata:
@@ -108,6 +114,12 @@ kind: Namespace
 metadata:
   name: apple
 ---
+apiVersion: servicebus.azure.com/v1api20210101preview
+kind: Namespace
+metadata:
+  name: aso-namespace
+  namespace: default
+---
 apiVersion: v1
 kind: Role
 metadata:
@@ -228,6 +240,12 @@ kind: Namespace
 metadata:
   name: apple
 ---
+apiVersion: servicebus.azure.com/v1api20210101preview
+kind: Namespace
+metadata:
+  name: aso-namespace
+  namespace: default
+---
 apiVersion: v1
 kind: Deployment
 metadata:
@@ -306,6 +324,12 @@ kind: Namespace
 metadata:
   name: apple
 ---
+apiVersion: servicebus.azure.com/v1api20210101preview
+kind: Namespace
+metadata:
+  name: aso-namespace
+  namespace: default
+---
 apiVersion: v1
 kind: Deployment
 metadata: