]> review.fuel-infra Code Review - puppet-modules/puppetlabs-apt.git/commitdiff
Install apt-transport-https if needed
authorBruno Travouillon <devel@travouillon.fr>
Tue, 7 Nov 2017 23:24:37 +0000 (00:24 +0100)
committerBruno Travouillon <devel@travouillon.fr>
Tue, 7 Nov 2017 23:24:37 +0000 (00:24 +0100)
Without this package, the module fails to update a repository with a
location using the https method.

This extends to Debian stretch, Ubuntu trusty and xenial.

https support has been moved into the apt package for future releases
of these distibutions, thus this change won't need to be updated.

manifests/source.pp

index 3e6d0cb814870d0b30a4b3bd0fab11282d4e3935..7ce759794b1e758149c180b04a48708ca5c7cbbc 100644 (file)
@@ -29,10 +29,13 @@ define apt::source(
     $_release = $release
   }
 
+  # Some releases do not support https transport with default installation
+  $_transport_https_releases = [ 'jessie', 'stretch', 'trusty', 'xenial' ]
+
   if $ensure == 'present' {
     if ! $location {
       fail('cannot create a source entry without specifying a location')
-    } elsif $_release == 'jessie' {
+    } elsif $_release in $_transport_https_releases {
       $method = split($location, '[:\/]+')[0]
       if $method == 'https' {
         ensure_packages('apt-transport-https')