Skip to content

Remove ROS 2 Humble support#792

Merged
JWhitleyWork merged 2 commits into
mainfrom
chore/remove-ros-humble
Jul 17, 2026
Merged

Remove ROS 2 Humble support#792
JWhitleyWork merged 2 commits into
mainfrom
chore/remove-ros-humble

Conversation

@marty-mcfly-bot

@marty-mcfly-bot marty-mcfly-bot Bot commented Jul 17, 2026

Copy link
Copy Markdown

[written by AI]

Summary

  • Remove ROS 2 Humble compatibility branches from the vendored Robotiq driver/controllers and Kinova protective-stop manager.
  • Update the Navigation2 behavior-tree path and continuous-joint controller configuration for Jazzy.
  • Update the Robotiq resource-manager test fixture for Jazzy URDF validation.

Companion to PickNikRobotics/moveit_pro#20494.

needs: moveit_pro/#20494

Dependency

The shared .github/workflows/ci.yaml and Dockerfile changes moved upstream to PickNikRobotics#58. After that PR merges, moveit_pro_example_ws/main should be rebased on moveit_pro_empty_ws/main before this PR is merged.

Verification

  • pre-commit run --all-files
  • Built and tested serial, robotiq_driver, robotiq_controllers, and moveit_studio_kinova_pstop_manager in the Jazzy image from PickNikRobotics/moveit_pro#20494: 29 tests, 0 errors, 0 failures.
  • Verified the configured Jazzy Navigation2 behavior-tree file exists in the paired image and no Humble installation is present.

Release notes

Enhancement: Removed ROS 2 Humble support from the MoveIt Pro example configurations and custom hardware packages, which now target ROS 2 Jazzy.

@marty-mcfly-bot marty-mcfly-bot Bot added this to the 10.0.0 milestone Jul 17, 2026
@coderabbitai

coderabbitai Bot commented Jul 17, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Changes

The PR makes Jazzy the default ROS distribution, updates related configuration and workflow documentation, and removes Humble/Jazzy conditional build and runtime branches from Robotiq and Kinova components.

Jazzy migration

Layer / File(s) Summary
Jazzy defaults and configuration
Dockerfile, .github/workflows/ci.yaml, src/hangar_sim/...
Docker image defaults, CI comments, Nav2 behavior-tree paths, controller configuration, and an integration-test comment are aligned with Jazzy.
Robotiq Jazzy API path
src/external_dependencies/ros2_robotiq_gripper/...
Robotiq build definitions, hardware initialization, activation handling, and test setup now use one unconditional API path.
Kinova controller API path
src/moveit_pro_kinova_configs/...
Protective-stop recovery and its build configuration now use the unified controller request fields without distro-specific compilation.

Possibly related issues

  • PickNikRobotics/moveit_pro#20309 — The PR implements the issue’s Jazzy-default migration by removing Humble-specific build, API, and runtime conditionals.

Possibly related PRs

Suggested reviewers: jwhitleywork


Caution

Pre-merge checks failed

Please resolve all errors before merging. Addressing warnings is optional.

  • Ignore (reviewers only)

❌ Failed checks (1 error)

Check name Status Explanation Resolution
Human Review Check ❌ Error Touches CI/Docker config, spans multiple subsystems, and changes the exported Robotiq on_init signature; not low-risk. This PR requires review by a requested human reviewer. After review, a non-author requested reviewer should override this pre-merge check.
✅ Passed checks (3 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description check ✅ Passed The description matches the changeset, covering Humble-to-Jazzy migration, test updates, CI/Docker notes, and verification steps.

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown

MoveIt Pro Example WS - Objectives Integration Test Report

@github-actions

Copy link
Copy Markdown

MoveIt Pro Example WS - Objectives Integration Test Report

@github-actions

Copy link
Copy Markdown

MoveIt Pro Example WS - Objectives Integration Test Report

@JWhitleyWork
JWhitleyWork merged commit 6cfc8c0 into main Jul 17, 2026
16 checks passed
@JWhitleyWork
JWhitleyWork deleted the chore/remove-ros-humble branch July 17, 2026 21: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.

1 participant