From 658a0beea4ce79e0de9db9750bde711b0e1e0690 Mon Sep 17 00:00:00 2001 From: Elizabeth Krumbach Date: Tue, 26 Mar 2013 11:59:16 -0700 Subject: [PATCH] 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 Approved: James E. Blair Reviewed-by: James E. Blair Tested-by: Jenkins --- modules/reviewday/manifests/init.pp | 8 ++++++++ modules/reviewday/manifests/site.pp | 10 +++++++++- 2 files changed, 17 insertions(+), 1 deletion(-) diff --git a/modules/reviewday/manifests/init.pp b/modules/reviewday/manifests/init.pp index 830afefb8f..84847e6e3d 100644 --- a/modules/reviewday/manifests/init.pp +++ b/modules/reviewday/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/modules/reviewday/manifests/site.pp b/modules/reviewday/manifests/site.pp index 3825b5ed58..94df6173ce 100644 --- a/modules/reviewday/manifests/site.pp +++ b/modules/reviewday/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':