FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Redhat > EPEL Development

 
 
LinkBack Thread Tools
 
Old 03-19-2011, 05:08 AM
Ian Ward Comfort
 
Default Fix --mtu option to kickstart network command (#689081)

See the Bugzilla issue in the subject. This is based on rhel6-branch (but I
can rebase if that's wrong) and can be pulled from:

git://github.com/icomfort/anaconda.git rhel6-fix-mtu

Please keep me cc'd on any discussion; I am not subscribed to the list.

-- >8 --
mtu was not among the fields that NetworkData.execute was copying from the
kickstart data structure to the NetworkDevice object.
---
kickstart.py | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/kickstart.py b/kickstart.py
index 3ce4ad1..5e3a15f 100644
--- a/kickstart.py
+++ b/kickstart.py
@@ -631,6 +631,9 @@ class NetworkData(commands.network.RHEL6_NetworkData):
else:
dev.set (("ONBOOT", "no"))

+ if self.mtu:
+ dev.set(("MTU", self.mtu))
+
if self.ethtool:
dev.set(("ETHTOOL_OPTS", self.ethtool))

--
1.7.3.2

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 
Old 03-22-2011, 01:03 PM
Chris Lumens
 
Default Fix --mtu option to kickstart network command (#689081)

> See the Bugzilla issue in the subject. This is based on rhel6-branch (but I
> can rebase if that's wrong) and can be pulled from:
>
> git://github.com/icomfort/anaconda.git rhel6-fix-mtu
>
> Please keep me cc'd on any discussion; I am not subscribed to the list.
>
> -- >8 --
> mtu was not among the fields that NetworkData.execute was copying from the
> kickstart data structure to the NetworkDevice object.
> ---
> kickstart.py | 3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/kickstart.py b/kickstart.py
> index 3ce4ad1..5e3a15f 100644
> --- a/kickstart.py
> +++ b/kickstart.py
> @@ -631,6 +631,9 @@ class NetworkData(commands.network.RHEL6_NetworkData):
> else:
> dev.set (("ONBOOT", "no"))
>
> + if self.mtu:
> + dev.set(("MTU", self.mtu))
> +
> if self.ethtool:
> dev.set(("ETHTOOL_OPTS", self.ethtool))

The patch looks good. I've assigned the bug to rvykydal, who handles
most all of our networking stuff and he'll take a look at applying it.
Thanks.

- Chris

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 
Old 03-22-2011, 01:28 PM
Radek Vykydal
 
Default Fix --mtu option to kickstart network command (#689081)

On 03/22/2011 03:03 PM, Chris Lumens wrote:

The patch looks good. I've assigned the bug to rvykydal, who handles
most all of our networking stuff and he'll take a look at applying it.



Applied to master branch, thanks for the patch.

Radek

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 

Thread Tools




All times are GMT. The time now is 06:56 PM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org