automation/gitlab: add https transport support to Debian images
diff mbox series

Message ID 20200327114947.47654-1-roger.pau@citrix.com
State New
Headers show
Series
  • automation/gitlab: add https transport support to Debian images
Related show

Commit Message

Roger Pau Monné March 27, 2020, 11:49 a.m. UTC
The LLVM repos have switched from http to https, and trying to access
using http will get redirected to https. Add the apt-transport-https
package to the x86 Debian containers that use the LLVM repos, in order
to support the https transport method.

Note that on Arm we only test with gcc, so don't add the package for
the Debian Arm container.

This fixes the following error seen on the QEMU smoke tests:

E: The method driver /usr/lib/apt/methods/https could not be found.

Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
---
Cc: Wei Liu <wl@xen.org>
---
Should I try to push the updated containers myself?
---
 automation/build/debian/stretch-i386.dockerfile  | 1 +
 automation/build/debian/stretch.dockerfile       | 1 +
 automation/build/debian/unstable-i386.dockerfile | 1 +
 automation/build/debian/unstable.dockerfile      | 1 +
 4 files changed, 4 insertions(+)

Comments

Wei Liu March 27, 2020, 1:29 p.m. UTC | #1
On Fri, Mar 27, 2020 at 12:49:47PM +0100, Roger Pau Monne wrote:
> The LLVM repos have switched from http to https, and trying to access
> using http will get redirected to https. Add the apt-transport-https
> package to the x86 Debian containers that use the LLVM repos, in order
> to support the https transport method.
> 
> Note that on Arm we only test with gcc, so don't add the package for
> the Debian Arm container.
> 
> This fixes the following error seen on the QEMU smoke tests:
> 
> E: The method driver /usr/lib/apt/methods/https could not be found.
> 
> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>

Acked-by: Wei Liu <wl@xen.org>

> ---
> Should I try to push the updated containers myself?

Yes if you can do that, that would be great.

Wei.

Patch
diff mbox series

diff --git a/automation/build/debian/stretch-i386.dockerfile b/automation/build/debian/stretch-i386.dockerfile
index 4797ae3442..7b6f8eff69 100644
--- a/automation/build/debian/stretch-i386.dockerfile
+++ b/automation/build/debian/stretch-i386.dockerfile
@@ -45,6 +45,7 @@  RUN apt-get update && \
         wget \
         git \
         nasm \
+        apt-transport-https \
         && \
         apt-get autoremove -y && \
         apt-get clean && \
diff --git a/automation/build/debian/stretch.dockerfile b/automation/build/debian/stretch.dockerfile
index cfbb2e9b0b..32742f7f39 100644
--- a/automation/build/debian/stretch.dockerfile
+++ b/automation/build/debian/stretch.dockerfile
@@ -44,6 +44,7 @@  RUN apt-get update && \
         git \
         nasm \
         gnupg \
+        apt-transport-https \
         && \
         apt-get autoremove -y && \
         apt-get clean && \
diff --git a/automation/build/debian/unstable-i386.dockerfile b/automation/build/debian/unstable-i386.dockerfile
index 1a73b3b1ec..86ff3585df 100644
--- a/automation/build/debian/unstable-i386.dockerfile
+++ b/automation/build/debian/unstable-i386.dockerfile
@@ -45,6 +45,7 @@  RUN apt-get update && \
         wget \
         git \
         nasm \
+        apt-transport-https \
         && \
         apt-get autoremove -y && \
         apt-get clean && \
diff --git a/automation/build/debian/unstable.dockerfile b/automation/build/debian/unstable.dockerfile
index 2a834f6719..d0aa5ad2bb 100644
--- a/automation/build/debian/unstable.dockerfile
+++ b/automation/build/debian/unstable.dockerfile
@@ -44,6 +44,7 @@  RUN apt-get update && \
         git \
         nasm \
         gnupg \
+        apt-transport-https \
         && \
         apt-get autoremove -y && \
         apt-get clean && \