Fix permissions on reviewday directories

Reviewday home and git repository directories were owned by root,
fix those permissions and make http directory executable.

Change-Id: I3ae856a3c1ffa13d9f8a7217284d89d8035f2613
Reviewed-on: https://review.openstack.org/25424
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Approved: James E. Blair <corvus@inaugust.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins

2 files changed, 17 insertions, 1 deletions

diff --git a/manifests/init.pp b/manifests/init.pp
index 830afef..84847e6 100644
--- a/manifests/init.pp
+++ b/manifests/init.pp
@@ -24,6 +24,14 @@ class reviewday {
     ensure => present,
   }
 
+  file {'/var/lib/reviewday':
+    ensure  => directory,
+    owner   => 'reviewday',
+    group   => 'reviewday',
+    mode    => '0755',
+    require => User['reviewday'],
+  }
+
   group { 'reviewday':
     ensure => present,
   }
diff --git a/manifests/site.pp b/manifests/site.pp
index 3825b5e..94df617 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -67,6 +67,14 @@ define reviewday::site(
     }
   }
 
+  file {'/var/lib/reviewday/reviewday':
+    ensure  => directory,
+    owner   => 'reviewday',
+    group   => 'reviewday',
+    mode    => '0755',
+    require => File['/var/lib/reviewday/'],
+  }
+
   vcsrepo { '/var/lib/reviewday/reviewday':
     ensure   => present,
     provider => git,
@@ -77,7 +85,7 @@ define reviewday::site(
     ensure => directory,
     owner  => 'reviewday',
     group  => 'reviewday',
-    mode   => '0644',
+    mode   => '0755',
   }
 
   file { '/var/lib/reviewday/.ssh/config':