Skip to content

ci: trigger builds on ros2_medkit_updated dispatch#36

Merged
bburda merged 3 commits intomainfrom
feat/repository-dispatch-trigger
Mar 3, 2026
Merged

ci: trigger builds on ros2_medkit_updated dispatch#36
bburda merged 3 commits intomainfrom
feat/repository-dispatch-trigger

Conversation

@bburda
Copy link
Contributor

@bburda bburda commented Mar 3, 2026

Description

Adds a repository_dispatch trigger (ros2_medkit_updated event type) so ros2_medkit CI can notify selfpatch_demos to run a docker-build verification whenever ros2_medkit merges to main. Without this, breakages in ros2_medkit that affect the demo images go undetected until someone manually runs selfpatch_demos CI.

Also adds a Show triggering source step as the first step in the docker-build job. When triggered by repository_dispatch, it writes a step summary with the source commit SHA and a link to the ros2_medkit run - making it easy to trace a failed demo build back to the ros2_medkit change that caused it.

Related Issue

closes selfpatch/ros2_medkit#224

Checklist

  • Tested locally
  • README updated (if needed)

Copilot AI review requested due to automatic review settings March 3, 2026 07:45
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates CI to allow ros2_medkit to trigger selfpatch_demos docker image build verification via repository_dispatch, and adds a small traceability summary when that happens. Also adjusts demo runner scripts to consolidate docker-compose invocation and adds cleanup behavior on build failure, plus aligns the MoveIt demo Dockerfile with the other demos’ ROS2MedkitCompat.cmake handling.

Changes:

  • Add repository_dispatch trigger (ros2_medkit_updated) to CI and show source metadata in the job summary when dispatch-triggered.
  • Refactor demo run-demo.sh scripts to select a COMPOSE_CMD and add a “build failed” cleanup path.
  • Update MoveIt Pick-and-Place Dockerfile to install ROS2MedkitCompat.cmake into ${COLCON_WS}/cmake.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
demos/turtlebot3_integration/run-demo.sh Consolidates compose command selection; adds cleanup on build failure.
demos/sensor_diagnostics/run-demo.sh Same compose refactor; adds cleanup on build failure (with a ShellCheck concern).
demos/moveit_pick_place/run-demo.sh Same compose refactor; adds cleanup on build failure.
demos/moveit_pick_place/Dockerfile Moves ROS2MedkitCompat.cmake into workspace cmake dir like other demos.
.github/workflows/ci.yml Adds repository_dispatch trigger and dispatch-only “source” step summary output.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@bburda bburda force-pushed the feat/repository-dispatch-trigger branch from c170348 to 20998f2 Compare March 3, 2026 07:54
@bburda bburda marked this pull request as draft March 3, 2026 08:39
@bburda bburda requested a review from Copilot March 3, 2026 16:26
@bburda bburda self-assigned this Mar 3, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@bburda bburda marked this pull request as ready for review March 3, 2026 16:34
@bburda bburda requested a review from mfaferek93 March 3, 2026 16:45
@bburda bburda merged commit b1af18d into main Mar 3, 2026
2 checks passed
@bburda bburda deleted the feat/repository-dispatch-trigger branch March 3, 2026 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Trigger selfpatch_demos CI on merge to main

3 participants