feat(deployment): add CPU mock Compose E2E - #12
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
env,agent, andexperimentas three independent Compose services on a private ROS 2 networkImage boundary
The development container remains the supported source-mounted environment for editing, incremental Colcon builds, lint, and tests. The runtime Dockerfile builds a clean merged install tree in a temporary builder stage and copies only that overlay into the final image.
The pinned upstream
ros:humble-ros-baseimage itself includes some general ROS build utilities. The runtime does not add repository source, Ruff, Colcon build directories, or development caches, and no runtime service performs a build during startup. Removing upstream Colcon packages with APT would also remove required ROS runtime packages, includingrclpy, so PR 12 preserves the base image package graph.Validation
make dev-check— 16 packages, 207 tests, 0 failuresmake mock-e2e— three services discovered, three Episodes committed as expected, container IDs/PIDs stable, restart count 0git diff --check origin/main...HEADScope
CPU mock deployment only. GPU, Isaac Sim, and GUI runtime images remain out of scope for PR 12.