Rename agent_bridge Compose service to runtime - #62
Merged
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughSummary by CodeRabbit
WalkthroughChangesCompose service configuration
Suggested reviewers: 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
Comment |
L4co77
marked this pull request as ready for review
August 3, 2026 19:47
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.
[written by AI]
Motivation
Follow-up to PickNikRobotics/moveit_pro#20893, which renames the
agent_bridgeDocker Compose service and its image toruntime. A user workspace'sdocker-compose.yamlis merged with the one shipped in the MoveIt Pro image and mirrors its service names, so the rename has to happen on this side too or the stanza no longer matches any base service.Brief description
Renames the
agent_bridgeservice stanza toruntime. That is the only occurrence in this repo — there are no config packages here, so noagent_bridge.launch.xmlto rename.How it was tested
grep -rn agent_bridgeover the repo returns nothing after the change. Nothing else in this workspace references the service name.Coordination
Merge PickNikRobotics/moveit_pro#20893 first. Until it lands, the MoveIt Pro image still defines
agent_bridge, so this branch'sruntime: {}is an override with no matching base service.