Skip to content

Commit d135be1

Browse files
committed
ci: builds all trigger on succesful ci orch
Signed-off-by: Ramon Roche <mrpollo@gmail.com>
1 parent 4723a47 commit d135be1

File tree

2 files changed

+9
-28
lines changed

2 files changed

+9
-28
lines changed

.github/workflows/build_all_targets.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ on:
1111
workflows: ["CI Pipeline (Orchestrator)"]
1212
types: [completed]
1313
branches: ['**']
14-
# Direct trigger ONLY for tagged releases (orchestrator doesn't run on tags)
14+
# Direct trigger for tagged releases (orchestrator doesn't run on tags)
1515
push:
1616
tags:
1717
- 'v*'
@@ -32,8 +32,8 @@ jobs:
3232
group_targets:
3333
name: Scan for Board Targets
3434
# Only run if:
35-
# 1. Direct push/tag event, OR
36-
# 2. Orchestrator workflow_run completed successfully, OR
35+
# 1. Direct push to tag (independent trigger for releases), OR
36+
# 2. Orchestrator workflow_run completed successfully (for all branches/PRs), OR
3737
# 3. Manual workflow_dispatch
3838
if: |
3939
github.event_name == 'push' ||

.github/workflows/ci-orchestrator.yml

Lines changed: 6 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -1033,32 +1033,13 @@ jobs:
10331033
10341034
# ═══════════════════════════════════════════════════════════════════
10351035
# TIER 5: Full Build Matrix (30-60 min, AWS 8cpu)
1036-
# Only runs on main/stable/beta/release branches or tags
1037-
# This is the most expensive job - builds all board targets
1036+
# Runs via build_all_targets.yml workflow after orchestrator completes
1037+
# This is the most expensive job - builds all ~200 board targets
10381038
# ═══════════════════════════════════════════════════════════════════
1039-
1040-
build-all-targets-gate:
1041-
name: Full Build Check
1042-
needs: [sitl-tests, ros-integration-tests, mavros-tests, ros-translation-node]
1043-
if: |
1044-
always() &&
1045-
(github.ref == 'refs/heads/main' ||
1046-
github.ref == 'refs/heads/stable' ||
1047-
github.ref == 'refs/heads/beta' ||
1048-
startsWith(github.ref, 'refs/heads/release/') ||
1049-
startsWith(github.ref, 'refs/tags/v')) &&
1050-
needs.sitl-tests.result == 'success' &&
1051-
needs.ros-integration-tests.result == 'success'
1052-
runs-on: ubuntu-latest
1053-
steps:
1054-
- name: Build All Targets Gate Passed
1055-
run: echo "All prerequisites passed, proceeding with full build matrix"
1056-
outputs:
1057-
should_build: ${{ steps.check.outputs.should_build }}
1058-
1059-
# Note: The actual build-all-targets job will be triggered separately
1060-
# by modifying the existing build_all_targets.yml workflow
1061-
# to depend on this orchestrator completing successfully
1039+
# The build_all_targets.yml workflow is triggered automatically when:
1040+
# 1. This orchestrator workflow completes successfully (any branch/PR)
1041+
# 2. A tagged release is pushed (v*)
1042+
# See: .github/workflows/build_all_targets.yml
10621043

10631044
# ═══════════════════════════════════════════════════════════════════
10641045
# SUMMARY JOB

0 commit comments

Comments
 (0)