Merge #270: Fix docker compose configuration with the new tracker container image

josecelano · josecelano · commit 158757b25596 · 2023-09-11T15:28:48.000+01:00
bf95d85 fix: docker compose with the new tracker container image (Jose Celano) Pull request description: The [Torrust Tracker container image has changed](torrust/torrust-tracker@7d133ae). We have to upgrade the configuration to use the latest version. Integration tests fail for pull requests because the tracker container cannot start. The significant changes introduced were: - Rename some env vars: TORRUST_TRACKER_API_TOKEN->TORRUST_TRACKER_API_ADMIN_TOKEN - Database folder was moved to a new location Top commit has no ACKs. Tree-SHA512: 397ba6dbcd99ca38a6dd9d932e899303ea8e67068fdb2180bcf313887bea1972353f5017ca30139b9e48a79a7f746f29dc644e2c7ecf32a12597adbe6a7c5f62
diff --git a/.env.local b/.env.local
@@ -2,5 +2,5 @@ DATABASE_URL=sqlite://storage/database/data.db?mode=rwc
 TORRUST_IDX_BACK_CONFIG=
 TORRUST_IDX_BACK_USER_UID=1000
 TORRUST_TRACKER_CONFIG=
-TORRUST_TRACKER_USER_UID=1000
-TORRUST_TRACKER_API_TOKEN=MyAccessToken
+TORRUST_TRACKER_DATABASE=sqlite3
+TORRUST_TRACKER_API_ADMIN_TOKEN=MyAccessToken
diff --git a/compose.yaml b/compose.yaml
@@ -38,29 +38,21 @@ services:
 
   tracker:
     image: torrust/tracker:develop
-    user: ${TORRUST_TRACKER_USER_UID:-1000}:${TORRUST_TRACKER_USER_UID:-1000}
     tty: true
     environment:
       - TORRUST_TRACKER_CONFIG=${TORRUST_TRACKER_CONFIG}
-      - TORRUST_TRACKER_API_TOKEN=${TORRUST_TRACKER_API_TOKEN:-MyAccessToken}
+      - TORRUST_TRACKER_DATABASE=${TORRUST_TRACKER_DATABASE:-sqlite3}
+      - TORRUST_TRACKER_API_ADMIN_TOKEN=${TORRUST_TRACKER_API_ADMIN_TOKEN:-MyAccessToken}
     networks:
       - server_side
     ports:
       - 6969:6969/udp
-      - 1212:1212/tcp
-    # todo: implement healthcheck
-    #healthcheck:
-    #  test:
-    #    [
-    #      "CMD-SHELL",
-    #      "/app/main healthcheck"
-    #    ]
-    #  interval: 10s
-    #  retries: 5
-    #  start_period: 10s
-    #  timeout: 3s
+      - 7070:7070
+      - 1212:1212
     volumes:
-      - ./storage:/app/storage
+      - ./storage/tracker/lib:/var/lib/torrust/tracker:Z
+      - ./storage/tracker/log:/var/log/torrust/tracker:Z
+      - ./storage/tracker/etc:/etc/torrust/tracker:Z
     depends_on:
       - mysql
 
diff --git a/config-tracker.local.toml b/config-tracker.local.toml
@@ -1,7 +1,7 @@
 log_level = "info"
 mode = "public"
 db_driver = "Sqlite3"
-db_path = "./storage/database/torrust_tracker_e2e_testing.db"
+db_path = "/var/lib/torrust/tracker/database/torrust_tracker_e2e_testing.db"
 announce_interval = 120
 min_announce_interval = 120
 max_peer_timeout = 900
diff --git a/docker/README.md b/docker/README.md
@@ -65,8 +65,9 @@ Build and run it locally:
 ```s
 TORRUST_IDX_BACK_USER_UID=${TORRUST_IDX_BACK_USER_UID:-1000} \
     TORRUST_IDX_BACK_CONFIG=$(cat config-idx-back.local.toml) \
+    TORRUST_TRACKER_DATABASE=${TORRUST_TRACKER_DATABASE:-mysql} \
     TORRUST_TRACKER_CONFIG=$(cat config-tracker.local.toml) \
-    TORRUST_TRACKER_API_TOKEN=${TORRUST_TRACKER_API_TOKEN:-MyAccessToken} \
+    TORRUST_TRACKER_API_ADMIN_TOKEN=${TORRUST_TRACKER_API_ADMIN_TOKEN:-MyAccessToken} \
     docker compose up -d --build
 ```
 
diff --git a/docker/bin/e2e/mysql/e2e-env-reset.sh b/docker/bin/e2e/mysql/e2e-env-reset.sh
@@ -19,15 +19,14 @@ mysql -h $MYSQL_HOST -u $MYSQL_USER -p$MYSQL_PASSWORD -e "DROP DATABASE IF EXIST
 # Tracker
 
 # Delete tracker database
-rm -f ./storage/database/torrust_tracker_e2e_testing.db
+rm -f ./storage/tracker/lib/database/torrust_tracker_e2e_testing.db
 
 # Generate storage directory if it does not exist
-mkdir -p "./storage/database"
+mkdir -p "./storage/tracker/lib/database"
 
 # Generate the sqlite database for the tracker if it does not exist
-if ! [ -f "./storage/database/torrust_tracker_e2e_testing.db" ]; then
-    touch ./storage/database/torrust_tracker_e2e_testing.db
-    echo ";" | sqlite3 ./storage/database/torrust_tracker_e2e_testing.db
+if ! [ -f "./storage/tracker/lib/database/torrust_tracker_e2e_testing.db" ]; then
+    sqlite3 ./storage/tracker/lib/database/torrust_tracker_e2e_testing.db "VACUUM;"
 fi
 
 ./docker/bin/e2e/mysql/e2e-env-up.sh
diff --git a/docker/bin/e2e/mysql/e2e-env-up.sh b/docker/bin/e2e/mysql/e2e-env-up.sh
@@ -7,6 +7,7 @@ TORRUST_IDX_BACK_USER_UID=${TORRUST_IDX_BACK_USER_UID:-1000} \
     TORRUST_IDX_BACK_CONFIG=$(cat config-idx-back.mysql.local.toml) \
     TORRUST_IDX_BACK_MYSQL_DATABASE="torrust_index_backend_e2e_testing" \
     TORRUST_TRACKER_CONFIG=$(cat config-tracker.local.toml) \
-    TORRUST_TRACKER_API_TOKEN=${TORRUST_TRACKER_API_TOKEN:-MyAccessToken} \
+    TORRUST_TRACKER_DATABASE=${TORRUST_TRACKER_DATABASE:-mysql} \
+    TORRUST_TRACKER_API_ADMIN_TOKEN=${TORRUST_TRACKER_API_ADMIN_TOKEN:-MyAccessToken} \
     docker compose up -d
 
diff --git a/docker/bin/e2e/sqlite/e2e-env-reset.sh b/docker/bin/e2e/sqlite/e2e-env-reset.sh
@@ -5,22 +5,20 @@
 docker compose down
 
 rm -f ./storage/database/torrust_index_backend_e2e_testing.db
-rm -f ./storage/database/torrust_tracker_e2e_testing.db
+rm -f ./storage/tracker/lib/database/torrust_tracker_e2e_testing.db
 
 # Generate storage directory if it does not exist
 mkdir -p "./storage/database"
 
 # Generate the sqlite database for the index backend if it does not exist
 if ! [ -f "./storage/database/torrust_index_backend_e2e_testing.db" ]; then
     # todo: it should get the path from config.toml and only do it when we use sqlite
-    touch ./storage/database/torrust_index_backend_e2e_testing.db
-    echo ";" | sqlite3 ./storage/database/torrust_index_backend_e2e_testing.db
+    sqlite3 ./storage/database/torrust_index_backend_e2e_testing.db "VACUUM;"
 fi
 
 # Generate the sqlite database for the tracker if it does not exist
-if ! [ -f "./storage/database/torrust_tracker_e2e_testing.db" ]; then
-    touch ./storage/database/torrust_tracker_e2e_testing.db
-    echo ";" | sqlite3 ./storage/database/torrust_tracker_e2e_testing.db
+if ! [ -f "./storage/tracker/lib/database/torrust_tracker_e2e_testing.db" ]; then
+    sqlite3 ./storage/tracker/lib/database/torrust_tracker_e2e_testing.db "VACUUM;"
 fi
 
 ./docker/bin/e2e/sqlite/e2e-env-up.sh
diff --git a/docker/bin/e2e/sqlite/e2e-env-up.sh b/docker/bin/e2e/sqlite/e2e-env-up.sh
@@ -5,6 +5,9 @@ TORRUST_IDX_BACK_USER_UID=${TORRUST_IDX_BACK_USER_UID:-1000} \
 
 TORRUST_IDX_BACK_USER_UID=${TORRUST_IDX_BACK_USER_UID:-1000} \
     TORRUST_IDX_BACK_CONFIG=$(cat config-idx-back.sqlite.local.toml) \
+    TORRUST_IDX_BACK_MYSQL_DATABASE="torrust_index_backend_e2e_testing" \
     TORRUST_TRACKER_CONFIG=$(cat config-tracker.local.toml) \
-    TORRUST_TRACKER_API_TOKEN=${TORRUST_TRACKER_API_TOKEN:-MyAccessToken} \
+    TORRUST_TRACKER_DATABASE=${TORRUST_TRACKER_DATABASE:-sqlite3} \
+    TORRUST_TRACKER_API_ADMIN_TOKEN=${TORRUST_TRACKER_API_ADMIN_TOKEN:-MyAccessToken} \
     docker compose up -d
+
