[Mageia-sysadm] [526] - move the postgresql server bits to a subclass

root at mageia.org root at mageia.org
Tue Dec 7 03:40:25 CET 2010


Revision: 526
Author:   misc
Date:     2010-12-07 03:40:24 +0100 (Tue, 07 Dec 2010)
Log Message:
-----------
- move the postgresql server bits to a subclass

Modified Paths:
--------------
    puppet/manifests/nodes.pp
    puppet/modules/postgresql/manifests/init.pp

Modified: puppet/manifests/nodes.pp
===================================================================
--- puppet/manifests/nodes.pp	2010-12-07 02:40:23 UTC (rev 525)
+++ puppet/manifests/nodes.pp	2010-12-07 02:40:24 UTC (rev 526)
@@ -45,7 +45,7 @@
 # 
     include default_mageia_server
     include bind::bind_master
-    include postgresql
+    include postgresql::server
     include dns_server 
     timezone::timezone { "Europe/Paris": }
 

Modified: puppet/modules/postgresql/manifests/init.pp
===================================================================
--- puppet/modules/postgresql/manifests/init.pp	2010-12-07 02:40:23 UTC (rev 525)
+++ puppet/modules/postgresql/manifests/init.pp	2010-12-07 02:40:24 UTC (rev 526)
@@ -1,72 +1,74 @@
 class postgresql {
+    class server { 
+        $pgsql_data = "/var/lib/pgsql/data/"
+        $pg_version = '9.0'
     
-    $pgsql_data = "/var/lib/pgsql/data/"
-    $pg_version = '9.0'
-
-    # missing requires is corrected in cooker, 
-    # should be removed
-    # once the fix is in a stable release 
-    package { "postgresql$pg_version-plpgsql":
-        alias => "postgresql-plpgsql",
-        ensure => installed,
-    }
-
-    package { "postgresql$pg_version-server":
-        alias => "postgresql-server",
-        ensure => installed,
-        require => Package['postgresql-plpgsql'],
-    }
-
-    service { postgresql:
-        ensure => running,
-        subscribe => Package["postgresql-server"],
-        hasstatus => true,
-    }
-
-    exec { "service postgresql reload":
-        refreshonly => true,
-        subscribe => [ File["postgresql.conf"], 
-                       File["pg_ident.conf"],
-                       File["pg_hba.conf"] ]
-    }
-
-    file { '/etc/pam.d/postgresql':
-        ensure => present,
-        owner  => root,
-        group  => root,
-        mode   => 644,
-        content => template("postgresql/pam"),
-    }
-
-    file { "postgresql.conf":
-        path => "$pgsql_data/postgresql.conf",
-        ensure => present,
-        owner => postgres,
-        group => postgres,
-        mode => 600,
-        content => template("postgresql/postgresql.conf"),
-        require => Package["postgresql-server"],
-    }
+        # missing requires is corrected in cooker, 
+        # should be removed
+        # once the fix is in a stable release 
+        package { "postgresql$pg_version-plpgsql":
+            alias => "postgresql-plpgsql",
+            ensure => installed,
+        }
     
-    file { 'pg_hba.conf':
-        path => "$pgsql_data/pg_hba.conf",
-        ensure => present,
-        owner => postgres,
-        group => postgres,
-        mode => 600,
-        content => template("postgresql/pg_hba.conf"),
-        require => Package["postgresql-server"],
+        package { "postgresql$pg_version-server":
+            alias => "postgresql-server",
+            ensure => installed,
+            require => Package['postgresql-plpgsql'],
+        }
+    
+        service { postgresql:
+            ensure => running,
+            subscribe => Package["postgresql-server"],
+            hasstatus => true,
+        }
+    
+        exec { "service postgresql reload":
+            refreshonly => true,
+            subscribe => [ File["postgresql.conf"], 
+                           File["pg_ident.conf"],
+                           File["pg_hba.conf"] ]
+        }
+    
+        file { '/etc/pam.d/postgresql':
+            ensure => present,
+            owner  => root,
+            group  => root,
+            mode   => 644,
+            content => template("postgresql/pam"),
+        }
+    
+        file { "postgresql.conf":
+            path => "$pgsql_data/postgresql.conf",
+            ensure => present,
+            owner => postgres,
+            group => postgres,
+            mode => 600,
+            content => template("postgresql/postgresql.conf"),
+            require => Package["postgresql-server"],
+        }
+        
+        file { 'pg_hba.conf':
+            path => "$pgsql_data/pg_hba.conf",
+            ensure => present,
+            owner => postgres,
+            group => postgres,
+            mode => 600,
+            content => template("postgresql/pg_hba.conf"),
+            require => Package["postgresql-server"],
+        }
+    
+        file { 'pg_ident.conf':
+            path => "$pgsql_data/pg_ident.conf",
+            ensure => present,
+            owner => postgres,
+            group => postgres,
+            mode => 600,
+            content => template("postgresql/pg_ident.conf"),
+            require => Package["postgresql-server"],
+        }
     }
 
-    file { 'pg_ident.conf':
-        path => "$pgsql_data/pg_ident.conf",
-        ensure => present,
-        owner => postgres,
-        group => postgres,
-        mode => 600,
-        content => template("postgresql/pg_ident.conf"),
-        require => Package["postgresql-server"],
-    }
 
     # TODO convert it to a regular type ( so we can later change user and so on )
     define database($description="", $user="postgres") {
-------------- next part --------------
An HTML attachment was scrubbed...
URL: </pipermail/mageia-sysadm/attachments/20101207/c892377e/attachment-0001.html>


More information about the Mageia-sysadm mailing list