Fix describe locks bug

kufu-ai · Nov 8, 2024 · c781096 · c781096
1 parent 272632a
commit c781096
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/deploy/configmap.go b/deploy/configmap.go
@@ -68,8 +68,8 @@ func configMapKey(project, environment string) string {
 }
 
 func splitConfigMapKey(key string) (string, string) {
-	parts := strings.Split(key, Sep)
-	return parts[0], parts[1]
+	i := strings.LastIndex(key, Sep)
+	return key[:i], key[i+1:]
 }
 
 func (c *Coordinator) getOrCreateConfigMap(ctx context.Context) (*corev1.ConfigMap, error) {

diff --git a/deploy/lock_encoding_test.go b/deploy/lock_encoding_test.go
@@ -58,11 +58,9 @@ func TestKeysAndValuesEncoding(t *testing.T) {
 					},
 				},
 			},
-			// TODO There's a known bug where "myproject2-api/prod" are conflated as "myproject2/prod".
-			// },
-			// "myproject2-api": {
-			// 	"prod": {
-			"api": {
+		},
+		"myproject2-api": {
+			"prod": {
 				Locked: true,
 				LockHistory: []LockHistoryItem{
 					{