[bigtop] branch master updated: BIGTOP-3126. [Puppet] Failed to deploy QFS due to Permission denied error at initailization (#447)

[evansye]

This is an automated email from the ASF dual-hosted git repository.

evansye pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/bigtop.git


The following commit(s) were added to refs/heads/master by this push:
     new 3d000a6  BIGTOP-3126. [Puppet] Failed to deploy QFS due to Permission 
denied error at initailization (#447)
3d000a6 is described below

commit 3d000a6865522d4e9d87dc6286d51378a52dc048
Author: Evans Ye <evan...@apache.org>
AuthorDate: Sat Jan 19 15:41:32 2019 +0800

    BIGTOP-3126. [Puppet] Failed to deploy QFS due to Permission denied error 
at initailization (#447)
    
    * Revert "BIGTOP-3126. [Puppet] Failed to deploy QFS due to Permission 
denied error at initailization"
    
    This reverts commit 471276e1b743f143d7ef714e7cbc1ea83c2bc287.
    
    * BIGTOP-3126. [Puppet] Failed to deploy QFS due to Permission denied error 
at initailization
---
 bigtop-deploy/puppet/modules/qfs/manifests/init.pp | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/bigtop-deploy/puppet/modules/qfs/manifests/init.pp 
b/bigtop-deploy/puppet/modules/qfs/manifests/init.pp
index 080701a..1290d34 100644
--- a/bigtop-deploy/puppet/modules/qfs/manifests/init.pp
+++ b/bigtop-deploy/puppet/modules/qfs/manifests/init.pp
@@ -36,8 +36,8 @@ class qfs {
     hadoop::create_storage_dir { $qfs::common::storage_dirs: } ->
     file { $qfs::common::storage_dirs:
       ensure => directory,
-      owner => qfs,
-      group => qfs,
+      owner => root,
+      group => root,
       mode => '0755',
     }
   }
@@ -74,8 +74,8 @@ class qfs {
     exec { "mkfs":
       command => "/usr/bin/metaserver -c $metaserver_conf",
       creates => 
"${qfs::common::storage_dirs[0]}/metaserver/checkpoint/latest",
-      # BIGTOP-3126: workaround that the qfs init script requires to run under 
a permitted directory
-      cwd => "${qfs::common::storage_dirs[0]}",
+      # BIGTOP-3126: qfs init script requires to run under a permitted 
directory
+      cwd => "/tmp",
       user => qfs,
       group => qfs,
       require => [