On Tue, Dec 30, 2008 at 06:32:51PM -0500, Michael DeHaan wrote:
> Axel Thimm wrote:
> > It looks like cobbler sync will copy /boot/memtest* to the tftp top
> > level directort and will later check under /images to see whether it
> > had copied it or not. So it misses it and one needs to manually add
> > memtest profiles.
> >
> > Is this intended behaviour or a bug? If it's a bug where is the proper
> > location for memtest, top level or under /images?
> 
> This does seem like a bug -- generally I'd say we shouldn't mess with 
> top-level content of tftpboot (much) as other apps could be sharing it, 
> so "/images" seems right.
> 
> File a ticket at fedorahosted.org/cobbler if you haven't already (or if 
> you want to send in a patch, that's great too) -- either way, it should 
> be simple to fix up.

This patch works:
--- /usr/lib/python2.5/site-packages/cobbler/pxegen.py.org      2008-12-19 
18:17:37.000000000 +0200
+++ /usr/lib/python2.5/site-packages/cobbler/pxegen.py          2008-12-31 
02:09:00.000000000 +0200
@@ -78,7 +78,7 @@
             utils.copyfile_pattern('/usr/share/syslinux/pxelinux.0', dst, 
api=self.api)
  
         # copy memtest only if we find it
-        utils.copyfile_pattern('/boot/memtest*', dst, require_match=False, 
api=self.api)
+        utils.copyfile_pattern('/boot/memtest*', dst + "/images", 
require_match=False, api=self.api)
   
         # copy elilo which we include for IA64 targets
         utils.copyfile_pattern('/var/lib/cobbler/elilo-3.8-ia64.efi', dst, 
api=self.api)


-- 
Axel.Thimm at ATrpms.net

Attachment: pgp5sCZhMrr7d.pgp
Description: PGP signature

_______________________________________________
cobbler mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/cobbler

Reply via email to