[ PATCH ] Re: Cobbler cobbler-0.8.0-1.el5 not importing 64bit RHEL 4.6 properly FIXED

["Lester M." <needwork@xxxxxxxxx>]

 was having this same problem a while ago when that version was released. I made a patch that will fix the issue that I would like to share with everyone here.

 

The architecture type is calculated in many ways I noticed which was my first clue, the first way it uses directories (either as i386 or x86_64 and ia64 I believe from looking at the source codes.), the 2nd way is thathe 3rd way is looking for a certain rpm file. (Kernel-header) of course in RHEL4 the kernel-header rpm does not exist. I still find it very weird that the software could not deduct that if the rpm did not exist fall back to using another method of checking but what the heck.  :-P

 

The only thing I did was in action_import.py make another statement to look for a RHEL4 version rpm, an rpm specific to RHEL4.. I used the code for the search pattern that is used in RHEL5 (kernel-header).

 

 

Please review my patch for inclusion, as it fixes that problem.

-- Les

Name : cobbler
Version : 0.8.2-2

 

 <-- ##### Start Patch ##### -->

--- action_import.py    2008-03-18 21:48:40.000000000 -0400
+++ /usr/lib/python2.4/site-packages/cobbler/action_import.py   2008-03-18 22:08:11.000000000 -0400
@@ -565,7 +565,23 @@
                elif x.find("ia64") != -1:
                    foo["result"] = "ia64"
                    return
-
+
+
+# This extra code block is a temporary fix for rhel4.x 64bit distro ARCH identification-- L.M.
+       for x in fnames:
+           if not x.endswith("rpm"):
+               continue
+           if x.find("kernel-largesmp") != -1:
+               print _("- kernel header found: %s") % x
+               if x.find("i386") != -1:
+                   foo["result"] = "x86"
+                   return
+               elif x.find("x86_64") != -1:
+                   foo["result"] = "x86_64"
+                   return
+               elif x.find("ia64") != -1:
+                   foo["result"] = "ia64"
+                   return

    def learn_arch_from_tree(self,dirname):
        """

 <-- ##### End Patch ##### -->

_______________________________________________
et-mgmt-tools mailing list
et-mgmt-tools@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/et-mgmt-tools