From 8552465408e19f44498e0bcf3f89c3fb9ffe7a94 Mon Sep 17 00:00:00 2001
From: Ramon Roche <mrpollo@gmail.com>
Date: Thu, 9 Apr 2026 13:29:43 -0700
Subject: [PATCH] ci(workflows): shallow checkout and fail-fast in checks

Port checkout hygiene from the CI orchestrator branch
(mrpollo/ci_orchestration) to current workflows without merging the
orchestrator itself.

- checks.yml: enable fail-fast (99% success rate observed, cancel on
  first failure saves runner time), switch to fetch-depth 1, extract
  safe.directory to its own step
- itcm_check.yml: fetch-depth 1, drop submodules: recursive (the
  Makefile bootstraps submodules as a prerequisite of board targets)
- sitl_tests.yml, ros_integration_tests.yml, mavros_mission_tests.yml,
  mavros_offboard_tests.yml, python_checks.yml: fetch-depth 1

Each change matches the corresponding job in ci-orchestrator.yml on
mrpollo/ci_orchestration 1:1. Workflows that legitimately need history
(clang-tidy, flash_analysis, failsafe_sim, ros_translation_node,
ekf_*_change_indicator, build_all_targets) are left alone.

Signed-off-by: Ramon Roche <mrpollo@gmail.com>
---
 .github/workflows/checks.yml                | 8 +++++---
 .github/workflows/itcm_check.yml            | 3 +--
 .github/workflows/mavros_mission_tests.yml  | 2 +-
 .github/workflows/mavros_offboard_tests.yml | 2 +-
 .github/workflows/python_checks.yml         | 2 +-
 .github/workflows/ros_integration_tests.yml | 2 +-
 .github/workflows/sitl_tests.yml            | 2 +-
 7 files changed, 11 insertions(+), 10 deletions(-)

diff --git a/.github/workflows/checks.yml b/.github/workflows/checks.yml
index b765fc8ad8..cf44592d10 100644
--- a/.github/workflows/checks.yml
+++ b/.github/workflows/checks.yml
@@ -24,7 +24,7 @@ jobs:
       image: ghcr.io/px4/px4-dev:v1.17.0-rc2
 
     strategy:
-      fail-fast: false
+      fail-fast: true
       matrix:
         check: [
           "check_format",
@@ -43,14 +43,16 @@ jobs:
     steps:
       - uses: actions/checkout@v4
         with:
-          fetch-depth: 0
+          fetch-depth: 1
+
+      - name: Configure Git Safe Directory
+        run: git config --system --add safe.directory '*'
 
       - name: Building [${{ matrix.check }}]
         env:
           PX4_SBOM_DISABLE: 1
         run: |
           cd "$GITHUB_WORKSPACE"
-          git config --system --add safe.directory '*'
           make ${{ matrix.check }}
 
       - name: Uploading Coverage to Codecov.io
diff --git a/.github/workflows/itcm_check.yml b/.github/workflows/itcm_check.yml
index bdecc58406..a48559114a 100644
--- a/.github/workflows/itcm_check.yml
+++ b/.github/workflows/itcm_check.yml
@@ -48,8 +48,7 @@ jobs:
     steps:
       - uses: actions/checkout@v4
         with:
-          fetch-depth: 0
-          submodules: recursive
+          fetch-depth: 1
 
       - name: Git ownership workaround
         run: git config --system --add safe.directory '*'
diff --git a/.github/workflows/mavros_mission_tests.yml b/.github/workflows/mavros_mission_tests.yml
index 29b854b4de..af0ec62ec5 100644
--- a/.github/workflows/mavros_mission_tests.yml
+++ b/.github/workflows/mavros_mission_tests.yml
@@ -26,7 +26,7 @@ jobs:
     steps:
       - uses: actions/checkout@v4
         with:
-          fetch-depth: 0
+          fetch-depth: 1
 
       - name: Build SITL and Run Tests (inside old ROS container)
         run: |
diff --git a/.github/workflows/mavros_offboard_tests.yml b/.github/workflows/mavros_offboard_tests.yml
index e59f340752..da0f3ce1b6 100644
--- a/.github/workflows/mavros_offboard_tests.yml
+++ b/.github/workflows/mavros_offboard_tests.yml
@@ -26,7 +26,7 @@ jobs:
     steps:
       - uses: actions/checkout@v4
         with:
-          fetch-depth: 0
+          fetch-depth: 1
 
       - name: Build SITL and Run Tests (inside old ROS container)
         run: |
diff --git a/.github/workflows/python_checks.yml b/.github/workflows/python_checks.yml
index c0fdc73e8b..265db9bf3c 100644
--- a/.github/workflows/python_checks.yml
+++ b/.github/workflows/python_checks.yml
@@ -18,7 +18,7 @@ jobs:
     steps:
     - uses: actions/checkout@v4
       with:
-        fetch-depth: 0
+        fetch-depth: 1
 
     - name: Install Python3
       run: sudo apt-get install python3 python3-setuptools python3-pip -y
diff --git a/.github/workflows/ros_integration_tests.yml b/.github/workflows/ros_integration_tests.yml
index 973bf6ae29..5421c8b4fa 100644
--- a/.github/workflows/ros_integration_tests.yml
+++ b/.github/workflows/ros_integration_tests.yml
@@ -30,7 +30,7 @@ jobs:
     steps:
     - uses: actions/checkout@v4
       with:
-        fetch-depth: 0
+        fetch-depth: 1
 
     - name: Git Ownership Workaround
       run: git config --system --add safe.directory '*'
diff --git a/.github/workflows/sitl_tests.yml b/.github/workflows/sitl_tests.yml
index 9284a700cf..ba01d2b438 100644
--- a/.github/workflows/sitl_tests.yml
+++ b/.github/workflows/sitl_tests.yml
@@ -41,7 +41,7 @@ jobs:
     steps:
       - uses: actions/checkout@v4
         with:
-          fetch-depth: 0
+          fetch-depth: 1
 
       - name: Git Ownership Workaround
         run: git config --system --add safe.directory '*'