Skip to content

Update Dockerfile and docker-compose to match example_ws#53

Merged
JWhitleyWork merged 1 commit intomainfrom
updates-for-9.0
Feb 13, 2026
Merged

Update Dockerfile and docker-compose to match example_ws#53
JWhitleyWork merged 1 commit intomainfrom
updates-for-9.0

Conversation

@JWhitleyWork
Copy link
Member

Closes PickNikRobotics/moveit_pro#16835.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Aligns this workspace’s Docker/Docker Compose setup with the example_ws template so users get a more up-to-date container setup (including hardware access and development tooling).

Changes:

  • Updated docker-compose.yaml to mount host /dev into drivers and dev services for hardware access.
  • Updated Dockerfile to use a new default base image arg/name, add realtime limits setup, add a realtime group, and install tmux.
  • Removed the GPU-specific build stages from the Dockerfile.

Reviewed changes

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

File Description
Dockerfile Updates base image arg/defaults, adds realtime limits/group handling, tweaks bind mount target, adds tmux, removes GPU stages.
docker-compose.yaml Adds /dev:/dev volume mounts to drivers and dev for host hardware access.

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

@JWhitleyWork JWhitleyWork enabled auto-merge (squash) February 12, 2026 20:04
Copy link
Member

@nbbrooks nbbrooks left a comment

Choose a reason for hiding this comment

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

Diff against example_ws with removed Franka info is empty, lgtm

@JWhitleyWork JWhitleyWork merged commit fd975dc into main Feb 13, 2026
1 check passed
@JWhitleyWork JWhitleyWork deleted the updates-for-9.0 branch February 13, 2026 01:44
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