Re: [systemd-devel] systemd restart patch

2011-02-08 Thread Lennart Poettering
On Thu, 03.02.11 21:35, Michael Biebl (mbi...@gmail.com) wrote: 2011/2/3 Cristian Patrascu cristian.patra...@windriver.com: So, for these last two features I created a patch, which adds two more options for a .service : - RestartRetries=n (where n is the number of restart retries) -

Re: [systemd-devel] systemd restart patch

2011-02-08 Thread Lennart Poettering
On Thu, 03.02.11 22:19, Michael Biebl (mbi...@gmail.com) wrote: 2011/2/3 Tomasz Torcz to...@pipebreaker.pl: On Thu, Feb 03, 2011 at 09:35:55PM +0100, Michael Biebl wrote: ExecStartOnFailure=/some/cmd (send an email or what not)  Isn't “OnFailure=send-email-to-admin.service” sufficent?

Re: [systemd-devel] systemd restart patch

2011-02-08 Thread Cristian Patrascu
On 02/08/2011 12:34 PM, Lennart Poettering wrote: On Mon, 07.02.11 10:01, Cristian Patrascu (cristian.patra...@windriver.com) wrote: On 02/03/2011 11:19 PM, Michael Biebl wrote: 2011/2/3 Tomasz Torczto...@pipebreaker.pl: On Thu, Feb 03, 2011 at 09:35:55PM +0100, Michael Biebl wrote:

Re: [systemd-devel] systemd restart patch

2011-02-07 Thread Cristian Patrascu
On 02/03/2011 11:19 PM, Michael Biebl wrote: 2011/2/3 Tomasz Torczto...@pipebreaker.pl: On Thu, Feb 03, 2011 at 09:35:55PM +0100, Michael Biebl wrote: ExecStartOnFailure=/some/cmd (send an email or what not) Isn't “OnFailure=send-email-to-admin.service” sufficent? Having to use a service

[systemd-devel] systemd restart patch

2011-02-03 Thread Cristian Patrascu
In the company I work for, I must have/implement the following functionality of a systemd service: - when a service stops with an exit code not 0, that is when it fails, it must be restarted. This is implemented, by using the option *Restart=on-failure*. Eventually, also *RestartSec=* can be

Re: [systemd-devel] systemd restart patch

2011-02-03 Thread Michael Biebl
2011/2/3 Cristian Patrascu cristian.patra...@windriver.com: So, for these last two features I created a patch, which adds two more options for a .service : - RestartRetries=n (where n is the number of restart retries) - ExecPostRestarts=/some/cmd (where the command /some/cmd will be executed

Re: [systemd-devel] systemd restart patch

2011-02-03 Thread Tomasz Torcz
On Thu, Feb 03, 2011 at 09:35:55PM +0100, Michael Biebl wrote: 2011/2/3 Cristian Patrascu cristian.patra...@windriver.com: So, for these last two features I created a patch, which adds two more options for a .service : - RestartRetries=n (where n is the number of restart retries) -

Re: [systemd-devel] systemd restart patch

2011-02-03 Thread Michael Biebl
2011/2/3 Tomasz Torcz to...@pipebreaker.pl: On Thu, Feb 03, 2011 at 09:35:55PM +0100, Michael Biebl wrote: ExecStartOnFailure=/some/cmd (send an email or what not)  Isn't “OnFailure=send-email-to-admin.service” sufficent? Having to use a service file for this use case is imho cumbersome,