From 036f79cd134139954a79afc4301f91008608b548 Mon Sep 17 00:00:00 2001
From: Jeremy Baumont <jeremy@baumont.org>
Date: Tue, 10 Nov 2009 13:46:00 +0100
Subject: [PATCH] fix correct group oinstall to oracle directories

assign correct groups and ownership to inventory oracle directories,
check out database_server class for details.
---
 modules/oracle/manifests/database_server.pp |  8 ++++----
 modules/oracle/manifests/virt_users.pp      | 10 ++++++----
 2 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/modules/oracle/manifests/database_server.pp b/modules/oracle/manifests/database_server.pp
index 2a727ee..4c0f0db 100644
--- a/modules/oracle/manifests/database_server.pp
+++ b/modules/oracle/manifests/database_server.pp
@@ -93,8 +93,8 @@ class oracle::database_server::directories {
 #        "var_opt_oracle":
 #            path => "/var/opt/oracle",
 #            ensure => directory,
-#            owner => "root",
-#            group => "root",
+#            owner => "oracle",
+#            group => "oinstall",
 #            require => File["var_opt"],
 #            mode => 755
 #    }
@@ -103,8 +103,8 @@ class oracle::database_server::directories {
 #        "/var/opt/oracle/oraInst.loc":
 #            ensure => present,
 #            force => true,
-#            owner => "root",
-#            group => "root",
+#            owner => "oracle",
+#            group => "oinstall",
 #            mode => 644
 #    }
 
diff --git a/modules/oracle/manifests/virt_users.pp b/modules/oracle/manifests/virt_users.pp
index 8fe950f..be7bba3 100644
--- a/modules/oracle/manifests/virt_users.pp
+++ b/modules/oracle/manifests/virt_users.pp
@@ -18,12 +18,14 @@ class oracle::virt_users  {
 
     @user { oracle:
         ensure  => present,
-        uid     => 500,
-        gid     => 500,
-        groups  => ["dba", "oinstall" ],
+        gid     => "dba",
+        groups  =>  [ "oinstall" , "other" ],
         comment => "Oracle Application User",
         home    => "/users/oracle",
-        require => [ Group["dba"], Group["oinstall"] ,]
+        require => [ 
+                    Group["dba"], 
+                    Group["oinstall"] 
+                    ] ,
         shell   => "/usr/bin/bash",
     }
 
-- 
2.11.4.GIT