svn commit: r1483456 - in /incubator/ambari/trunk: CHANGES.txt ambari-agent/src/main/puppet/modules/hdp-hadoop/manifests/hdfs/directory.pp ambari-agent/src/main/puppet/modules/hdp-hadoop/manifests/init.pp

[smohanty]

Author: smohanty
Date: Thu May 16 17:16:03 2013
New Revision: 1483456

URL: http://svn.apache.org/r1483456
Log:
AMBARI-2143. HBASE fails to start on master. (smohanty)

Modified:
    incubator/ambari/trunk/CHANGES.txt
    
incubator/ambari/trunk/ambari-agent/src/main/puppet/modules/hdp-hadoop/manifests/hdfs/directory.pp
    
incubator/ambari/trunk/ambari-agent/src/main/puppet/modules/hdp-hadoop/manifests/init.pp

Modified: incubator/ambari/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/incubator/ambari/trunk/CHANGES.txt?rev=1483456&r1=1483455&r2=1483456&view=diff
==============================================================================
--- incubator/ambari/trunk/CHANGES.txt (original)
+++ incubator/ambari/trunk/CHANGES.txt Thu May 16 17:16:03 2013
@@ -856,6 +856,8 @@ Trunk (unreleased changes):
 
  BUG FIXES
 
+ AMBARI-2143. HBASE fails to start on master. (smohanty
+
  AMBARI-2141. When hbase user is changed hbase does not start properly
  after upgrade. (smohanty)
 

Modified: 
incubator/ambari/trunk/ambari-agent/src/main/puppet/modules/hdp-hadoop/manifests/hdfs/directory.pp
URL: 
http://svn.apache.org/viewvc/incubator/ambari/trunk/ambari-agent/src/main/puppet/modules/hdp-hadoop/manifests/hdfs/directory.pp?rev=1483456&r1=1483455&r2=1483456&view=diff
==============================================================================
--- 
incubator/ambari/trunk/ambari-agent/src/main/puppet/modules/hdp-hadoop/manifests/hdfs/directory.pp
 (original)
+++ 
incubator/ambari/trunk/ambari-agent/src/main/puppet/modules/hdp-hadoop/manifests/hdfs/directory.pp
 Thu May 16 17:16:03 2013
@@ -28,7 +28,7 @@ define hdp-hadoop::hdfs::directory(
   $recursive_chmod = false
 ) 
 {
-  $unless_cmd = "hadoop fs -ls ${name} >/dev/null 2>&1"
+  $dir_exists = "hadoop fs -ls ${name} >/dev/null 2>&1"
   $tries = 30
   $try_sleep = 10
  
@@ -42,7 +42,7 @@ define hdp-hadoop::hdfs::directory(
     }
     hdp-hadoop::exec-hadoop { $mkdir_cmd:
       command   => $mkdir_cmd,
-      unless    => $unless_cmd,
+      unless    => $dir_exists,
       try_sleep => $try_sleep,
       tries     => $tries
     }
@@ -65,7 +65,7 @@ define hdp-hadoop::hdfs::directory(
       }
       hdp-hadoop::exec-hadoop {$chown_cmd :
         command   => $chown_cmd,
-        unless    => $unless_cmd,
+        onlyif    => $dir_exists,
         try_sleep => $try_sleep,
         tries     => $tries
       }
@@ -81,7 +81,7 @@ define hdp-hadoop::hdfs::directory(
       }
       hdp-hadoop::exec-hadoop {$chmod_cmd :
         command   => $chmod_cmd,
-        unless    => $unless_cmd,
+        onlyif    => $dir_exists,
         try_sleep => $try_sleep,
         tries     => $tries
       }

Modified: 
incubator/ambari/trunk/ambari-agent/src/main/puppet/modules/hdp-hadoop/manifests/init.pp
URL: 
http://svn.apache.org/viewvc/incubator/ambari/trunk/ambari-agent/src/main/puppet/modules/hdp-hadoop/manifests/init.pp?rev=1483456&r1=1483455&r2=1483456&view=diff
==============================================================================
--- 
incubator/ambari/trunk/ambari-agent/src/main/puppet/modules/hdp-hadoop/manifests/init.pp
 (original)
+++ 
incubator/ambari/trunk/ambari-agent/src/main/puppet/modules/hdp-hadoop/manifests/init.pp
 Thu May 16 17:16:03 2013
@@ -340,7 +340,8 @@ define hdp-hadoop::exec-hadoop(
   $timeout = 900,
   $try_sleep = undef,
   $user = undef,
-  $logoutput = undef
+  $logoutput = undef,
+  $onlyif = undef
 )
 {
   include hdp-hadoop::params
@@ -382,6 +383,7 @@ define hdp-hadoop::exec-hadoop(
     tries       => $tries,
     timeout     => $timeout,
     try_sleep   => $try_sleep,
-    logoutput   => $logoutput
+    logoutput   => $logoutput,
+    onlyif      => $onlyif
   }
 }