Sort automount configmaps and secrets to ensure deterministic ordering

dkwon17 · dkwon17 · commit 7728c25cc4d0 · 2026-01-29T20:58:36.000-05:00
Signed-off-by: David Kwon &lt;dakwon@redhat.com&gt;
diff --git a/pkg/provision/automount/configmap.go b/pkg/provision/automount/configmap.go
@@ -36,6 +36,7 @@ func getDevWorkspaceConfigmaps(namespace string, api sync.ClusterAPI) (*Resource
 	}); err != nil {
 		return nil, err
 	}
+	sortConfigmaps(configmaps.Items)
 	var allAutoMountResouces []Resources
 	for _, configmap := range configmaps.Items {
 		if msg := checkAutomountVolumeForPotentialError(&configmap); msg != "" {
diff --git a/pkg/provision/automount/secret.go b/pkg/provision/automount/secret.go
@@ -36,6 +36,7 @@ func getDevWorkspaceSecrets(namespace string, api sync.ClusterAPI) (*Resources,
 	}); err != nil {
 		return nil, err
 	}
+	sortSecrets(secrets.Items)
 	var allAutoMountResouces []Resources
 	for _, secret := range secrets.Items {
 		if msg := checkAutomountVolumeForPotentialError(&secret); msg != "" {

Original file line number	Diff line number	Diff line change
`@@ -36,6 +36,7 @@ func getDevWorkspaceConfigmaps(namespace string, api sync.ClusterAPI) (*Resource`
`36`	`36`	`}); err != nil {`
`37`	`37`	`return nil, err`
`38`	`38`	`}`
	`39`	`+ sortConfigmaps(configmaps.Items)`
`39`	`40`	`var allAutoMountResouces []Resources`
`40`	`41`	`for _, configmap := range configmaps.Items {`
`41`	`42`	`if msg := checkAutomountVolumeForPotentialError(&configmap); msg != "" {`
Original file line number	Diff line number	Diff line change
`@@ -36,6 +36,7 @@ func getDevWorkspaceSecrets(namespace string, api sync.ClusterAPI) (*Resources,`
`36`	`36`	`}); err != nil {`
`37`	`37`	`return nil, err`
`38`	`38`	`}`
	`39`	`+ sortSecrets(secrets.Items)`
`39`	`40`	`var allAutoMountResouces []Resources`
`40`	`41`	`for _, secret := range secrets.Items {`
`41`	`42`	`if msg := checkAutomountVolumeForPotentialError(&secret); msg != "" {`