Draft: Generalize task status checks

katprep/cli/maintenance.py

@@ -384,8 +384,14 @@ def status(options, args):
     :param args: argparse options dictionary containing parameters
     :type args: argparse options dict
     """
-    # I'M A PIECE OF SHIT AND NEED TO BE REPLACED
-"""     tasks = {
+    raise NotImplementedError("This functions needs a generic implementation")
+    # TODO: we have to find a way to handle this
+    # TODO: how to handle the call to the SAT_CLIENT?
+    # TODO: what it be good if SAT_CLIENT was a dict like for monitoring and virt?
+    # TODO: make the results generic
+
+    #verify snapshot/downtime per host
+    tasks = {
         "Erratum": "Actions::Katello::Host::Erratum::Install",
         "Package": "Actions::Katello::Host::Update"
     }
@@ -438,7 +444,7 @@ def status(options, args):
                                 task, host, result["result"]
                             )
                     except KeyError as kerr:
-                        LOGGER.debug("Failed showing result: %r", kerr) """
+                        LOGGER.debug("Failed showing result: %r", kerr)
 
 
 def cleanup(options, args):

katprep/management/base.py

@@ -35,3 +35,14 @@ def is_reboot_required(self, host) -> bool:
         Checks if the client requires a reboot.
         """
 
+    @abstractmethod
+    def get_errata_task_status(self, host):
+        """
+        Get the status of errata installations for the given host.
+        """
+
+    @abstractmethod
+    def get_upgrade_task_status(self, host):
+        """
+        Get the status of package upgrades for the given host.
+        """