Previously the update method returned true even if yum didn't actually update.

Now it returns the list of packages updated.

1 files changed, 3 insertions, 4 deletions

diff --git a/func/minion/modules/yumcmd.py b/func/minion/modules/yumcmd.py
index 3f30ec9..0048fc6 100644
--- a/func/minion/modules/yumcmd.py
+++ b/func/minion/modules/yumcmd.py
@@ -26,23 +26,22 @@ class Yum(func_module.FuncModule):
     description = "Package updates through yum."
 
     def update(self, pkg=None):
-        # XXX support updating specific rpms
         ayum = yum.YumBase()
         ayum.doGenericSetup()
         ayum.doRepoSetup()
         try:
             ayum.doLock()
             if pkg != None:
-                ayum.update(name=pkg)
+                tx_result = ayum.update(name=pkg)
             else:
-                ayum.update()
+                tx_result = ayum.update()
             ayum.buildTransaction()
             ayum.processTransaction(
                     callback=DummyCallback())
         finally:
             ayum.closeRpmDB()
             ayum.doUnlock()
-        return True
+        return map(str, tx_result)
 
     def check_update(self, filter=[], repo=None):
         """Returns a list of packages due to be updated