Skip to content

Recommend APT package as default installation method. Update CI to use new Rolling. - #39

Merged
sgillen merged 5 commits into
NVIDIA-ISAAC-ROS:mainfrom
bmchalenv:update-readme
Aug 12, 2026
Merged

Recommend APT package as default installation method. Update CI to use new Rolling.#39
sgillen merged 5 commits into
NVIDIA-ISAAC-ROS:mainfrom
bmchalenv:update-readme

Conversation

@bmchalenv

@bmchalenv bmchalenv commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

The APT package was released but the README.md was never updated to recommend its use.

Also forced to update CI to use the newest Rolling which is on Ubuntu resolute and its apt packages are found on the ros2-testing apt server.

Signed-off-by: Blake McHale <bmchale@nvidia.com>
@greptile-apps

greptile-apps Bot commented Aug 11, 2026

Copy link
Copy Markdown

Greptile Summary

The PR makes the released APT package the recommended installation method and updates Rolling support to Ubuntu Resolute with the ROS 2 testing repository.

  • Adds an APT installation command and ROS environment prerequisite.
  • Moves Rolling CI and Docker targets from Noble to Resolute.
  • Configures Rolling package builds and tests to use ros2-testing.
  • Adds Resolute support to Debian package generation.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
README.md Recommends APT installation, requires the intended ROS environment to be sourced, and removes the incorrect Jazzy fallback.
.github/workflows/debian-packages.yml Moves Rolling package generation and smoke tests to Resolute and uses the ROS 2 testing repository.
.github/workflows/ros-tests.yml Updates the Rolling test target to Resolute with packages sourced from ros2-testing.
scripts/build_debian_packages.sh Adds Resolute packaging support and selects ros2-testing when building Rolling packages.
scripts/docker-test.sh Updates the Rolling development container image from Noble to Resolute.

Reviews (5): Last reviewed commit: "Use testing apt repo for ROS Rolling" | Re-trigger Greptile

Comment thread README.md
Signed-off-by: Blake McHale <bmchale@nvidia.com>
Comment thread README.md Outdated
Signed-off-by: Blake McHale <bmchale@nvidia.com>
Signed-off-by: Blake McHale <bmchale@nvidia.com>
Signed-off-by: Blake McHale <bmchale@nvidia.com>
@bmchalenv bmchalenv changed the title Recommend APT package as default installation method Recommend APT package as default installation method. Update CI to use new Rolling. Aug 11, 2026
@sgillen
sgillen merged commit 94c3927 into NVIDIA-ISAAC-ROS:main Aug 12, 2026
18 checks passed
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.

2 participants