On Sat, Dec 14, 2013 at 08:24:18PM +0000, Ian Jackson wrote:
> Steve Langasek writes ("Re: Bug#732124: "initctl reload" behaviour should be
> configurable in job file"):
> > On Sat, Dec 14, 2013 at 12:49:01PM +0000, Ian Jackson wrote:
> > > Package: upstart
> > > Version: 1.20-1
> > > Severity: wishlist> > (Where does this version number come from? Is this just a transposition > > error of 1.10-2, the current version in tetisting/unstable?) > Yes. Oh bum. Shall I fix all of the reports I submitted today ? Yes please :) > > Do you consider fixing the manpage to be a sufficient solution for this bug, > > or do you think that support for arbitrary reload commands is needed? For > > my part I don't think that there's a need for abstract 'reload's when one > > can do a 'restart' instead. > I haven't done a survey but I imagine there are daemons out there > which don't like being randomly HUPped. Of course they could be > fixed, but asking them to handle a signal is rather unfriendly. Agreed; but a 'reload' is something that happens only under admin direction, and the upstart job syntax gives maintainers the flexibility to set a different reload signal if appropriate - remembering that 'reload' has always been an optional init.d command in Debian for precisely the reason that many services don't support a graceful reload, I think seasoned admins are unlikely to use it and are instead probably going to use 'restart'. The common case for an unhandled SIGHUP is to kill the process anyway, at which point upstart will restart it if the job is set up with 'respawn'... so I think the default semantics are pretty good for the common case (now) and we mostly just need to step up the documentation. > So I think the best answer would be to provide a command-based > override. But it's hardly a high priority and if you feel like > closing this bug on the basis that it's a wishlist item you're not > going to get round to, I won't mind. Ok. I'll consider this bug resolved once the documentation is fixed. Thanks, -- Steve Langasek Give me a lever long enough and a Free OS Debian Developer to set it on, and I can move the world. Ubuntu Developer http://www.debian.org/ [email protected] [email protected]
signature.asc
Description: Digital signature
