← Back to team overview

duplicity-team team mailing list archive

[Merge] lp:~aaron-whitehouse/duplicity/08-dockerfixes into lp:duplicity

 

Aaron Whitehouse has proposed merging lp:~aaron-whitehouse/duplicity/08-dockerfixes into lp:duplicity.

Commit message:
Update duplicity_test Dockerfile:
* Use 18.04 instead of 16.04
* Use Ubuntu 18.04 version of pip
* Add Python3 and 2to3 as a dependencies
* Set docker locale as UTF-8


Requested reviews:
  duplicity-team (duplicity-team)

For more details, see:
https://code.launchpad.net/~aaron-whitehouse/duplicity/08-dockerfixes/+merge/369367

Update duplicity_test Dockerfile:
* Use 18.04 instead of 16.04
* Use Ubuntu 18.04 version of pip
* Add Python3 and 2to3 as a dependencies
* Set docker locale as UTF-8

All tests now pass on trunk with this docker setup:
cd testing/infrastructure
sudo ./build-duplicity_test.sh
./setup.sh
cd testing
./run-tests
-- 
Your team duplicity-team is requested to review the proposed merge of lp:~aaron-whitehouse/duplicity/08-dockerfixes into lp:duplicity.
=== modified file 'testing/infrastructure/duplicity_test/Dockerfile'
--- testing/infrastructure/duplicity_test/Dockerfile	2017-05-14 07:17:59 +0000
+++ testing/infrastructure/duplicity_test/Dockerfile	2019-06-26 21:38:44 +0000
@@ -18,12 +18,21 @@
 # along with duplicity; if not, write to the Free Software Foundation,
 # Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
 
-FROM ubuntu:16.04
+FROM ubuntu:18.04
+
+# Set locale to prevent UTF-8 errors
+ENV LANG C.UTF-8
+ENV LC_ALL C.UTF-8
+
+
+# Set to non-interactive so no tzdata prompt
+ARG DEBIAN_FRONTEND=noninteractive
 
 # Installing some pre-requisites and some
 # packages needed for testing duplicity
 RUN apt-get update \
     && apt-get install -y \
+            2to3 \
             build-essential \
             bzr \
             intltool \
@@ -34,8 +43,11 @@
             openssl \
             par2 \
             python-dev \
+            python-pip \
+            python3-dev \
+            python3-pip \
             rdiff \
-            tzdata
+            tzdata  # required for testing/unit/test_statistics.py
 
 # The following packages are not necessary for testing but make life easier or support debugging
 RUN apt-get install -y \
@@ -46,12 +58,6 @@
             ftp \
     && rm -rf /var/lib/apt/lists/*
 
-# Need to make gpg2 the default gpg
-RUN mv /usr/bin/gpg /usr/bin/gpg1 && ln -s /usr/bin/gpg2 /usr/bin/gpg
-
-# Installing pip
-RUN curl https://bootstrap.pypa.io/get-pip.py | python
-
 # Installing requirements for pip
 COPY requirements.txt /tmp
 RUN pip install --requirement /tmp/requirements.txt
@@ -77,5 +83,10 @@
 # Branch the duplicity repo for testing
 RUN bzr branch lp:duplicity duplicity
 
+# Build rsync binding
+WORKDIR duplicity/duplicity
+RUN python2 compilec.py
+RUN python3 compilec.py
+
 # Set final workdir to duplicity
-WORKDIR duplicity
+WORKDIR ..


Follow ups