From: Thierry Parmentelat <thierry.parmentelat@inria.fr>
Date: Fri, 8 Jan 2016 15:49:59 +0000 (+0100)
Subject: for fedora23 nodes - yet again
X-Git-Tag: nodeupdate-0.5-14~1
X-Git-Url: http://git.onelab.eu/?p=nodeupdate.git;a=commitdiff_plain;h=cfd8982b5f11cde95613fc85c7478ae45731a346

for fedora23 nodes - yet again
it appears that right after installation
fedora23 nodes do not know about the PlanetLab group at all
so just trying to upgrade that group doesn't do anything
---

diff --git a/NodeUpdate.py b/NodeUpdate.py
index 09183fb..fccfcb3 100755
--- a/NodeUpdate.py
+++ b/NodeUpdate.py
@@ -130,14 +130,19 @@ class YumDnf:
     def update_group(self, group):
         # it is important to invoke dnf group *upgrade* and not *update*
         # because the semantics of groups has changed within dnf
+        overall = True
         if HAS_DNF:
-            cmd = \
-                "{} {} -y group upgrade {}".format(self.command, self.options, group)
+            cmd = "{} {} -y group install {}".format(self.command, self.options, group)
+            Message("Invoking {}".format(cmd))
+            if os.system(cmd) != 0: overall = False
+            cmd = "{} {} -y group upgrade {}".format(self.command, self.options, group)
+            Message("Invoking {}".format(cmd))
+            if os.system(cmd) != 0: overall = False
         else:
-            cmd = \
-                "{} {} -y groupinstall {}".format(self.command, self.options, group)
-        Message("Invoking {}".format(cmd))
-        return os.system(cmd) == 0
+            cmd = "{} {} -y groupinstall {}".format(self.command, self.options, group)
+            Message("Invoking {}".format(cmd))
+            if os.system(cmd) != 0: overall = False
+        return overall
 
     ########## update the whole system
     def update_system(self):