Skip to content

Commit

Permalink
fix: Add CI status and change build-check to ci (#833)
Browse files Browse the repository at this point in the history
* fix: Add CI status and change build-check to ci

Align zenoh-pico with the rest of the zenoh ecosystem in terms of CI,
 - rename build-check.yml to ci.yml
 - Include CI status check to ci.yml

* chore: fix test name

* fix: experiment with reusable workflow

* fix: make integration and build-static reusable

* fix: make multicast tests reusable

* fix: typo in job name
  • Loading branch information
diogomatsubara authored Dec 18, 2024
1 parent e40c236 commit 197ab2c
Show file tree
Hide file tree
Showing 5 changed files with 38 additions and 22 deletions.
5 changes: 1 addition & 4 deletions .github/workflows/build-shared.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,10 +14,7 @@
name: build-shared

on:
push:
branches: [ '**' ]
pull_request:
branches: [ '**' ]
workflow_call:

jobs:
build:
Expand Down
5 changes: 1 addition & 4 deletions .github/workflows/build-static.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,10 +14,7 @@
name: build-static

on:
push:
branches: [ '**' ]
pull_request:
branches: [ '**' ]
workflow_call:

jobs:
build:
Expand Down
36 changes: 34 additions & 2 deletions .github/workflows/build-check.yaml → .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,13 +11,15 @@
# Contributors:
# ZettaScale Zenoh Team, <[email protected]>
#
name: build-check
name: CI

on:
push:
branches: ["**"]
pull_request:
branches: ["**"]
schedule:
- cron: "0 6 * * 1-5"

jobs:
run_tests:
Expand Down Expand Up @@ -282,7 +284,7 @@ jobs:
if: always()
run: kill ${{ steps.run-zenoh.outputs.zenohd-pid }}

no_routeur:
no_router:
name: Test examples without router
runs-on: ubuntu-latest
steps:
Expand All @@ -304,3 +306,33 @@ jobs:
with:
config: '.markdownlint.yaml'
globs: '**/README.md'

build_shared:
name: Build shared libs
uses: ./.github/workflows/build-shared.yaml

build_static:
name: Build static libs
uses: ./.github/workflows/build-static.yaml

integration:
name: Run integration tests
uses: ./.github/workflows/integration.yaml

multicast:
name: Run multicast tests
uses: ./.github/workflows/multicast.yaml

# NOTE: In GitHub repository settings, the "Require status checks to pass
# before merging" branch protection rule ensures that commits are only merged
# from branches where specific status checks have passed. These checks are
# specified manually as a list of workflow job names. Thus we use this extra
# job to signal whether all CI checks have passed.
ci:
name: CI status checks
runs-on: ubuntu-latest
needs: [run_tests, check_format, c99_build, raweth_build, zenoh_build, modular_build, unstable_build, st_build, fragment_test, attachment_test, memory_leak_test, no_router, markdown_lint, build_shared, build_static, integration, multicast]
if: always()
steps:
- name: Check whether all jobs pass
run: echo '${{ toJson(needs) }}' | jq -e 'all(.result == "success")'
7 changes: 1 addition & 6 deletions .github/workflows/integration.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,12 +14,7 @@
name: integration

on:
push:
branches: ["**"]
pull_request:
branches: ["**"]
schedule:
- cron: "0 6 * * 1-5"
workflow_call:

jobs:
build:
Expand Down
7 changes: 1 addition & 6 deletions .github/workflows/multicast.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,12 +14,7 @@
name: multicast

on:
push:
branches: ["**"]
pull_request:
branches: ["**"]
schedule:
- cron: "0 6 * * 1-5"
workflow_call:

jobs:
build:
Expand Down

0 comments on commit 197ab2c

Please sign in to comment.