]> review.fuel-infra Code Review - puppet-modules/puppetlabs-apt.git/commitdiff
Merge pull request #376 from raoulbhatia/unattended-upgrades-oldstable
authorDaniele Sluijters <daenney@users.noreply.github.com>
Thu, 13 Nov 2014 18:23:07 +0000 (19:23 +0100)
committerDaniele Sluijters <daenney@users.noreply.github.com>
Thu, 13 Nov 2014 18:23:07 +0000 (19:23 +0100)
Unattended upgrades oldstable for wheezy

manifests/builddep.pp
manifests/force.pp
manifests/params.pp
spec/classes/unattended_upgrades_spec.rb

index 3beadeb373078533201c6a23825af78dfc3414d3..3a059c273d389751da4359e1d2f38e6c6a9e27de 100644 (file)
@@ -6,7 +6,7 @@ define apt::builddep() {
   exec { "apt-builddep-${name}":
     command   => "/usr/bin/apt-get -y --force-yes build-dep ${name}",
     logoutput => 'on_failure',
-    require    => Exec['apt_update'],
+    require   => Exec['apt_update'],
   }
 
   # Need anchor to provide containment for dependencies.
index a73f7ab218ce4a1dc8b691cf73059e06cdacbe77..7524981da251452ec6e125c51afce9b55ef204e2 100644 (file)
@@ -25,15 +25,15 @@ define apt::force(
   }
 
   case $cfg_files {
-    'new':       { $config_files = '-o Dpkg::Options::="--force-confnew"' }
-    'old':       { $config_files = '-o Dpkg::Options::="--force-confold"' }
-    'unchanged': { $config_files = '-o Dpkg::Options::="--force-confdef"' }
-    'none':      { $config_files = '' }
+    'new':           { $config_files = '-o Dpkg::Options::="--force-confnew"' }
+    'old':           { $config_files = '-o Dpkg::Options::="--force-confold"' }
+    'unchanged':     { $config_files = '-o Dpkg::Options::="--force-confdef"' }
+    'none', default: { $config_files = '' }
   }
 
   case $cfg_missing {
-    true:    { $config_missing = '-o Dpkg::Options::="--force-confmiss"' }
-    false:   { $config_missing = '' }
+    true:           { $config_missing = '-o Dpkg::Options::="--force-confmiss"' }
+    false, default: { $config_missing = '' }
   }
 
   if $version == false {
index d33a401fe6df214ff93c85bccb83635ed7e27e1b..06925a341b34ede9084d6fd48c2093ffdba3e045 100644 (file)
@@ -45,7 +45,8 @@ class apt::params {
         'wheezy': {
           $backports_location = 'http://ftp.debian.org/debian/'
           $legacy_origin      = false
-          $origins            = ['origin=Debian,archive=stable,label=Debian-Security']
+          $origins            = ['origin=Debian,archive=stable,label=Debian-Security',
+                                  'origin=Debian,archive=oldstable,label=Debian-Security']
         }
         default: {
           $backports_location = 'http://http.debian.net/debian/'
index 57df21cbdfa3a2750ab38d167cf20393fd919d08..3742bf1c2add1dd9206ae849d4b6a797210b91ed 100644 (file)
@@ -134,6 +134,17 @@ describe 'apt::unattended_upgrades', :type => :class do
     it { is_expected.to_not contain_file("/etc/apt/apt.conf.d/10periodic").with_content %r{APT::Periodic::RandomSleep}}
   end
 
+  context 'wheezy' do
+    let :facts do
+      {
+        'lsbdistid'       => 'debian',
+        'lsbdistcodename' => 'wheezy',
+      }
+    end
+
+    it { is_expected.to contain_file("/etc/apt/apt.conf.d/50unattended-upgrades").with_content %r{Unattended-Upgrade::Origins-Pattern \{\n\t"origin=Debian,archive=stable,label=Debian-Security";\n\t"origin=Debian,archive=oldstable,label=Debian-Security";\n\};} }
+  end
+
   context 'anything but defaults' do
     let :facts do
       {