Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

run-dev.sh does not pass RMW_IMPLEMENTATION to docker container #156

Open
delenius opened this issue Nov 8, 2024 · 2 comments
Open

run-dev.sh does not pass RMW_IMPLEMENTATION to docker container #156

delenius opened this issue Nov 8, 2024 · 2 comments
Assignees
Labels
enhancement New feature or request verify to close Waiting on confirm issue is resolved

Comments

@delenius
Copy link

delenius commented Nov 8, 2024

In run-dev.sh, we have
DOCKER_ARGS+=("-e ROS_DOMAIN_ID")
which is good. But it would be great to also have
DOCKER_ARGS+=("-e RMW_IMPLEMENTATION")
in case the host uses a non-default one (in my case, I have found that rmw_cyclonedds_cpp usually works better).

@hemalshahNV hemalshahNV self-assigned this Nov 8, 2024
@hemalshahNV hemalshahNV added enhancement New feature or request verify to close Waiting on confirm issue is resolved labels Nov 8, 2024
@hemalshahNV
Copy link
Contributor

You can specify additional Docker run args using "-a | --docker_arg" like this one. No harm in adding RMW_IMPLEMENTATION if it has been defined in the host environment, I suppose.

@jekizen
Copy link

jekizen commented Nov 20, 2024

I installed on the home system: export ROS_DOMAIN_ID= <my_ID>, after running run_dev.sh, inside docker: echo $ROS_DOMAIN_ID was my_ID

docker must be running as root:
sudo systemctl enable docker.service
sudo systemctl enable docker.socket
systemctl --user disable docker.service
sudo systemctl start docker.service
sudo systemctl start docker.socket
sudo usermod -aG docker $USER

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request verify to close Waiting on confirm issue is resolved
Projects
None yet
Development

No branches or pull requests

3 participants