refactor cli/start exit

Author: gronke

libiocage/cli/start.py

@@ -58,7 +58,4 @@ def cli(ctx, rc, jails):
 
         logger.log(f"{jail.humanreadable_name} running as JID {jail.jid}")
 
-    if len(failed_jails) > 0:
-        exit(1)
-
-    exit(0)
+    exit(1) if len(failed_jails) > 0 else exit(0)

libiocage/lib/Jail.py

@@ -712,13 +712,13 @@ def _teardown_mounts(self):
 
     def _resolve_name(self, text):
 
-        if (text is None) or (len(text)==0):
+        if (text is None) or (len(text) == 0):
             raise libiocage.lib.errors.JailNotSupplied(logger=self.logger)
 
         jails_dataset = self.host.datasets.jails
 
         for dataset in list(jails_dataset.children):
-            
+
             dataset_name = dataset.name[(len(jails_dataset.name) + 1):]
             humanreadable_name = libiocage.lib.helpers.to_humanreadable_name(
                 dataset_name

libiocage/lib/JailConfig.py

@@ -213,7 +213,7 @@ def _set_name(self, name, **kwargs):
             self["id"] = name
         else:
             try:
-                self["id"] = str(uuid.UUID(name)) # legacy support
+                self["id"] = str(uuid.UUID(name))  # legacy support
             except:
                 raise libiocage.lib.errors.InvalidJailName(logger=self.logger)
 

libiocage/lib/Jails.py

@@ -50,7 +50,7 @@ def _create_jail(self, *args, **kwargs):
         kwargs["logger"] = self.logger
         kwargs["host"] = self.host
         kwargs["zfs"] = self.zfs
-        return libiocage.lib.Jail.Jail(*args,**kwargs)
+        return libiocage.lib.Jail.Jail(*args, **kwargs)
 
     @property
     def filters(self):
@@ -99,7 +99,7 @@ def _create_jail(self, *args, **kwargs):
         kwargs["logger"] = self.logger
         kwargs["host"] = self.host
         kwargs["zfs"] = self.zfs
-        return libiocage.lib.Jail.Jail(*args,**kwargs)
+        return libiocage.lib.Jail.Jail(*args, **kwargs)
 
     def __iter__(self):
         return list(JailsGenerator.__iter__(self))

libiocage/lib/Logger.py

@@ -85,7 +85,7 @@ def __init__(self, print_level=None, log_directory="/var/log/iocage"):
 
     @property
     def default_print_level(self):
-        return "spam"
+        return "info"
 
     @property
     def print_level(self):