Merge branch 'release-3.0.0' into stable

Author: BertrandGouny

CHANGELOG.md

@@ -4,6 +4,13 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project follows Zammad versioning.
 
+## [3.0.0] - 2019-06-10
+### Changed
+  - Upgrade Zammad version to 3.0.0
+
+### Fixed
+  - Shellcheck all scripts
+
 ## [2.9.0] - 2019-02-20
 ### Changed
   - Upgrade Zammad version to 2.9.0

Makefile

@@ -1,5 +1,5 @@
 NAME = osixia/zammad
-VERSION = 2.9.0
+VERSION = 3.0.0
 
 .PHONY: build build-nocache test tag-latest push push-latest release git-tag-version
 

README.md

@@ -4,7 +4,7 @@
 ![Docker Stars](https://img.shields.io/docker/stars/osixia/zammad.svg)
 ![](https://images.microbadger.com/badges/image/osixia/zammad.svg)
 
-Latest release: 2.9.0 - Zammad 2.9.0 - [Changelog](CHANGELOG.md) | [Docker Hub](https://hub.docker.com/r/osixia/zammad/) 
+Latest release: 3.0.0 - Zammad 3.0.0 - [Changelog](CHANGELOG.md) | [Docker Hub](https://hub.docker.com/r/osixia/zammad/) 
 
 **A docker image to run Zammad.**
 

example/docker-compose.yml

@@ -32,7 +32,7 @@ services:
       - mariadb:/var/lib/mysql
 
   zammad:
-    image: osixia/zammad:2.9.0
+    image: osixia/zammad:3.0.0
     command: -l info
     labels:
       - "io.rancher.container.pull_image: always"

example/kubernetes/zammad-deployment.yaml

@@ -15,7 +15,7 @@ spec:
       automountServiceAccountToken: false
       containers:
         - name: zammad
-          image: osixia/zammad:2.9.0
+          image: osixia/zammad:3.0.0
           imagePullPolicy: Always
           resources:
             requests:

image/Dockerfile

@@ -4,8 +4,8 @@ ENV ZAMMAD_DIR /home/zammad
 ENV ZAMMAD_USER zammad
 ENV RAILS_ENV production
 
-ARG ZAMMAD_VERSION=2.9.0
-ARG ZAMMAD_MD5=08bd2f55770f23ee780a7dbd5c8c4cc8
+ARG ZAMMAD_VERSION=3.0.0
+ARG ZAMMAD_MD5=5170cfc115ee90e853c1180578fc30da
 
 ARG GOSU_VERSION=1.10
 

image/service/backup/assets/tool/zammad-backup

@@ -6,29 +6,27 @@ backupFileSuffix="-zammad.tar.gz"
 backupDbSuffix="-zammad-db.tar.gz"
 databaseFile="database.sql"
 
-source /container/run/environment.sh
+. /container/run/environment.sh
 
 # delete backups that are over $ZAMMAD_BACKUP_TTL days
-find $backupPath -type f -mtime +$ZAMMAD_BACKUP_TTL -exec rm {} \;
+find "${backupPath}" -type f -mtime +"${ZAMMAD_BACKUP_TTL}" -exec rm {} \;
 
 # date format for the dump file name
 dateFileFormat="+%Y%m%dT%H%M%S"
-backupFilePath="$backupPath/$(date "$dateFileFormat")$backupFileSuffix"
+backupFilePath="${backupPath}/$(date "${dateFileFormat}")${backupFileSuffix}"
 
 # save config and plugins except default ones
-tar -czf $backupFilePath -C / ${ZAMMAD_DIR:1}
-chmod 600 $backupFilePath
+tar -czf "${backupFilePath}" -C / "${ZAMMAD_DIR:1}"
+chmod 600 "${backupFilePath}"
 
 # backup database
 . /container/service/backup/assets/tool/zammad-get-db-params
-rm -rf $databaseFile || true
+rm -rf "${databaseFile}" || true
 
-mysqldump -u $databaseUser -p$databasePassword --host $host $database > $databaseFile
+mysqldump -u "${databaseUser}" -p"${databasePassword}" --host "${host}" "${database}" > "${databaseFile}"
 
-backupFilePath="$backupPath/$(date "$dateFileFormat")$backupDbSuffix"
-tar -czf $backupFilePath $databaseFile
-chmod 600 $backupFilePath
+backupFilePath="${backupPath}/$(date "${dateFileFormat}")${backupDbSuffix}"
+tar -czf "${backupFilePath}" "${databaseFile}"
+chmod 600 "${backupFilePath}"
 
-rm -rf $databaseFile
-
-exit 0
+rm -rf "${databaseFile}"

image/service/backup/assets/tool/zammad-get-db-params

@@ -3,7 +3,7 @@
 # get database name, user and password from configuration
 # /!\ configuration must use simple quote :)
 # and it's a bad idea to have ' in your username, password and database name
-host="$(grep 'host:' < ${ZAMMAD_DIR}/config/database.yml | sed -e 's/.*host:\(\| \)//g')"
-databaseUser="$(grep 'username:' < ${ZAMMAD_DIR}/config/database.yml | sed -e 's/.*username:\(\| \)//g')"
-databasePassword="$(grep 'password:' < ${ZAMMAD_DIR}/config/database.yml | sed -e 's/.*password:\(\| \)//g')"
-database="$(grep 'database:' < ${ZAMMAD_DIR}/config/database.yml | sed -e 's/.*database:\(\| \) //g')"
+host="$(grep 'host:' < "${ZAMMAD_DIR}/config/database.yml" | sed -e 's/.*host:\(\| \)//g')"
+databaseUser="$(grep 'username:' < "${ZAMMAD_DIR}/config/database.yml" | sed -e 's/.*username:\(\| \)//g')"
+databasePassword="$(grep 'password:' < "${ZAMMAD_DIR}/config/database.yml" | sed -e 's/.*password:\(\| \)//g')"
+database="$(grep 'database:' < "${ZAMMAD_DIR}/config/database.yml" | sed -e 's/.*database:\(\| \) //g')"

image/service/backup/assets/tool/zammad-restore

@@ -8,26 +8,26 @@ databaseFile="database.sql"
 
 for file in "$@"
 do
-    echo "Processing file $file"
-
-    if $(echo "$file" | grep -q -e "$backupFileSuffix"); then
+    echo "Processing file ${file}"
+    
+    if echo "${file}" | grep -q -e "$backupFileSuffix"; then
         echo "Restore zammad files"
-        tar -xvzf $backupPath/$file -C /
-        echo "done :)"
-    elif $(echo "$file" | grep -q -e "$backupDbSuffix"); then
+        tar -xvzf "${backupPath}/${file}" -C /
+        echo "done :)"    
+    elif echo "${file}" | grep -q -e "${backupDbSuffix}"; then
         echo "Restore zammad database"
         . /container/service/backup/assets/tool/zammad-get-db-params
-
-        rm -rf $databaseFile || true
-        tar -xvzf $backupPath/$file
-
-        mysql -u $databaseUser -p$databasePassword --host $host $database < $databaseFile
-
-        rm -rf $databaseFile
-
+        
+        rm -rf "${databaseFile}" || true
+        tar -xvzf "${backupPath}/${file}"
+        
+        mysql -u "${databaseUser}" -p"${databasePassword}" --host "${host}" "${database}" < "${databaseFile}"
+        
+        rm -rf "${databaseFile}"
+        
         echo "done :)"
     else
-      echo "Error: Unknown file type"
+        echo "Error: Unknown file type"
     fi
 done
 

image/service/backup/startup.sh

@@ -5,20 +5,18 @@
 log-helper level eq trace && set -x
 
 # install image tools
-ln -sf ${CONTAINER_SERVICE_DIR}/backup/assets/tool/* /sbin/
+ln -sf "${CONTAINER_SERVICE_DIR}"/backup/assets/tool/* /sbin/
 
 # add cron jobs
-ln -sf ${CONTAINER_SERVICE_DIR}/backup/assets/cronjobs /etc/cron.d/backup
-chmod 600 ${CONTAINER_SERVICE_DIR}/backup/assets/cronjobs
+ln -sf "${CONTAINER_SERVICE_DIR}/backup/assets/cronjobs" /etc/cron.d/backup
+chmod 600 "${CONTAINER_SERVICE_DIR}/backup/assets/cronjobs"
 
 FIRST_START_DONE="${CONTAINER_STATE_DIR}/docker-backup-backup-first-start-done"
 # container first start
-if [ ! -e "$FIRST_START_DONE" ]; then
-
-  # adapt cronjobs file
-  sed -i "s|{{ ZAMMAD_BACKUP_CRON_EXP }}|${ZAMMAD_BACKUP_CRON_EXP}|g" ${CONTAINER_SERVICE_DIR}/backup/assets/cronjobs
-
-  touch $FIRST_START_DONE
+if [ ! -e "${FIRST_START_DONE}" ]; then
+    
+    # adapt cronjobs file
+    sed -i "s|{{ ZAMMAD_BACKUP_CRON_EXP }}|${ZAMMAD_BACKUP_CRON_EXP}|g" "${CONTAINER_SERVICE_DIR}/backup/assets/cronjobs"
+    
+    touch "${FIRST_START_DONE}"
 fi
-
-exit 0