fix: ensure correct format of memoryLimit in workspace config

modelix · Feb 11, 2025 · 24efa2d · 24efa2d
1 parent 6e0658a
commit 24efa2d
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/workspace-manager/src/main/kotlin/org/modelix/workspace/manager/WorkspaceManagerModule.kt b/workspace-manager/src/main/kotlin/org/modelix/workspace/manager/WorkspaceManagerModule.kt
@@ -52,6 +52,7 @@ import io.ktor.server.routing.post
 import io.ktor.server.routing.route
 import io.ktor.server.routing.routing
 import io.ktor.util.encodeBase64
+import io.kubernetes.client.custom.Quantity
 import kotlinx.html.DIV
 import kotlinx.html.FlowOrInteractiveOrPhrasingContent
 import kotlinx.html.FormEncType
@@ -1233,8 +1234,11 @@ suspend fun ApplicationCall.respondTarGz(body: (TarArchiveOutputStream) -> Unit)
 
 fun sanitizeReceivedWorkspaceConfig(receivedWorkspaceConfig: Workspace, existingWorkspaceConfig: Workspace): Workspace =
     mergeMaskedCredentialsWithPreviousCredentials(receivedWorkspaceConfig, existingWorkspaceConfig)
-        // set ID just in case the user copy-pastes a workspace and forgets to change the ID
-        .copy(id = existingWorkspaceConfig.id)
+        .copy(
+            // set ID just in case the user copy-pastes a workspace and forgets to change the ID
+            id = existingWorkspaceConfig.id,
+            memoryLimit = Quantity(receivedWorkspaceConfig.memoryLimit).toSuffixedString(),
+        )
 
 const val MASKED_CREDENTIAL_VALUE = "••••••••"