-
Notifications
You must be signed in to change notification settings - Fork 668
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
add check for timezone #3153
add check for timezone #3153
Conversation
Signed-off-by: Simon L <szaimen@e.mail.de>
869c55a
to
35c08c0
Compare
maybe also mention, that the timezone should be set inside the aio panel |
and also is doing exit 1 a good idea? |
Good idea!
Yes, I don't think many instances have set this. If there should be many reports, we can still remove the exit later. |
Signed-off-by: Simon L <szaimen@e.mail.de>
This change has caused issues on Unraid. the os adds a TZ to all docker run commands by default, hence preventing the running of the container. |
Interesting! I guess we will need to fix this then in order to not break all unraid instances. |
This is now fixed with v7.1.1 Beta. Testing and feedback is welcome! See https://github.com/nextcloud/all-in-one#how-to-switch-the-channel |
Can confirm this has fixed the issue, warning remains in the logs, but the container starts anyway. |
Close #3147