Summary: | net-dialup/freeradius-2.2.5 - /etc/init.d/radiusd fails to restart radiusd after upgrade from 2.2.0 because PID file location changed | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Mark <mputtick> |
Component: | [OLD] Server | Assignee: | Gentoo Dialup Developers <net-dialup> |
Status: | RESOLVED CANTFIX | ||
Severity: | normal | CC: | jer |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | x86 | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- |
Description
Mark
2014-05-19 14:38:15 UTC
2.2.0 used /var/lib/run/radiusd and 2.2.5 fixed that so /run/ is used instead. I would have thought configuration file protection would have warned you that radiusd.conf had changed. (In reply to Jeroen Roovers from comment #1) > 2.2.0 used /var/lib/run/radiusd and 2.2.5 fixed that so /run/ is used > instead. I would have thought configuration file protection would have > warned you that radiusd.conf had changed. It may well have done and I'm willing to admit that I may have missed this at the beginning of the differences shown, although I did pick up on some other minor changes that I had to make. I thought I'd report it since I wasn't sure if the intention was for the config to point at '/var/lib' anyway, as the init scripts were not working as they should for shutting down. (In reply to Mark from comment #2) > (In reply to Jeroen Roovers from comment #1) > > 2.2.0 used /var/lib/run/radiusd and 2.2.5 fixed that so /run/ is used > > instead. I would have thought configuration file protection would have > > warned you that radiusd.conf had changed. > > It may well have done and I'm willing to admit that I may have missed this > at the beginning of the differences shown, although I did pick up on some > other minor changes that I had to make. I thought I'd report it since I > wasn't sure if the intention was for the config to point at '/var/lib' > anyway, as the init scripts were not working as they should for shutting > down. Also, when I ran etc-update, I replaced radiusd.conf with the new one and then modified the 2 lines that I needed to, rather than making the changes to the old file. I have reviewed all the init script changes and I didn't find any problems there that could be fixed in the scripts themselves. The PID files are now written to the location OpenRC expects them, and the change was both likely to cause this (temporary) breakage and necessary. There isn't much we can do about run time variables like PID file locations that normally rarely change. OpenRC could probably be taught to use the old init.d script to stop a service and then use the new init.d script to (re)start it. OK, thanks for investigating. |