Setting additional_env in a ComposableNodeContainer will set the launch's environment as well
Open
@methylDragon is already working on this.
Since Jul 6, 2023.
- Dominant language
- Python
- Stars
- 155
- Forks
- 182
- Avg merge
- 2d 14h
- Merged PRs (30d)
- 6
Description
Bug report
Required Info:
- Operating System:
- Ubuntu 22.04
- Installation type:
- Binaries
- Version or commit hash:
- humble
- DDS implementation:
- CycloneDDS
- Client library (if applicable):
- rclcpp
Steps to reproduce issue
- Create 2 CycloneDDS profiles where each has its own unique participant ID.
- Set environment variable for
CYCLONEDDS_URI=file://<profile-path>. - Create a launch file which runs a ComposableNodeContainer with some components (as seem in this example
- **If you try to run this now, it will fail because both the ComposableNodeContainer and the ROS2 temporary node which is used to load components are trying to use the same ports. **
- Set
additional_enve.g.:{"CYCLONEDDS_URI": "file://<path-to-other-profile>} - Run the launch file.
Expected behavior
The composable node container should match the ports of the second profile (step 5), while the ROS2 node which send the LoadNode requests use the ports matching the ports of the first profile (step 2).
Actual behavior
You will get the same error as in step 4, but this time the ports match the ports of the second profile. This actually means that you can't run a composable node container with pre-defined participant ID.
Additional information
This worked fine in galactic, and when we try to port to humble, we stumbled upon this issue
Thank you
Omer
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.