for fedora23 nodes - yet again
authorThierry Parmentelat <thierry.parmentelat@inria.fr>
Fri, 8 Jan 2016 15:49:59 +0000 (16:49 +0100)
committerThierry Parmentelat <thierry.parmentelat@inria.fr>
Fri, 8 Jan 2016 15:49:59 +0000 (16:49 +0100)
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

NodeUpdate.py

index 09183fb..fccfcb3 100755 (executable)
@@ -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
     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:
         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:
         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):
 
     ########## update the whole system
     def update_system(self):