Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pin demos to humble-2024.10.0 release (#197) #210

Merged
merged 2 commits into from
Feb 1, 2025

Conversation

ivanperez-keera
Copy link
Contributor

The current dockerfile of the MoveIt 2 and navigation2 demo depends on space-ros:latest. This may prevent the image from building if we release a new version of Space ROS but we have not yet adapted the either demo as needed, breaking the build.

This commit pins the version of both demos to the latest version of Space ROS that we have tested it with (humble-2024.10.0).

The current dockerfile of the moveit2 demo depends on space-ros:latest.  This
may prevent the image from building if we release a new version of Space ROS
but we have not yet adapted the MoveIt 2 demo as needed, breaking the build.

This commit pins the version of the MoveIt 2 demo to the latest version of
Space ROS that we have tested it with (humble-2024.10.0). We modify both the
Dockerfile and the build script so that the result is the same (and thus
unsurprising) no matter how the image is built.
@Bckempa Bckempa added this to the jazzy-2025.01.0 milestone Jan 31, 2025
The current dockerfile of the navigation2 demo depends on space-ros:latest.
This may prevent the image from building if we release a new version of Space
ROS but we have not yet adapted the Navigation 2 demo as needed, breaking the
build.

This commit pins the version of the Navigation 2 demo to the latest version of
Space ROS that we have tested it with (humble-2024.10.0). We modify both the
Dockerfile and the build script so that the result is the same (and thus
unsurprising) no matter how the image is built.
@Bckempa Bckempa linked an issue Jan 31, 2025 that may be closed by this pull request
@ivanperez-keera ivanperez-keera force-pushed the pin-demos-spaceros-humble branch from 7431652 to 7126d6d Compare January 31, 2025 23:18
@Bckempa
Copy link
Contributor

Bckempa commented Jan 31, 2025

Thanks,confirmed this pins both the moveit and nav stacks to the last known working version which will implicitly keep space robots on a working config.

@Bckempa
Copy link
Contributor

Bckempa commented Jan 31, 2025

@EzraBrooks, FYI one of the required CI checks didn't auto-fire. I manually started the space_robots build for this branch but the merge UI isn't linking the build in progress so I'm not sure if it will take. Guess we'll find out in 2 hours?

@Bckempa
Copy link
Contributor

Bckempa commented Feb 1, 2025

There is a successful build of this, but it isn't acknowledged by the merge requirements:
image

@Bckempa Bckempa closed this Feb 1, 2025
@Bckempa Bckempa reopened this Feb 1, 2025
@Bckempa Bckempa merged commit 1f4f847 into main Feb 1, 2025
7 checks passed
@Bckempa Bckempa deleted the pin-demos-spaceros-humble branch February 1, 2025 05:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Pin Space ROS Version Used by Dockerfiles
2 participants