Fwd: Patch for dealing with standalone kernel options

[Partha Aji]

Forwarded the patch to wrong mailing list previously..

----- Forwarded Message -----
From: "Partha Aji" <paji(a)redhat.com&gt;
To: cobbler(a)lists.fedorahosted.org
Sent: Monday, March 22, 2010 6:43:35 PM GMT -05:00 US/Canada Eastern
Subject: Patch for dealing with standalone kernel options


So basically in a kickstart profile/distro/system record if you set standalone kernel
options as 

Kernel Options                 : {'vnc': '', 'vncpassword':
'foo'}

 the kickstart will get rendered as
/sbin/grubby --update-kernel=`/sbin/grubby --default-kernel` --args="vnc=
vncpassword=foo"
instead of 
/sbin/grubby --update-kernel=`/sbin/grubby --default-kernel` --args="vnc
vncpassword=foo"

(extra '=' after vnc). The attached patch resolves this issue. Originally this was
reported as part of spacewalk https://bugzilla.redhat.com/show_bug.cgi?id=529277 and the
investigation led to this cobbler issue and fix.

Partha

Patch for dealing with standalone kernel options

[Devan Goodwin]

Patch applied, thanks Partha.

Cheers,

Devan

On Tue, Mar 23, 2010 at 12:18 PM, Partha Aji <paji(a)redhat.com&gt; wrote:
>  Forwarded the patch to wrong mailing list previously..
>
>  ----- Forwarded Message -----
>  From: "Partha Aji" <paji(a)redhat.com&gt;
>  To: cobbler(a)lists.fedorahosted.org
>  Sent: Monday, March 22, 2010 6:43:35 PM GMT -05:00 US/Canada Eastern
>  Subject: Patch for dealing with standalone kernel options
>
>
>  So basically in a kickstart profile/distro/system record if you set standalone kernel
> options as
>
>  Kernel Options                 : {'vnc': '', 'vncpassword':
> 'foo'}
>
>   the kickstart will get rendered as
>  /sbin/grubby --update-kernel=`/sbin/grubby --default-kernel` --args="vnc=
> vncpassword=foo"
>  instead of
>  /sbin/grubby --update-kernel=`/sbin/grubby --default-kernel` --args="vnc
> vncpassword=foo"
>
>  (extra '=' after vnc). The attached patch resolves this issue. Originally this
> was reported as part of spacewalk https://bugzilla.redhat.com/show_bug.cgi?id=529277 and
> the investigation led to this cobbler issue and fix.
>
>  Partha
>  _______________________________________________
>  cobbler-devel mailing list
>  cobbler-devel(a)lists.fedorahosted.org
>  https://fedorahosted.org/mailman/listinfo/cobbler-devel
>
>  


-- 
Devan Goodwin <dgoodwin(a)rm-rf.ca&gt;
http://rm-rf.ca