Bug 230157

Summary: initscripts fail to change hostname from localhost.localdomain
Product: [Fedora] Fedora Reporter: Michal Jaegermann <michal>
Component: initscriptsAssignee: Bill Nottingham <notting>
Status: CLOSED RAWHIDE QA Contact: Brock Organ <borgan>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: rvokal
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: 8.52-1 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-04-16 22:44:37 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
change to make ifup-post to work again none

Description Michal Jaegermann 2007-02-26 22:15:24 UTC 
Description of problem:

After recent initscripts update starting up a network failed to
rename a machine with a DHCP assigned address from 'localhost'.
The culprit turned out to be ifup-post.  It find address
by doing this:

 IPADDR=$(LANG=C ip -o -4 addr ls dev ${DEVICE} | awk '{ print $4 }')

If ${DEVICE} in question happen to have also some additional alias
addresses this returns all of these and makes a subsequnt call
to 'ipcalc' very unhappy.  A solution is to use only the first address
as reported by 'ip'.  A simple patch is attached.

BTW - earlier in 'ifup-post' there is the following line:

if [ "$PEERDNS" != "no" -o -n "$RESOLV_MODS" -a "$RESOLV_MODS" != "no" ]; then

Is this really what is supposed to be there or rather

if [ "$PEERDNS" != "no" ] || \
     [ -n "$RESOLV_MODS" -a "$RESOLV_MODS" != "no" ]; then

was really meant?  Quite possibly what is there is fine but a logic
is surely not so clear on the first look.

Version-Release number of selected component (if applicable):
initscripts-8.51-1
Comment 1 Michal Jaegermann 2007-02-26 22:15:24 UTC 
Created attachment 148836 [details]
change to make ifup-post to work again
Comment 2 Bill Nottingham 2007-04-16 22:44:37 UTC 
Added in CVS, will be in 8.52-1. Thanks!