https://bugzilla.suse.com/show_bug.cgi?id=1173402 https://bugzilla.suse.com/show_bug.cgi?id=1173402#c17 --- Comment #17 from Thomas Blume <thomas.blume@suse.com> --- (In reply to Thomas Blume from comment #16)

(In reply to Fabian Vogt from comment #15)

...

...

Hm, did you check the latest code? There is:

if [ $? -eq 0 ] && [ -n "$(ls /tmp/leaseinfo.${netif}*)" ]; then

the leasinfo file won't be there if there was no dhcp setup.

Yes, but the use of $? there is still wrong AFAICT.

Ah, you mean when there is no ip= option at all and no bootdev either? But how should the network get set up via ifup then? Using rd.neednet without specifying an interface would make the script bail out early:

--> # Huh? No $1? [ -z "$1" ] && exit 1 --<

There is the ifname boot parameter, but that is again covered in another script, e.g. parse-ifname.sh.

Sorry, that reply was incomplete, $? is also needed in the do_dhcp case, because the function has parts where the return code is set explicitly. checking for leaseinfo is not the same, because this file is created by wicked in yet another function (dhcp_wicked_apply). -- You are receiving this mail because: You are the assignee for the bug.