Implement time zone support.

Author: bovender

Diff for: CHANGELOG.md

@@ -1,5 +1,12 @@
 # dora changelog
 
+## Version 1.2.0 (2020-03-06)
+
+### New feature
+
+- Configure the container's time zone using the `$TIMEZONE` environment
+  variable.
+  
 ## Version 1.1.0 (2020-03-06)
 
 ### New feature

Diff for: Dockerfile

@@ -17,6 +17,7 @@ ENV RAILS_DB_HOST ""
 ENV RAILS_DB_NAME ${APP_NAME}
 ENV RAILS_DB_USER ${APP_NAME}
 ENV RAILS_DB_PASS ""
+ENV TIMEZONE="UCT"
 ENV WKHTMLTOPDF ""
 ENV WKHTMLTOPDF_URL "https://github.com/wkhtmltopdf/wkhtmltopdf/releases/download/0.12.5/wkhtmltox_0.12.5-1.bionic_amd64.deb"
 ENV WKHTMLTOPDF_SUM "db48fa1a043309c4bfe8c8e0e38dc06c183f821599dd88d4e3cea47c5a5d4cd3"
@@ -30,7 +31,8 @@ RUN /pd_build/nodejs.sh
 # Install yarn
 RUN curl -sS https://dl.yarnpkg.com/debian/pubkey.gpg | apt-key add - &&\
     echo "deb https://dl.yarnpkg.com/debian/ stable main" > /etc/apt/sources.list.d/yarn.list &&\
-    apt-get update && apt-get install yarn
+    apt-get update &&\
+    apt-get install -y --no-install-recommends yarn tzdata
 
 # This is from passenger-docker's README.
 ENV HOME /root
@@ -57,6 +59,8 @@ ADD bootstrap-container.sh /etc/my_init.d/10_bootstrap_container.sh
 RUN chmod +x /etc/my_init.d/10_bootstrap_container.sh
 ADD install-wkhtmltopdf.sh /etc/my_init.d/90_install_wkhtmltopdf.sh
 RUN chmod +x /etc/my_init.d/90_install_wkhtmltopdf.sh
+ADD set-timezone.sh /etc/my_init.d/01_set_timezone.sh
+RUN chmod +x /etc/my_init.d/01_set_timezone.sh
 
 RUN mkdir -p /etc/service/sidekiq
 ADD run-sidekiq.sh /etc/service/sidekiq/run

Diff for: README.md

@@ -42,6 +42,7 @@ Customization is mostly done with environment variables.
 | `RAILS_SMTP_USER` | SMTP user name | `$APP_NAME`
 | `RAILS_SMTP_PASS` | SMTP password |
 | `SECRET_KEY_BASE` | Rails' secret key base |
+| `TIMEZONE` | Time zone of the container | `UCT`
 | `NO_WKHTMLTOPDF` | Do not attempt to install [wkhtmltopdf][] | (empty)
 | `WKHTMLTOPDF_URL` | Download URL for [wkhtmltopdf][] | 
 
@@ -332,6 +333,19 @@ via environment variables. A `.env` file lends itself well to this
 configuration. The composition consists of the rails app, Postgres, and Redis.
 See `sample.env` for usage instructions.
 
+## Container time zone
+
+`passenger-docker` does not configure a time zone for the container. Dora does
+do it by installing the `tzdata` package and supporting a `$TIMEZONE` variable.
+This variable _must_ be set to a directory and file unter `/usr/share/zoneinfo`,
+e.g. `Europe/Berlin`.
+
+To see all possible values for `$TIMEZONE`, issue:
+
+```bash
+find /usr/share/zoneinfo -follow | sed -E 's_(/[^/]+){3}/__'
+```
+
 ## Troubleshooting
 
 ### Sending mail

Diff for: docker-compose.yml

@@ -24,6 +24,7 @@ services:
       - RAILS_DB_NAME
       - RAILS_DB_USER
       - RAILS_DB_PASS
+      - TIMEZONE
     depends_on:
       - db
       - redis

Diff for: dora-banner.sh

@@ -29,6 +29,7 @@ echo "= RAILS_DB_PASS:             $RAILS_DB_PASS"
 echo "= RAILS_SMTP_HOST:           $RAILS_SMTP_HOST"
 echo "= RAILS_SMTP_USER:           $RAILS_SMTP_USER"
 echo "= RAILS_SMTP_PASS:           $RAILS_SMTP_PASS"
+echo "= TIMEZONE:                  $TIMEZONE"
 echo "= NO_WKHTMLTOPDF:            $NO_WKHTMLTOPDF"
 echo "= \`which wkhtmltopdf\`:       $(which wkhtmltopdf)"
 echo

Diff for: set-timezone.sh

@@ -0,0 +1,6 @@
+#!/usr/bin/env bash
+
+if [ "$TIMEZONE" != "" ]; then
+  echo "Setting time zone to $TIMEZONE"
+  ln -sf "/usr/share/zoneinfo/$TIMEZONE" /etc/localtime
+fi