ppodgorsek / docker-robot-framework

Robot Framework in Docker
https://cloud.docker.com/repository/docker/ppodgorsek/robot-framework
MIT License
333 stars 235 forks source link

Allow /home directory for user in base image #410

Closed hajdukd closed 5 months ago

hajdukd commented 1 year ago

Is your feature request related to a problem? Please describe. I know that dependencies should be built into base image, but for development purposes its sometimes useful to allow install of dependencies for 1000 user. Home directory is required for that.

Describe the solution you'd like Please add 1000 user and group as regular named user and groups.

Describe alternatives you've considered Bulding everything into the image, which is not really developer friendly.

Additional context N/A

ppodgorsek commented 1 year ago

If you need to install dependencies during runtime, the best practice would be to mount a volume, in which the dependencies will be installed. This will avoid those dependencies having to be downloaded every time, hence speeding up your pipeline.

You can change the user running the container with the --user=1000:1000 command line argument.

hajdukd commented 1 year ago

Hi, This won't work as volumes are not shared between CI runners nor runners that are spawned when needed. So you would install them on volume every time over and over. It's also highly dependent on the CI engine you are using.

ps. This feature is just a sugestion. One can always build own image with required changes. Feel free to close if you think it's a bad idea.

background. Was to provide a way for feature-branches to test various dependencies if needed without having to modify anything in the pipeline or image or volumes etc.

ppodgorsek commented 5 months ago

The feature allowing to dynamically install additional dependencies has finally been implemented as part of #376 and the details can be found in the README.