ci: Build the image only once

Author: madadam

.github/workflows/ci.yml

@@ -29,7 +29,7 @@ jobs:
         run: echo "CONTAINER=ouisync-unit-test-linux-$GITHUB_RUN_ID" >> $GITHUB_ENV
 
       - name: Start container
-        run: ./docker/linux.sh --container $CONTAINER --srcdir . --cache start
+        run: ./docker/linux.sh --container $CONTAINER --srcdir . --cache container start
 
       - name: Analyze
         run: ./docker/linux.sh --container $CONTAINER analyze
@@ -38,7 +38,7 @@ jobs:
         run: ./docker/linux.sh --container $CONTAINER unit-test
 
       - name: Stop container
-        run: ./docker/linux.sh --container $CONTAINER stop
+        run: ./docker/linux.sh --container $CONTAINER container stop
         if: always()
 
   test_windows:
@@ -278,9 +278,16 @@ jobs:
           name: ${{ matrix.name }}
           path: releases/release_*/${{ matrix.artifact-files }}
 
+  build_image_for_integration_test_android:
+    name: "build docker image to run integration tests on android"
+    runs-on: self-hosted
+    steps:
+      - run: ./docker/linux.sh container build
+
   integration_test_android:
     name: "run integration tests on android ${{ matrix.api-level }}"
     runs-on: self-hosted
+    needs: ["build_image_for_integration_test_android"]
     strategy:
       fail-fast: false
       matrix:

docker/linux.sh

@@ -90,17 +90,15 @@ function print_help() {
             echo
             echo "Usage: $0 analyze"
             ;;
-        "start")
-            echo "Explicitly start the container"
+        "container")
+            echo "Explicitly manage the container"
             echo
-            echo "Usage: $0 start"
+            echo "Usage: $0 container <start|stop|build>"
             echo
-            echo "Subsequent commands run on this container instead of starting a new one."
-            ;;
-        "stop")
-            echo "Explicitly stop the container"
-            echo
-            echo "Usage: $0 stop"
+            echo "Commands:"
+            echo "    start Start the container"
+            echo "    stop  Stop the container"
+            echo "    build Build the container image"
             ;;
         "shell")
             echo "Run a shell session in the container"
@@ -128,8 +126,7 @@ function print_help() {
             echo "    unit-test         Run unit tests"
             echo "    integration-test  Run integration tests"
             echo "    analyze           Analyze the dart source code"
-            echo "    start             Explicitly start the container"
-            echo "    stop              Explicitly stop the container"
+            echo "    container         Explicitly manage the container"
             echo
             echo "See '$0 help <command> for more information on a specific command"
             ;;
@@ -326,6 +323,20 @@ function is_container_running() {
     fi
 }
 
+function manage_container() {
+    case $1 in
+        start)
+            start_container tail -f /dev/null
+            ;;
+        stop)
+            stop_container
+            ;;
+        build)
+            build_container
+            ;;
+    esac
+}
+
 function init() {
     # Auto-start the container unless already running
     is_container_running || auto_start_container
@@ -638,12 +649,6 @@ case "$1" in
         print_help ${@:2}
         exit
         ;;
-    start)
-        start_container tail -f /dev/null
-        ;;
-    stop)
-        stop_container
-        ;;
     build|b)
         build ${@:2}
         ;;
@@ -656,6 +661,9 @@ case "$1" in
     analyze)
         analyze ${@:2}
         ;;
+    container)
+        manage_container ${@:2}
+        ;;
     shell|sh)
         init
         shell=1