Browse Source

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
changes/97/151397/1
Elizabeth Krumbach 6 years ago
parent
commit
b054f34b27
2 changed files with 17 additions and 1 deletions
  1. 8
    0
      manifests/init.pp
  2. 9
    1
      manifests/site.pp

+ 8
- 0
manifests/init.pp View File

@@ -24,6 +24,14 @@ class reviewday {
24 24
     ensure => present,
25 25
   }
26 26
 
27
+  file {'/var/lib/reviewday':
28
+    ensure  => directory,
29
+    owner   => 'reviewday',
30
+    group   => 'reviewday',
31
+    mode    => '0755',
32
+    require => User['reviewday'],
33
+  }
34
+
27 35
   group { 'reviewday':
28 36
     ensure => present,
29 37
   }

+ 9
- 1
manifests/site.pp View File

@@ -67,6 +67,14 @@ define reviewday::site(
67 67
     }
68 68
   }
69 69
 
70
+  file {'/var/lib/reviewday/reviewday':
71
+    ensure  => directory,
72
+    owner   => 'reviewday',
73
+    group   => 'reviewday',
74
+    mode    => '0755',
75
+    require => File['/var/lib/reviewday/'],
76
+  }
77
+
70 78
   vcsrepo { '/var/lib/reviewday/reviewday':
71 79
     ensure   => present,
72 80
     provider => git,
@@ -77,7 +85,7 @@ define reviewday::site(
77 85
     ensure => directory,
78 86
     owner  => 'reviewday',
79 87
     group  => 'reviewday',
80
-    mode   => '0644',
88
+    mode   => '0755',
81 89
   }
82 90
 
83 91
   file { '/var/lib/reviewday/.ssh/config':

Loading…
Cancel
Save