Merge pull request #568 from NERSC/update-envs

clarify cfg.LOCAL and remove DOCKER_COMPATIBILITY_MODE

Author: swelborn

backend/agent/.env.example

@@ -1,5 +1,4 @@
-DOCKER_COMPATIBILITY_MODE=true
-LOCAL=true
+USE_NORMAL_PODMAN=true
 NATS_SERVER_URL=nats://localhost:4222
 NATS_SERVER_URL_IN_CONTAINER=nats://host.containers.internal:4222
 PODMAN_SERVICE_URI=unix:///var/folders/np/......./something.sock

backend/agent/interactem/agent/agent.py

@@ -72,7 +72,7 @@
     PODMAN_SERVICE_URI = None
 
 # Use configuration, as we always expect to install podman-hpc-client
-if cfg.LOCAL:
+if cfg.USE_NORMAL_PODMAN:
     PODMAN_COMMAND = "podman"
     from podman import PodmanClient
 else:
@@ -992,7 +992,7 @@ async def create_container(
                 await handle_name_conflict(client, op_name)
 
             create_kwargs: dict[str, object] = {}
-            if not cfg.LOCAL and operator.requires_gpus:
+            if not cfg.USE_NORMAL_PODMAN and operator.requires_gpus:
                 create_kwargs["gpu"] = True
 
             return await to_thread.run_sync(

backend/agent/interactem/agent/config.py

@@ -25,8 +25,7 @@ def _load_vector_template() -> str:
 class Settings(BaseSettings):
     model_config = SettingsConfigDict(env_file=".env", extra="ignore")
     ID: uuid.UUID = Field(default_factory=uuid.uuid4)
-    LOCAL: bool = False
-    DOCKER_COMPATIBILITY_MODE: bool = False
+    USE_NORMAL_PODMAN: bool = False
     PODMAN_SERVICE_URI: str | None = None
     PODMAN_BINARY_PATH: Path | None = None
     NATS_SERVER_URL: AnyWebsocketUrl | NatsDsn = NatsDsn("nats://localhost:4222")