Bug 142041 - KickStart with network --ethtool="..." doesn't quote ETHTOOL_OPTS
Summary: KickStart with network --ethtool="..." doesn't quote ETHTOOL_OPTS
Alias: None
Product: Red Hat Enterprise Linux 3
Classification: Red Hat
Component: anaconda (Show other bugs)
(Show other bugs)
Version: 3.0
Hardware: All Linux
Target Milestone: ---
Assignee: Paul Nasrat
QA Contact: Mike McLean
Whiteboard: RHEL3U5
Depends On: 142045
Blocks: 132991
TreeView+ depends on / blocked
Reported: 2004-12-06 21:19 UTC by John Kerbawy
Modified: 2007-11-30 22:07 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2005-05-18 14:28:57 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

External Trackers
Tracker ID Priority Status Summary Last Updated
Red Hat Product Errata RHEA-2005:191 normal SHIPPED_LIVE anaconda enhancement update 2005-05-18 04:00:00 UTC

Description John Kerbawy 2004-12-06 21:19:32 UTC
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.5)
Gecko/20041107 Firefox/1.0

Description of problem:
While doing a KickStart and passing --ethtool="..." to the "network"
keyword in the KickStart configuration, the resulting value in
ETHTOOL_OPTS isn't quoted properly causing ifup to error on boot.

network --bootproto=dhcp --ethtool="duplex full speed 100"

This little patch seems to fix the problem for me.

--- network.py.orig     2004-12-02 18:59:23.000000000 -0500
+++ network.py  2004-12-02 19:08:28.000000000 -0500
@@ -114,7 +114,7 @@
                s = s + key + "=" + 'no' + "\n"
             # make sure we include autoneg in the ethtool line
             elif key == 'ETHTOOL_OPTS' and
self.info[key].find("autoneg")== -1:
-                s = s + key + "=" + "autoneg off " + self.info[key] +
+                s = s + key + """="autoneg off %s"\n""" %
                s = s + key + "=" + self.info[key] + "\n"

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:
1. KickStart with network --ethtool="duplex full speed 100"
2. Reboot and view /etc/sysconfig/network-scripts/ifcfg-eth0
3. Run ifup eth0 and watch for "/sbin/ifup: line 4: off: command not
found" because the ETHTOOL_OPTS value isn't quoted.

Actual Results:  See above.

Additional info:

I tested this by modifying network.py and network.pyc in stage2.img,
netstg2.img, and hdstg2.img instead of properly rebuilding anaconda.
 I don't remember where I found out about --ethtool, and maybe it
isn't supported, but the current quoting in network.py doesn't make
sense to me.

Comment 2 Paul Nasrat 2005-01-04 20:19:45 UTC
Thanks for the patch commited.  Targetting for U5

Comment 4 Dennis Gregorovic 2005-05-18 14:28:58 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on the solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.


Note You need to log in before you can comment on or make changes to this bug.