Blob Blame History Raw
commit fdf493ae4095c3266997e8b9c9cbcb31767b06d6
Author: Orion Poplawski <orion@nwra.com>
Date:   Wed Dec 8 20:19:51 2021 -0700

    Remove get-loaders

diff --git a/cobbler/action_check.py b/cobbler/action_check.py
index c1980a39..825e666d 100644
--- a/cobbler/action_check.py
+++ b/cobbler/action_check.py
@@ -335,7 +335,7 @@ class BootCheck:
              not_found.append(loader_name)
 
        if len(not_found) > 0:
-          status.append("Some network boot-loaders are missing from /var/lib/cobbler/loaders, you may run 'cobbler get-loaders' to download them, or, if you only want to handle x86/x86_64 netbooting, you may ensure that you have installed a *recent* version of the syslinux package installed and can ignore this message entirely.  Files in this directory, should you want to support all architectures, should include pxelinux.0, menu.c32, elilo.efi, and yaboot. The 'cobbler get-loaders' command is the easiest way to resolve these requirements.")
+          status.append("Some network boot-loaders are missing from /var/lib/cobbler/loaders.  If you only want to handle x86/x86_64 netbooting, you may ensure that you have installed a *recent* version of the syslinux package installed and can ignore this message entirely.  Files in this directory, should you want to support all architectures, should include pxelinux.0, menu.c32, elilo.efi, and yaboot.")
 
    def check_tftpd_bin(self,status):
        """
diff --git a/cobbler/api.py b/cobbler/api.py
index 77ffc689..30f4b878 100644
--- a/cobbler/api.py
+++ b/cobbler/api.py
@@ -36,7 +36,6 @@ import action_report
 import action_power
 import action_log
 import action_hardlink
-import action_dlcontent
 from cexceptions import CX
 import kickgen
 import yumgen
@@ -723,19 +722,6 @@ class BootAPI:
 
     # ==========================================================================
 
-    def dlcontent(self,force=False,logger=None):
-        """
-        Downloads bootloader content that may not be avialable in packages
-        for the given arch, ex: if installing on PPC, get syslinux. If installing
-        on x86_64, get elilo, etc.
-        """
-        # FIXME: teach code that copies it to grab from the right place
-        self.log("dlcontent")
-        grabber = action_dlcontent.ContentDownloader(self._config, logger=logger)
-        return grabber.run(force)
-
-    # ==========================================================================
-
     def validateks(self, logger=None):
         """
         Use ksvalidator (from pykickstart, if available) to determine
diff --git a/cobbler/cli.py b/cobbler/cli.py
index a1aa815e..71af6853 100755
--- a/cobbler/cli.py
+++ b/cobbler/cli.py
@@ -56,7 +56,7 @@ OBJECT_TYPES = OBJECT_ACTIONS_MAP.keys()
 OBJECT_ACTIONS = []
 for actions in OBJECT_ACTIONS_MAP.values():
    OBJECT_ACTIONS += actions
-DIRECT_ACTIONS = "aclsetup buildiso import list replicate report reposync sync validateks version signature get-loaders hardlink".split()
+DIRECT_ACTIONS = "aclsetup buildiso import list replicate report reposync sync validateks version signature hardlink".split()
 
 ####################################################
 
@@ -501,10 +501,6 @@ class BootCLI:
         elif action_name == "validateks":
             (options, args) = self.parser.parse_args()
             task_id = self.start_task("validateks",options)
-        elif action_name == "get-loaders":
-            self.parser.add_option("--force", dest="force", action="store_true", help="overwrite any existing content in /var/lib/cobbler/loaders")
-            (options, args) = self.parser.parse_args()
-            task_id = self.start_task("dlcontent",options)
         elif action_name == "import":
             self.parser.add_option("--arch",         dest="arch",           help="OS architecture being imported")
             self.parser.add_option("--breed",        dest="breed",          help="the breed being imported")
diff --git a/cobbler/remote.py b/cobbler/remote.py
index f7b93abc..d2b5d0b5 100644
--- a/cobbler/remote.py
+++ b/cobbler/remote.py
@@ -168,14 +168,6 @@ class CobblerXMLRPCInterface:
             )
         return self.__start_task(runner, token, "aclsetup", "(CLI) ACL Configuration", options)
 
-    def background_dlcontent(self, options, token):
-        """
-        Download bootloaders and other support files.
-        """
-        def runner(self):
-            return self.remote.api.dlcontent(self.options.get("force",False), self.logger)
-        return self.__start_task(runner, token, "get_loaders", "Download Bootloader Content", options)
-
     def background_sync(self, options, token):
         def runner(self):
             return self.remote.api.sync(self.options.get("verbose",False),logger=self.logger)
diff --git a/docs/appendix.rst b/docs/appendix.rst
index b1945c72..7dc73411 100644
--- a/docs/appendix.rst
+++ b/docs/appendix.rst
@@ -611,12 +611,6 @@ Steps
 DBAN 2.2.6
 **********
 
-Retrieve the extra loader parts that DBAN 2.2.6 needs:
-
-.. code-block:: bash
-
-    cobbler get-loaders
-
 Download DBAN:
 
 .. code-block:: bash