Merge #404: dev: vairous fixes

18ca78f dev: vairous fixes (Cameron Garnham)

Pull request description:

ACKs for top commit:
  da2ce7:
    ACK 18ca78f

Tree-SHA512: fdfd7cf23cfc2fdaeb39d73bb455738e44929fa2dfacf4898dd06ba8c6671ca67bf55dff9ebb86c4894ba08b039adaecee2f3ea2b43eae3b238a4a60a23027ef

Author: da2ce7

.github/workflows/container.yaml

@@ -65,11 +65,15 @@ jobs:
       - id: check
         name: Check Context
         run: |
-          if [[ "${{ github.event_name }}" == "push" && ( "${{ github.ref }}" == "refs/heads/main" || "${{ github.ref }}" == "refs/heads/develop" || "${{ github.ref }}" == "refs/heads/docker" ) ]] ||
-             [[ "${{ github.event_name }}" == "push" && "${{ github.ref }}" =~ ^refs/tags/v[0-9]+\.[0-9]+\.[0-9]+$ ]]; then
-            if [[ "${{ github.repository }}" == "torrust/torrust-tracker" ]]; then
-              echo "Context is torrust/torrust-tracker, and push is: main, develop, docker, or a tag of v*.*.*"
-              echo "continue=true" >> $GITHUB_OUTPUT
+          if [[ "${{ github.repository }}" == "torrust/torrust-tracker" ]]; then
+            if [[ "${{ github.event_name }}" == "push" ]]; then
+              if [[ "${{ github.ref }}" == "refs/heads/main" ||
+                    "${{ github.ref }}" == "refs/heads/develop" ||
+                    "${{ github.ref }}" =~ ^v(0|[1-9]\d*)\.(0|[1-9]\d*)\.(0|[1-9]\d*)(?:-((?:0|[1-9]\d*|\d*[a-zA-Z-][0-9a-zA-Z-]*)(?:\.(?:0|[1-9]\d*|\d*[a-zA-Z-][0-9a-zA-Z-]*))*))?(?:\+([0-9a-zA-Z-]+(?:\.[0-9a-zA-Z-]+)*))?$ ]]; then
+
+                  echo "Context is torrust/torrust-tracker, and push is: main, develop, docker, or a tag with a semantic version"
+                  echo "continue=true" >> $GITHUB_OUTPUT
+              fi
             fi
           fi
 
@@ -126,7 +130,7 @@ jobs:
         uses: docker/build-push-action@v4
         with:
           file: ./Containerfile
-          push: ${{ github.event_name != 'pull_request' }}
+          push: true
           tags: ${{ steps.meta.outputs.tags }}
           labels: ${{ steps.meta.outputs.labels }}
           cache-from: type=gha

docs/containers.md

@@ -196,7 +196,7 @@ mkdir -p ./storage/tracker/lib/ ./storage/tracker/log/ ./storage/tracker/etc/
 ## Run Torrust Tracker Container Image
 docker run -it \
     --env TORRUST_TRACKER_API_ADMIN_TOKEN="MySecretToken" \
-    --env USER_ID"$(id -u)" \
+    --env USER_ID="$(id -u)" \
     --publish 0.0.0.0:7070:7070/tcp \
     --publish 0.0.0.0:6969:6969/udp \
     --publish 0.0.0.0:1212:1212/tcp \
@@ -218,7 +218,7 @@ mkdir -p ./storage/tracker/lib/ ./storage/tracker/log/ ./storage/tracker/etc/
 ## Run Torrust Tracker Container Image
 podman run -it \
     --env TORRUST_TRACKER_API_ADMIN_TOKEN="MySecretToken" \
-    --env USER_ID"$(id -u)" \
+    --env USER_ID="$(id -u)" \
     --publish 0.0.0.0:7070:7070/tcp \
     --publish 0.0.0.0:6969:6969/udp \
     --publish 0.0.0.0:1212:1212/tcp \
@@ -367,6 +367,7 @@ And finally, you can run the container:
 
 ```s
 docker run \
+    --env USER_ID="$(id -u)" \
     --publish 6969:6969/udp \
     --publish 7070:7070/tcp \
     --publish 1212:1212/tcp \
@@ -381,6 +382,7 @@ Detach from container logs when the container starts. By default, the command li
 ```s
 docker run \
     --detach
+    --env USER_ID="$(id -u)" \
     --publish 6969:6969/udp \
     --publish 7070:7070/tcp \
     --publish 1212:1212/tcp \latest

share/container/message

@@ -1,2 +1,4 @@
 
+Lovely welcome to our Torrust Tracker Container!
+
 run  'torrust-tracker'  to  start tracker