Always remove containers when using just run (#4171)

WordPress · Apr 20, 2024 · 0ed4071 · 0ed4071
1 parent b29c6a9
commit 0ed4071
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/catalog/justfile b/catalog/justfile
@@ -84,7 +84,6 @@ shell:
 # Launch an IPython shell in a new container under `SERVICE`
 ipython *args: up-deps
     env DC_USER="airflow" just ../run \
-        --rm \
         --workdir /opt/airflow/catalog/dags \
         {{ SERVICE }} \
         bash -c \'ipython {{ args }}\'
@@ -100,7 +99,6 @@ pgcli db_user_pass="deploy" db_name="openledger": up
 # Run a command in a test container under `SERVICE`
 _mount-test command: up-deps
     env DC_USER="airflow" just ../run \
-        --rm \
         -e AIRFLOW_VAR_INGESTION_LIMIT=1000000 \
         -w /opt/airflow/catalog \
         --volume \'{{ justfile_directory() }}/../docker\':/opt/airflow/docker/ \

diff --git a/justfile b/justfile
@@ -230,7 +230,7 @@ exec +args:
 
 # Execute statement in a new service container using Docker Compose
 run +args:
-    just dc run -u {{ env_var_or_default("DC_USER", "root") }} {{ EXEC_DEFAULTS }} "{{ args }}"
+    just dc run --rm -u {{ env_var_or_default("DC_USER", "root") }} {{ EXEC_DEFAULTS }} "{{ args }}"
 
 # Execute pgcli against one of the database instances
 _pgcli container db_user_pass db_name db_host db_port="5432":