ekumenlabs / terminus

Library to create cities and terrains
Apache License 2.0
56 stars 29 forks source link

Update docker image with driver permissions #156

Closed agalbachicar closed 7 years ago

agalbachicar commented 7 years ago

Update the base Dockerfile adding permissions to the file /dev/dri/card0 to the user gazebo.

agalbachicar commented 7 years ago

@Shokman , I made some tests on how to change permissions to the file /dev/dri/card0 so as gazebo user can load Gazebo without problems, but this file does not exists in Docker compilation time so any command trying to modify it will fail.

I suggest creating a script, which is run at the beginning, and copy to the image at compilation time. Then, the first thing to do will be changing the permissions to that file and then running tmux. Do you agree?

basicNew commented 7 years ago

@agalbachicar do you have an update the status of this one?

agalbachicar commented 7 years ago

@basicNew I have done some preliminary tests on this. We cannot add a command to the Dockerfile to change the permissions to this file, because at docker build time it does not exist. Some possible solutions to this are:

The use of a startup script will have two uses now, one will be change the permissions of that file and the other is to source the variables in setup.bash file.

basicNew commented 7 years ago

Thanks for the update @agalbachicar . Let's tackle this on next sprint so we can forget about this issue.

basicNew commented 7 years ago

Btw @agalbachicar , shouldn't we move this issue to the BB repo now that we are doing the reorganization?

agalbachicar commented 7 years ago

@basicNew you're right. I'll move it to the new repository and I'll try to make a revision of the docker related issues too so we don't lose the history.

basicNew commented 7 years ago

Great, thanks!

agalbachicar commented 7 years ago

This issue has been moved to https://bitbucket.org/ekumen/terminus-simulation/issues/85