foreman icon indicating copy to clipboard operation
foreman copied to clipboard

Feature request: restart processes when SIGHUP is received

Open perlun opened this issue 7 years ago • 5 comments

At the moment, SIGTERM, SIGINT and SIGHUP triggers a shutdown of Foreman.

I think it would be useful to be able to change the semantics for SIGHUP a bit, so that it instead will trigger a restart of all processes monitored by Foreman. That way, you can conveniently make it so that one of the watched processes sends a SIGHUP to itself which will make the entire set of processes restart, which can be very, very convenient sometmies.

@ddollar - would you accept a PR that implements this? I realize it's a semantical change, so if you want this to be opt-in rather than the default, that's an option as well.

perlun avatar Jun 01 '17 07:06 perlun

@perlun are you still looking for this feature?

andrewmcodes avatar Mar 18 '19 22:03 andrewmcodes

@andrewmcodes No longer using Foreman, so in practice, not really. But feel free to investigate it anyway if you like.

perlun avatar Mar 20 '19 08:03 perlun

I’m just trying to help out the maintainer. Would you mind closing if this is no longer an issue for you?

andrewmcodes avatar Mar 21 '19 04:03 andrewmcodes

@andrewmcodes It's not an "issue" in the sense that it causes problems for me, but it's still a useful feature I think we could keep on the backlog. Even though I have no use for it myself at the moment, this does not mean that the same holds true for everyone everywhere.

Nevertheless, it's up to you and the maintainer to determine what kind of issues you want to leave open on the GitHub project page.

perlun avatar Mar 29 '19 20:03 perlun

I’m just trying to help out the maintainer. Would you mind closing if this is no longer an issue for you?

I think it's will be a great feature. Currently I'm using Foreman to development and running four processes and from time to time I want to restart one of daemons.

shingonoide avatar Dec 16 '19 09:12 shingonoide