Summary: | /etc/init.d/net.eth? stops dhcpcd incorrectly | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Matt Boehlig <mboehlig> |
Component: | [OLD] Core system | Assignee: | Martin Schlemmer (RETIRED) <azarah> |
Status: | RESOLVED FIXED | ||
Severity: | minor | CC: | alwyn, grandmasterlinux, vapier, woodchip |
Priority: | Low | ||
Version: | 1.2 | ||
Hardware: | x86 | ||
OS: | Linux | ||
URL: | http://forums.gentoo.org/viewtopic.php?t=17056 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- |
Description
Matt Boehlig
2002-10-22 23:50:37 UTC
err, ill let azarah decide ;) I dont have a problem changing it. Problem though is that I do not use dhcp ... so cannot test this .... Anybody with dhcp experience willing to vouch on this ? Ok, what about if we are shutting down the machine (no reboot), we send a SIGHUP, but if we are rebooting, we send a TERM ? Let's do what you propose in comment #5, if it is easy to detect that. The kill -TERM `cat /var/run/dhcpcd-${IFACE}.pid` should indeed work. Alternative is to not do anything, and let init send the SIGTERM :) It will need some magic in /sbin/rc, but not too much ;) Ill have a look and attach a patch. Everybody ok if we use 1.8.6.4 as testing ground ? baselayout-1.8.6.5 should have: HALT=reboot or HALT=shutdown depending on what it is doing. net.eth0 have changed a bit though, so I did not do any changes to it. Am I correct that the approach proposed is different between whether you shut down or reboot? This is wrong. Whether I reboot or shutdown I always want to request the same IP address. I see only 2 options: 1) Always SIGTERM, so you always keep the same address. If the address you want is not available anymore then you just get a new one anyway. 2) Have a configuration option somewhere where the use indicates his preference e.g. KeepAddress = Yes/No. dhcpcd-1.3.22_p4-r1 now has a -z option which will allow shutdown of dhcpcd *but* will keep the cache file. So if baselayout starts using -z and depending on the new dhcpcd in some way, this problem should be resolved. Baselayout now uses -z. Resolving as fixed, as per comment #10 |