Migrate - in class names to _

In newer puppet releases the old way to name class/modules with a -,
won't anymore be supported.

diff --git a/README b/README
index fdae462452e610e15cba19d05ec7e78a726fe993..3d2b70f79da5866279eb308fa8b84b33845a05df 100644 (file)
--- a/README
+++ b/README
@@ -36,11 +36,11 @@ From apt.conf(5), 0.7.2:
      packages."
 
 To change the default setting for DSelect::Clean, you can create a file named
-"03clean" or "03clean_vserver" in your site-apt module's files directory. You
+"03clean" or "03clean_vserver" in your site_apt module's files directory. You
 can also define this for a specific host by creating a file in a subdirectory of
-the site-apt modules' files directory that is named the same as the
-host. (example: site-apt/files/some.host.com/03clean, or
-site-apt/files/some.host.com/03clean_vserver)
+the site_apt modules' files directory that is named the same as the
+host. (example: site_apt/files/some.host.com/03clean, or
+site_apt/files/some.host.com/03clean_vserver)
 
 Variables
 =========
@@ -61,9 +61,9 @@ e.g. changing the sections included in the source, etc. you can set
 this variable to the content that you desire to use instead.
 
 For example, setting the following variable before including this class will
-pull in the templates/site-apt/sources.list file:
+pull in the templates/site_apt/sources.list file:
 
-  $custom_sources_list = template("site-apt/sources.list")
+  $custom_sources_list = template("site_apt/sources.list")
 
 $custom_preferences
 -------------------
@@ -77,8 +77,8 @@ unless you explicitly specify the version number. This file will be
 complemented with all of the preferences_snippet calls (see below).
 
 If the default preferences template doesn't suit your needs, you can create a
-template located in your site-apt module, and set $custom_preferences with the
-content (eg. $custom_preferences = template('site-apt/preferences') )
+template located in your site_apt module, and set $custom_preferences with the
+content (eg. $custom_preferences = template('site_apt/preferences') )
 
 Setting this variable to false before including this class will force the
 apt/preferences file to be absent:
@@ -198,7 +198,7 @@ be passed as the "hours" parameter of a cronjob. Example:
 
 Note that the default 4 AM cronjob won't be disabled.
 
-apt::cron::dist-upgrade
+apt::cron::dist_upgrade
 -----------------------
 
 This class sets up cron-apt so that it dist-upgrades the system and
@@ -230,8 +230,8 @@ file's content changes. The initiator file is copied from the first
 available source amongst the following ones, in decreasing priority
 order:
 
-- puppet:///site-apt/${fqdn}/upgrade_initiator
-- puppet:///site-apt/upgrade_initiator
+- puppet:///site_apt/${fqdn}/upgrade_initiator
+- puppet:///site_apt/upgrade_initiator
 - puppet:///apt/upgrade_initiator
 
 This is useful when one does not want to setup a fully automated
@@ -298,7 +298,7 @@ meta-parameter to define content inline or with the help of a template.
 Example:
 
   apt::apt_conf { "80download-only":
-    source => "puppet:///modules/site-apt/80download-only",
+    source => "puppet:///modules/site_apt/80download-only",
   }
 
 apt::preferences_snippet
@@ -320,7 +320,7 @@ apt::preseeded_package
 This simplifies installation of packages for which you wish to preseed the
 answers to debconf. For example, if you wish to provide a preseed file for the
 locales package, you would place the locales.seed file in
-'site-apt/templates/$lsbdistcodename/locales.seeds' and then include the
+'site_apt/templates/$lsbdistcodename/locales.seeds' and then include the
 following in your manifest:
 
   apt::preseeded_package { locales: }
@@ -343,8 +343,8 @@ meta-parameter to define content inline or with the help of a template.
 Example:
 
   apt::sources_list { "company_internals.list":
-    source => ["puppet:///modules/site-apt/${fqdn}/company_internals.list",
-               "puppet:///modules/site-apt/company_internals.list"],
+    source => ["puppet:///modules/site_apt/${fqdn}/company_internals.list",
+               "puppet:///modules/site_apt/company_internals.list"],
   }
 
 apt::upgrade_package

diff --git a/manifests/cron/dist-upgrade.pp b/manifests/cron/dist_upgrade.pp
similarity index 89%
rename from manifests/cron/dist-upgrade.pp
rename to manifests/cron/dist_upgrade.pp
index 2568cb7845e951e2efab3ff31893e67afb03b4f9..1c8adb1ae3ea9111e3efe6e08eb35f6e6fae1ca3 100644 (file)
--- a/manifests/cron/dist-upgrade.pp
+++ b/manifests/cron/dist_upgrade.pp
@@ -1,4 +1,4 @@
-class apt::cron::dist-upgrade inherits apt::cron::base {
+class apt::cron::dist_upgrade inherits apt::cron::base {
 
   $action = "autoclean -y
 dist-upgrade -y -o APT::Get::Show-Upgraded=true -o 'DPkg::Options::=--force-confold'

diff --git a/manifests/dist_upgrade/initiator.pp b/manifests/dist_upgrade/initiator.pp
index 6d579477e49573758c45a4ddfa555b7a34ba41d9..571cd516553a8f11bc9b1d1f2c760da4ab0008a5 100644 (file)
--- a/manifests/dist_upgrade/initiator.pp
+++ b/manifests/dist_upgrade/initiator.pp
@@ -10,8 +10,8 @@ class apt::dist_upgrade::initiator inherits apt::dist_upgrade {
     path     => "${initiator_abs}",
     checksum => md5,
     source   => [
-                 "puppet:///modules/site-apt/${fqdn}/${initiator}",
-                 "puppet:///modules/site-apt/${initiator}",
+                 "puppet:///modules/site_apt/${fqdn}/${initiator}",
+                 "puppet:///modules/site_apt/${initiator}",
                  "puppet:///modules/apt/${initiator}",
                 ],
   }

diff --git a/manifests/init.pp b/manifests/init.pp
index 960f18d4a6f2c5e2dd563d64fb3510155e7e2dbb..2ae691fa61c4aead194414975ab311a206580297 100644 (file)
--- a/manifests/init.pp
+++ b/manifests/init.pp
@@ -90,23 +90,23 @@ class apt {
   }
 
   apt_conf { "02show_upgraded":
-    source => [ "puppet:///modules/site-apt/${fqdn}/02show_upgraded",
-                "puppet:///modules/site-apt/02show_upgraded",
+    source => [ "puppet:///modules/site_apt/${fqdn}/02show_upgraded",
+                "puppet:///modules/site_apt/02show_upgraded",
                 "puppet:///modules/apt/02show_upgraded" ]
   }
 
   if ( $virtual == "vserver" ) {
     apt_conf { "03clean_vserver":
-      source => [ "puppet:///modules/site-apt/${fqdn}/03clean_vserver",
-                  "puppet:///modules/site-apt/03clean_vserver",
+      source => [ "puppet:///modules/site_apt/${fqdn}/03clean_vserver",
+                  "puppet:///modules/site_apt/03clean_vserver",
                   "puppet:///modules/apt/03clean_vserver" ],
       alias => "03clean";
     }
   }
   else {
     apt_conf { "03clean":
-      source => [ "puppet:///modules/site-apt/${fqdn}/03clean",
-                  "puppet:///modules/site-apt/03clean",
+      source => [ "puppet:///modules/site_apt/${fqdn}/03clean",
+                  "puppet:///modules/site_apt/03clean",
                   "puppet:///modules/apt/03clean" ]
     }
   }

diff --git a/manifests/preseeded_package.pp b/manifests/preseeded_package.pp
index 839f2007fe725950018ecf2d3dae373ab4a4a6fb..d831934d2dcf25f98bce4f26d61b5819ce1a5ba2 100644 (file)
--- a/manifests/preseeded_package.pp
+++ b/manifests/preseeded_package.pp
@@ -1,7 +1,7 @@
 define apt::preseeded_package ($content = "", $ensure = "installed") {
   $seedfile = "/var/cache/local/preseeding/$name.seeds"
   $real_content = $content ? { 
-    ""      => template ( "site-apt/$lsbdistcodename/$name.seeds" ),
+    ""      => template ( "site_apt/$lsbdistcodename/$name.seeds" ),
     default => $content
   }   
 

diff --git a/manifests/unattended_upgrades.pp b/manifests/unattended_upgrades.pp
index 2904f36ec15f2cb36c923f4242885d117584b457..2a199838b753894aa8623bd3a9aa2bf2e2b5d902 100644 (file)
--- a/manifests/unattended_upgrades.pp
+++ b/manifests/unattended_upgrades.pp
@@ -6,8 +6,8 @@ class apt::unattended_upgrades {
   }
 
   apt_conf { "50unattended-upgrades":
-    source  => ["puppet:///modules/site-apt/$lsbdistcodename/50unattended-upgrades",
-               "puppet:///modules/site-apt/50unattended-upgrades",
+    source  => ["puppet:///modules/site_apt/$lsbdistcodename/50unattended-upgrades",
+               "puppet:///modules/site_apt/50unattended-upgrades",
                "puppet:///modules/apt/$lsbdistcodename/50unattended-upgrades" ],
     require => Package['unattended-upgrades'],
   }