Skip to content

Switch Tutorial job to target ROS Jazzy to improve stability#3758

Merged
nbbrooks merged 2 commits into
mainfrom
feat-tutorial-ci
Jun 17, 2026
Merged

Switch Tutorial job to target ROS Jazzy to improve stability#3758
nbbrooks merged 2 commits into
mainfrom
feat-tutorial-ci

Conversation

@nbbrooks

@nbbrooks nbbrooks commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

From MoveIt maintainer meeting June 16, Isaac and I decided to have the Tutorial job target ROS jazzy, with a source build of main. The reasons are:

  • Rolling for resolute does not build right now
    • Even if it was, it can be hard to keep up with breaking changes on rolling
  • We do not yet have a MoveIt2 Lyrical release, the most recent LTS distro
  • We enable users to use the latest features on any active distro via a source build of main, so new features (that might be breaking changes) featured the docs can be used on Jazzy

Add table to README describing the use cases for each branch.

Description

Please explain the changes you made, including a reference to the related issue if applicable

Checklist

  • Required by CI: Code is auto formatted using clang-format
  • Extend the tutorials / documentation reference
  • Document API changes relevant to the user in the MIGRATION.md notes
  • Create tests, which fail without this PR reference
  • Include a screenshot if changing a GUI
  • While waiting for someone to review your request, please help review another open pull request to support the maintainers

…tability

Add table to README describing the use cases for each branch.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates MoveIt 2’s tutorial Docker image workflow to build against ROS 2 Jazzy (instead of Rolling) for improved CI stability, and documents a clearer branch usage policy in the README.

Changes:

  • Switch .github/workflows/tutorial_docker.yaml tutorial Docker image build matrix from rolling to jazzy.
  • Add a “Branch Policy” table to README.md describing expected use cases and distro/buildfarm scope per branch.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
README.md Adds a Branch Policy table clarifying branch intent, breaking-change expectations, and distro/buildfarm scope.
.github/workflows/tutorial_docker.yaml Updates the tutorial Docker image build job to target ROS Jazzy instead of Rolling.

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

@nbbrooks nbbrooks changed the title Switch Tutorial job to target building against ROS Jazzy to improve stability Switch Tutorial job to target ROS Jazzy to improve stability Jun 16, 2026

@130s 130s left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There's one suggestion, otherwise LGTM.

Comment thread .github/workflows/tutorial_docker.yaml Outdated
Co-authored-by: Isaac Saito <130s@users.noreply.github.com>

@130s 130s left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not sure yet what the current maintenance rule is about moving forward when some CI jobs are failing, but once again LGTM. Leave the merge decision up to @nbbrooks and other maintainers.

@codecov

codecov Bot commented Jun 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 46.25%. Comparing base (77c5894) to head (1cebccc).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3758      +/-   ##
==========================================
+ Coverage   46.23%   46.25%   +0.03%     
==========================================
  Files         726      726              
  Lines       59501    59513      +12     
  Branches     7624     7624              
==========================================
+ Hits        27505    27523      +18     
+ Misses      31829    31822       -7     
- Partials      167      168       +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@nbbrooks

nbbrooks commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator Author

Some more info on the github action job failures:

  • CI / rolling-resolute (experimental) (pull_request)
    • This job is not required to pass - it is meant to fail to flag deprecations on Rolling (via treat warning as error)
  • SonarScan / rolling-ci-ccov-sonar (push)
    • This job is not required to pass because it has been broken for some time :(
  • Earlier the rolling-ci + ccov job failed on a MoveGroupFixture.MotionPlanRequestRoundTrip flake. I'm compiling a list of these flaky tests

@nbbrooks nbbrooks enabled auto-merge (squash) June 17, 2026 22:37
@nbbrooks nbbrooks disabled auto-merge June 17, 2026 22:37
@nbbrooks nbbrooks merged commit 8758854 into main Jun 17, 2026
16 of 20 checks passed
@nbbrooks nbbrooks deleted the feat-tutorial-ci branch June 17, 2026 22:38
@github-project-automation github-project-automation Bot moved this to ✅ Done in MoveIt Jun 17, 2026
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.

3 participants