Needs root user to "set up a new integration"

seblucas / alpine-homeassistant

Alpine Homeassistant docker image with some plugins (not all)

GNU General Public License v2.0

16 stars 8 forks source link

Needs root user to "set up a new integration" #5

Closed 2sheds closed 6 years ago

2sheds commented 6 years ago

Since pip runs as root during build, running the resulting container as regular user makes installing new integrations not possible because it tries to use /usr/lib/python3.6 which is not writable. Should running in a virtualenv be considered?

2sheds commented 6 years ago

I tested running in virtualenv - a few little modifications required, no issues found - but I think this is ultimately an overcomplication. I adopted a multistage build that allows adding extra dependencies for HomeKit, Trådfri etc. at build time with very little effort so there's no sense to install them in runtime as container's user. But I'll send a PR with my venv mod for your consideration just in case.