Skip to content

Fix/docker setup update - #298

Open
hello-jkoubi wants to merge 7 commits into
mainfrom
fix/docker-setup-update
Open

Fix/docker setup update#298
hello-jkoubi wants to merge 7 commits into
mainfrom
fix/docker-setup-update

Conversation

@hello-jkoubi

Copy link
Copy Markdown
Contributor

Summary

Update Dockerfile and align image to 0.4.0, primarily updating Stretch Python packages to the latest versions.

Changes

  • Update Stretch Python packages
  • Refresh ROS apt key
  • Remove broken RealSense apt source
  • Remove baked HELLO_FLEET_ID
  • Add user-site Python cleanup step
  • Update stretch_ros2 and rebuild
  • Push updated image to Docker Hub

This ensures the container uses the latest Stretch packages and avoid issues from stale Python environments or broken apt sources.

Comment thread docker/Dockerfile.ros2_patch Outdated
@@ -0,0 +1,73 @@
FROM hellorobotinc/stretch-ai-ros2-bridge:latest

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.

Hey @hello-jkoubi, are Dockerfile.ros2_patch and fix_python_user_site.sh a part of this PR? I don't see where they are being called. Thanks for confirming!

@hello-jkoubi

Copy link
Copy Markdown
Contributor Author

@hello-robot-shehab I updated this PR by modifying Dockerfile.ros2_patch and integrating it into build-robot-docker.sh so the updated image can be built and pushed to Docker Hub.

I also added a docker-compose.yml that launches Terminator, allowing multiple terminal sessions within the container. This makes debugging easier and provides a more convenient way to interact with the container.

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