[et-mgmt-tools] Patches for cobbler and koan (for cobbler 0.9x)

[Pablo Iranzo Gómez]

	After aplying this patch: (previous mail + extended for koan -r):
--------------------
diff --git a/koan/app.py b/koan/app.py
index dba6c65..b5837c5 100755
--- a/koan/app.py
+++ b/koan/app.py
@@ -534,13 +534,10 @@ class Koan:
         # initialise the profile, from the server if any
         if self.profile:
             profile_data = self.get_profile_xmlrpc(self.profile)
-            filler = "kickstarts"
         elif self.system:
             profile_data = self.get_system_xmlrpc(self.system)
-            filler = "kickstarts_sys"
         else:
             profile_data = {}
-            filler = None

         if self.no_cobbler:
             # if the value given to no_cobbler has no url protocol
@@ -563,8 +560,8 @@ class Koan:
         if profile_data.has_key("kickstart"):

             # fix URLs
-            if filler and profile_data["kickstart"].startswith("/"):
-               profile_data["kickstart"] = "http://%s/cblr/%s/%s/ks.cfg"
% (profile_data['server'], filler, profile_data['name'])
+            if profile_data["kickstart"].startswith("/"):
+               profile_data["kickstart"] =
"http://%s/cblr/svc/?op=ks&profile=%s" % (profile_data['server'],
profile_data['profile'])

             # find_kickstart source tree in the kickstart file
             self.get_install_tree_from_kickstart(profile_data)
@@ -782,7 +779,7 @@ class Koan:
             if not os.path.exists("/sbin/grubby"):
                 raise InfoException, "grubby is not installed"
             k_args =
self.safe_load(profile_data,'kernel_options',default='')
-            k_args = k_args + " ks=file:ks.cfg"
+            k_args = k_args + " ks=" + profile_data["kickstart"]

             kickstart = self.safe_load(profile_data,'kickstart')
----------------------------



	I'm getting:


# koan  -s 172.26.0.254 -V Xen -v --profile=wks-CentOS
'profile'

	Any hint on what I did wrong?

	Thanks
	Pablo




-- 
Pablo Iranzo Gómez
(http://Alufis35.uv.es/~iranzo/)
(PGPKey Available on http://www.uv.es/~iranzop/PGPKey.pgp)
                  --
Postulado de Boling sobre la Ley de Murphy:

Si se encuentra bien, no se preocupe. Se le pasará

On Sat, 19 Apr 2008, Pablo Iranzo Gómez wrote:

>
> 	Hi
> 	Attached are a patch for cobbler missing the config entry to
> submited isc_set_host_name with default to 0.
>
> 	The patch for Koan is for being used with newer versions of
> cobbler 0.9x after Michael's patch for "svc/op=ks", as the path for the
> kickstart which koan uses, is "hard-coded".
>
> 	I think that the behaviour should be cobbler to provided the URL
> for the kickstart, removing dependencies between cobbler and koan, or at
> least, provide version number from cobbler to koan "client", for koan to
> construct URL's for kickstart, kernel, etc appropiately. ¿omments?
>
> 	Regards
> 	Pablo
>
>
>
> --
> Pablo Iranzo Gómez
> (http://Alufis35.uv.es/~iranzo/)
> (PGPKey Available on http://www.uv.es/~iranzop/PGPKey.pgp)
>                   --
> Postulado de Boling sobre la Ley de Murphy:
>
> Si se encuentra bien, no se preocupe. Se le pasará