Repository: ambari
Updated Branches:
  refs/heads/branch-2.1 8aa706c8b -> 598318148
  refs/heads/trunk 9a8d5d992 -> 8abab3106


AMBARI-12810 Host check shows no errors when registering non root agents 
manually (dlysnichenko)


Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/59831814
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/59831814
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/59831814

Branch: refs/heads/branch-2.1
Commit: 59831814829e4bb280157e124f551e11885e2b6d
Parents: 8aa706c
Author: Lisnichenko Dmitro <[email protected]>
Authored: Wed Aug 19 17:10:16 2015 +0300
Committer: Lisnichenko Dmitro <[email protected]>
Committed: Wed Aug 19 17:10:16 2015 +0300

----------------------------------------------------------------------
 .../libraries/functions/packages_analyzer.py              | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/59831814/ambari-common/src/main/python/resource_management/libraries/functions/packages_analyzer.py
----------------------------------------------------------------------
diff --git 
a/ambari-common/src/main/python/resource_management/libraries/functions/packages_analyzer.py
 
b/ambari-common/src/main/python/resource_management/libraries/functions/packages_analyzer.py
index 3693886..7f34cc1 100644
--- 
a/ambari-common/src/main/python/resource_management/libraries/functions/packages_analyzer.py
+++ 
b/ambari-common/src/main/python/resource_management/libraries/functions/packages_analyzer.py
@@ -91,11 +91,11 @@ def allInstalledPackages(allInstalledPackages):
 
   if OSCheck.is_suse_family():
     return _lookUpZypperPackages(
-      ["zypper", "search", "--installed-only", "--details"],
+      ["sudo", "zypper", "search", "--installed-only", "--details"],
       allInstalledPackages)
   elif OSCheck.is_redhat_family():
     return _lookUpYumPackages(
-      ["yum", "list", "installed"],
+      ["sudo", "yum", "list", "installed"],
       'Installed Packages',
       allInstalledPackages)
   elif OSCheck.is_ubuntu_family():
@@ -109,11 +109,11 @@ def allAvailablePackages(allAvailablePackages):
 
   if OSCheck.is_suse_family():
     return _lookUpZypperPackages(
-      ["zypper", "search", "--uninstalled-only", "--details"],
+      ["sudo", "zypper", "search", "--uninstalled-only", "--details"],
       allAvailablePackages)
   elif OSCheck.is_redhat_family():
     return _lookUpYumPackages(
-      ["yum", "list", "available"],
+      ["sudo", "yum", "list", "available"],
       'Available Packages',
       allAvailablePackages)
   elif OSCheck.is_ubuntu_family():
@@ -121,7 +121,7 @@ def allAvailablePackages(allAvailablePackages):
       LIST_AVAILABLE_PACKAGES_UBUNTU,
       allAvailablePackages)
 
-
+# ToDo: add execution via sudo for ubuntu (currently Ubuntu is not supported)
 def _lookUpAptPackages(command, allPackages):
   try:
     result = subprocessWithTimeout(command)

Reply via email to