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

Allow run 'ping' as non-root user #8

Closed
wants to merge 1 commit into from

Conversation

definitio
Copy link
Contributor

There is a problem with alpine images that prevents executing 'ping' as non root user.

@tribut
Copy link
Owner

tribut commented Jun 5, 2020

Thanks for opening this PR. I suggest starting the image with -e "PACKAGES=iputils", which installs the non-busybox version of ping (with the correct permissions). This is hinted at in the README, but maybe you have an idea how to make the documentation clearer?

@tribut tribut closed this Jun 5, 2020
@tribut tribut added enhancement New feature or request invalid This doesn't seem right labels Jun 5, 2020
@definitio
Copy link
Contributor Author

definitio commented Jun 5, 2020

I read the README. Installing packages on start is bad solution because it prevents Home Assistant to start without Internet connection:

[17:09:23] INFO: Creating user homeassistant with 998:997
[17:09:23] INFO: Installing extra packages: iputils
[cont-finish.d] executing container finish scripts...
[cont-finish.d] done.
[s6-finish] waiting for services.
[s6-finish] sending all processes the TERM signal.
[s6-finish] sending all processes the KILL signal and exiting.
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] udev.sh: executing... 
starting version 3.2.9
[17:12:08] INFO: Update udev information
[cont-init.d] udev.sh: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
[17:12:08] INFO: Creating user homeassistant with 998:997
[17:12:08] INFO: Installing extra packages: iputils
[cont-finish.d] executing container finish scripts...
[cont-finish.d] done.
[s6-finish] waiting for services.
[s6-finish] sending all processes the TERM signal.
[s6-finish] sending all processes the KILL signal and exiting.
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] udev.sh: executing... 
starting version 3.2.9
[17:15:15] INFO: Update udev information
[cont-init.d] udev.sh: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
[17:15:15] INFO: Creating user homeassistant with 998:997
[17:15:15] INFO: Installing extra packages: iputils

@tribut
Copy link
Owner

tribut commented Jun 5, 2020

Thanks for bringing this up. It still feels like we should have a more generic way to solve the problem. 🤔

@tribut tribut reopened this Jun 5, 2020
@tribut tribut added bug Something isn't working and removed invalid This doesn't seem right labels Jun 5, 2020
@tribut tribut closed this Jul 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants