[et-mgmt-tools] Cobbler - Templating not working?

[Foreman, Tim]

Wonderful!

Thanks for being so responsive. It appears to be all working now.

--Tim

> -----Original Message-----
> From: Michael DeHaan [mailto:mdehaan at redhat.com]
> Sent: Wednesday, October 25, 2006 1:20 PM
> To: Foreman, Tim
> Cc: 'et-mgmt-tools at redhat.com'
> Subject: Re: [et-mgmt-tools] Cobbler - Templating not working?
> 
> 
> Foreman, Tim wrote:
> > Well, I do have another question (now that you ask.)
> >
> > The cobbler sync command did build the ks.cfg file in the
> > kickstarts_sys/MAC_ADDRESS/ dir, but the pxelinux.cfg/MAC_ADDRESS
> > file has the following ks option: 
> > ks=http://kickstart/cobbler/kickstarts/base_test/ks.cfg
> >
> > I would have expected to see a ks url that looked more like:
> > ks=http://kickstart/cobbler/kickstarts_sys/MAC_ADDRESS/ks.cfg
> >
> > Am I still missing something?
> >
> > Thanks again for all your help.
> >
> > --Tim
> 
> Yup, another bug :)  
> 
> Here's the fix:
> 
> diff -r 1f764e5dd1cc cobbler/action_sync.py
> --- a/cobbler/action_sync.py    Wed Oct 25 12:03:55 2006 -0400
> +++ b/cobbler/action_sync.py    Wed Oct 25 14:15:58 2006 -0400
> @@ -468,8 +468,8 @@ class BootSync:
>          # kickstart path (if kickstart is used)
>          if kickstart_path is not None and kickstart_path != "":
>              # if kickstart path is on disk, we've already 
> copied it into
> -            # the HTTP mirror, so make it something anaconda 
> can get at
> -            if kickstart_path.startswith("/"):
> +            # the HTTP mirror, so make it something anaconda 
> can get at.
> +            if kickstart_path.startswith("/") or 
> kickstart_path.find("/cobbler/kickstarts/") != -1:
>                  pxe_fn = self.get_pxe_filename(system.name)
>                  kickstart_path = 
> "http://%s/cobbler/kickstarts_sys/%s/ks.cfg" % (self.settings.server, 
> pxe_fn)
>              append_line = "%s ks=%s" % (append_line, kickstart_path)
> 
> Already checked in and pushed as well.
> 
> --Michael
> 
>