fix(Dockerfile): Fix permission issue for startup script #58
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The startup script could not be started (in my case) because of permission issues. Looking at the log, I saw: ``` rssreader | [s6-init] making user provided files available at /var/run/s6/etc...exited 0. rssreader | [s6-init] ensuring user provided files have correct perms...exited 0. rssreader | [fix-attrs.d] applying ownership & permissions fixes... rssreader | [fix-attrs.d] done. rssreader | [cont-init.d] executing container initialization scripts... rssreader | [cont-init.d] 50-php: executing... rssreader | [cont-init.d] 50-php: exited 0. rssreader | [cont-init.d] 98-wait-for-db: executing... rssreader | Checking database responds within 1s on postgres:5432... rssreader | [cont-init.d] 98-wait-for-db: exited 0. rssreader | [cont-init.d] 99-ttrss: executing... rssreader | /var/run/s6/etc/cont-init.d/99-ttrss: line 2: ./setup-ttrss.sh: Permission denied rssreader | [cont-init.d] 99-ttrss: exited 126. rssreader | [cont-init.d] done. rssreader | [services.d] starting services rssreader | ./run: cd: line 4: can't cd to /var/www/ttrss: No such file or directory rssreader | nginx: [warn] the "ssl" directive is deprecated, use the "listen ... ssl" directive instead in /etc/nginx/nginx.conf:39 ``` By passing `--chown`, we set the file owner to what s6 expects.
|
FYI @x86dev : Even if I fix this issue, I still get an issue with FPM: Did you encounter that before? |
|
@bugwelle add -F to the exec php-fpm in the php run file, so it doesn't detach and s6 can manage it. alpine changed default behaviour with 8 so needs some tweaking, see my fork https://github.com/aashley/docker-ttrss i've made a stack of changes to how the fpm runs and its all good |
|
@aashley Awesome! Thanks. :) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The startup script could not be started (in my case) because of permission issues. Looking at the log, I saw:
By passing
--chown, we set the file owner to what s6 expects.