Merge #19089

19089: .github/tools-buildtest: factorize and parallelize using matrix strategy r=aabadie a=aabadie



Co-authored-by: Alexandre Abadie <alexandre.abadie@inria.fr>

.github/workflows/tools-buildtest.yml

@@ -15,69 +15,53 @@ on:
 jobs:
   tools-build:
     runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      max-parallel: 16
+      matrix:
+        tools:  # List of tools sorted by name in alphabetical order
+        - name: benchmark_udp
+          path: dist/tools
+        - name: bossa-1.8
+          path: dist/tools
+        - name: bossa-1.9
+          path: dist/tools
+        - name: bossa-nrf52
+          path: dist/tools
+        - name: calc_spi_scalers
+          path: cpu/kinetis/dist
+        - name: clk_conf
+          path: cpu/stm32/dist
+        - name: edbg
+          path: dist/tools
+        - name: ethos
+          path: dist/tools
+        - name: fixdep
+          path: dist/tools
+        - name: flatc
+          path: dist/tools
+        - name: lpc2k_pgm
+          path: dist/tools
+        - name: mosquitto_rsmb
+          path: dist/tools
+        - name: riotboot_serial
+          path: dist/tools
+        - name: setsid
+          path: dist/tools
+        - name: uhcpd
+          path: dist/tools
+        - name: zep_dispatch
+          path: dist/tools
     steps:
     - uses: actions/checkout@main
-    - name: Build flatc standalone
+    - name: Build ${{ matrix.tools.name }} in ${{ matrix.tools.path }}
       uses: aabadie/riot-action@main
       with:
-        cmd: make -C dist/tools/flatc
-    - name: Build mosquitto_rsmb standalone
-      uses: aabadie/riot-action@main
-      with:
-        cmd: make -C dist/tools/mosquitto_rsmb
-    - name: Build bossa-1.8 standalone
-      uses: aabadie/riot-action@main
-      with:
-        cmd: make -C dist/tools/bossa-1.8
-    - name: Build bossa-1.9 standalone
-      uses: aabadie/riot-action@main
-      with:
-        cmd: make -C dist/tools/bossa-1.9
-    - name: Build bossa-nrf52 standalone
-      uses: aabadie/riot-action@main
-      with:
-        cmd: make -C dist/tools/bossa-nrf52
-    - name: Build edbg standalone
-      uses: aabadie/riot-action@main
-      with:
-        cmd: make -C dist/tools/edbg
-    - name: Build setsid standalone
-      uses: aabadie/riot-action@main
-      with:
-        cmd: make -C dist/tools/setsid
-    - name: Build ethos
-      uses: aabadie/riot-action@main
-      with:
-        cmd: make -C dist/tools/ethos
-    - name: Build uhcpd
-      uses: aabadie/riot-action@main
-      with:
-        cmd: make -C dist/tools/uhcpd
-    - name: Build UDP Benchmark
-      uses: aabadie/riot-action@main
-      with:
-        cmd: make -C dist/tools/benchmark_udp
-    - name: Build stm32 clk_conf
-      uses: aabadie/riot-action@main
-      with:
-        cmd: make -C cpu/stm32/dist/clk_conf
-    - name: Build kinetis calc_spi_scalers
-      uses: aabadie/riot-action@main
-      with:
-        cmd: make -C cpu/kinetis/dist/calc_spi_scalers
-    - name: Build fixdep tool
-      uses: aabadie/riot-action@main
-      with:
-        cmd: make -C dist/tools/fixdep
-    - name: Build lpc2k_pgm tool
-      uses: aabadie/riot-action@main
-      with:
-        cmd: make -C dist/tools/lpc2k_pgm
-    - name: Build riotboot_serial tool
-      uses: aabadie/riot-action@main
-      with:
-        cmd: make -C dist/tools/riotboot_serial
-    - name: Build ZEP dispatcher & topology generator
-      uses: aabadie/riot-action@main
-      with:
-        cmd: make -C dist/tools/zep_dispatch
+        cmd: make -C ${{ matrix.tools.path }}/${{ matrix.tools.name }}
+  tools-build-success:
+    # this is only run if all jobs in tools-build succeeded
+    needs: tools-build
+    runs-on: ubuntu-latest
+    steps:
+    - name: tools-build succeeded
+      run: exit 0

bors.toml

@@ -2,7 +2,7 @@
 # (listed in same order as in the repo's branch protection settings)
 pr_status = [
  "python-tests",
- "tools-build",
+ "tools-build-success",
  "check-labels",
  "static-tests",
  "check-commits (commit-msg)",
@@ -15,7 +15,7 @@ pr_status = [
 status = [
   "Murdock",
   "static-tests",
-  "tools-build",
+  "tools-build-success",
   ]
 
 # Number of project members who must approve the PR (using GitHub Reviews)