Improve docker image for database backup/restore

* Decrease the image size by not installing unnecessary packages.
* Move the option 'backup_docker_image' to separate database config
  section.

Change-Id: I3362e1f8a7dc21b89c060352f2db2f138c16db07

2 files changed, 35 insertions, 24 deletions

diff --git a/backup/Dockerfile b/backup/Dockerfile
index c260e50b..86c19ede 100644
--- a/backup/Dockerfile
+++ b/backup/Dockerfile
@@ -1,41 +1,26 @@
 FROM ubuntu:18.04
 LABEL maintainer="anlin.kong@gmail.com"
 
+ARG DATASTORE="mysql"
 ARG APTOPTS="-y -qq --no-install-recommends --allow-unauthenticated"
 ARG PERCONA_XTRABACKUP_VERSION=24
-ENV DEBIAN_FRONTEND noninteractive
-ENV APT_KEY_DONT_WARN_ON_DANGEROUS_USAGE=1
+ENV DEBIAN_FRONTEND noninteractive \
+    APT_KEY_DONT_WARN_ON_DANGEROUS_USAGE=1
 
 RUN apt-get update \
     && apt-get install $APTOPTS gnupg2 lsb-release apt-utils apt-transport-https ca-certificates software-properties-common curl \
     && apt-get clean \
     && rm -rf /var/lib/apt/lists/*
 
-# Install percona-xtrabackup for mysql
-RUN curl -sSL https://repo.percona.com/apt/percona-release_latest.$(lsb_release -sc)_all.deb -o percona-release.deb \
-    && dpkg -i percona-release.deb \
-    && percona-release enable-only tools release \
-    && apt-get update \
-    && apt-get install $APTOPTS percona-xtrabackup-${PERCONA_XTRABACKUP_VERSION} \
-    && apt-get clean
-
-# Install mariabackup for mariadb
-Run apt-key adv --fetch-keys 'https://mariadb.org/mariadb_release_signing_key.asc' \
-    && add-apt-repository "deb [arch=amd64] http://mirror2.hs-esslingen.de/mariadb/repo/10.4/ubuntu $(lsb_release -cs) main" \
-    && apt-get update \
-    && apt-get install $APTOPTS mariadb-backup \
-    && apt-get clean
-
-RUN apt-get update \
-    && apt-get install $APTOPTS build-essential python3-setuptools python3-all python3-all-dev python3-pip libffi-dev libssl-dev libxml2-dev libxslt1-dev libyaml-dev \
-    && apt-get clean
-
 COPY . /opt/trove/backup
 WORKDIR /opt/trove/backup
+RUN ./install.sh $DATASTORE ${PERCONA_XTRABACKUP_VERSION}
 
-RUN pip3 --no-cache-dir install -U -r requirements.txt
-
-RUN curl -sSL https://github.com/Yelp/dumb-init/releases/download/v1.2.2/dumb-init_1.2.2_amd64 -o /usr/local/bin/dumb-init \
+RUN apt-get update \
+    && apt-get install $APTOPTS build-essential python3-setuptools python3-all python3-all-dev python3-pip libffi-dev libssl-dev libxml2-dev libxslt1-dev libyaml-dev \
+    && apt-get clean \
+    && pip3 --no-cache-dir install -U -r requirements.txt \
+    && curl -sSL https://github.com/Yelp/dumb-init/releases/download/v1.2.2/dumb-init_1.2.2_amd64 -o /usr/local/bin/dumb-init \
     && chmod +x /usr/local/bin/dumb-init
 
 ENTRYPOINT ["dumb-init", "--single-child", "--"]
diff --git a/backup/install.sh b/backup/install.sh
new file mode 100755
index 00000000..ad1c2e4a
--- /dev/null
+++ b/backup/install.sh
@@ -0,0 +1,26 @@
+#!/usr/bin/env bash
+set -e
+
+APTOPTS="-y -qq --no-install-recommends --allow-unauthenticated"
+
+case "$1" in
+"mysql")
+    curl -sSL https://repo.percona.com/apt/percona-release_latest.$(lsb_release -sc)_all.deb -o percona-release.deb
+    dpkg -i percona-release.deb
+    percona-release enable-only tools release
+    apt-get update
+    apt-get install $APTOPTS percona-xtrabackup-$2
+    apt-get clean
+    ;;
+"mariadb")
+    apt-key adv --fetch-keys 'https://mariadb.org/mariadb_release_signing_key.asc'
+    add-apt-repository "deb [arch=amd64] http://mirror2.hs-esslingen.de/mariadb/repo/10.4/ubuntu $(lsb_release -cs) main"
+    apt-get update
+    apt-get install $APTOPTS mariadb-backup
+    apt-get clean
+    ;;
+*)
+    echo "datastore $1 not supported"
+    exit 1
+    ;;
+esac