Review Request 54081: Added `--pidfile` option to master and agent binaries.

2016-11-25 Thread Ilya Pronin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54081/ --- Review request for mesos, Benjamin Mahler and Vinod Kone. Bugs: MESOS-1648

Re: Review Request 54081: Added `--pidfile` option to master and agent binaries.

2016-11-25 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54081/#review156935 --- Patch looks great! Reviews applied: [54081] Passed command: expo

Re: Review Request 54081: Added `--pidfile` option to master and agent binaries.

2016-11-28 Thread Ilya Pronin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54081/ --- (Updated Nov. 28, 2016, 12:15 p.m.) Review request for mesos, Benjamin Mahler a

Re: Review Request 54081: Added `--pidfile` option to master and agent binaries.

2016-11-28 Thread Ilya Pronin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54081/ --- (Updated Nov. 28, 2016, 12:16 p.m.) Review request for mesos, Benjamin Mahler a

Re: Review Request 54081: Added `--pidfile` option to master and agent binaries.

2016-11-28 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54081/#review157030 --- Patch looks great! Reviews applied: [54081] Passed command: expo

Re: Review Request 54081: Added `--pidfile` option to master and agent binaries.

2016-12-06 Thread Benjamin Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54081/#review158240 --- Thanks Ilya! Have you looked at other pidfile related libraries?

Re: Review Request 54081: Added `--pidfile` option to master and agent binaries.

2016-12-07 Thread Ilya Pronin
> On Dec. 6, 2016, 10:42 p.m., Benjamin Mahler wrote: > > Thanks Ilya! > > > > Have you looked at other pidfile related libraries? Looks like BSD provides > > some functions for this (they're also available on Linux): > > https://www.freebsd.org/cgi/man.cgi?query=pidfile&sektion=3&manpath=FreeB

Re: Review Request 54081: Added `--pidfile` option to master and agent binaries.

2016-12-12 Thread Alexander Rukletsov
> On Dec. 6, 2016, 10:42 p.m., Benjamin Mahler wrote: > > Thanks Ilya! > > > > Have you looked at other pidfile related libraries? Looks like BSD provides > > some functions for this (they're also available on Linux): > > https://www.freebsd.org/cgi/man.cgi?query=pidfile&sektion=3&manpath=FreeB

Re: Review Request 54081: Added `--pidfile` option to master and agent binaries.

2016-12-12 Thread Ilya Pronin
> On Dec. 6, 2016, 10:42 p.m., Benjamin Mahler wrote: > > Thanks Ilya! > > > > Have you looked at other pidfile related libraries? Looks like BSD provides > > some functions for this (they're also available on Linux): > > https://www.freebsd.org/cgi/man.cgi?query=pidfile&sektion=3&manpath=FreeB

Re: Review Request 54081: Added `--pidfile` option to master and agent binaries.

2016-12-12 Thread Benjamin Mahler
> On Dec. 6, 2016, 10:42 p.m., Benjamin Mahler wrote: > > Thanks Ilya! > > > > Have you looked at other pidfile related libraries? Looks like BSD provides > > some functions for this (they're also available on Linux): > > https://www.freebsd.org/cgi/man.cgi?query=pidfile&sektion=3&manpath=FreeB

Re: Review Request 54081: Added `--pidfile` option to master and agent binaries.

2016-12-14 Thread Ilya Pronin
> On Dec. 6, 2016, 10:42 p.m., Benjamin Mahler wrote: > > Thanks Ilya! > > > > Have you looked at other pidfile related libraries? Looks like BSD provides > > some functions for this (they're also available on Linux): > > https://www.freebsd.org/cgi/man.cgi?query=pidfile&sektion=3&manpath=FreeB

Re: Review Request 54081: Added `--pidfile` option to master and agent binaries.

2017-01-10 Thread Greg Mann
> On Dec. 6, 2016, 10:42 p.m., Benjamin Mahler wrote: > > Thanks Ilya! > > > > Have you looked at other pidfile related libraries? Looks like BSD provides > > some functions for this (they're also available on Linux): > > https://www.freebsd.org/cgi/man.cgi?query=pidfile&sektion=3&manpath=FreeB

Re: Review Request 54081: Added `--pidfile` option to master and agent binaries.

2017-03-01 Thread Ilya Pronin
> On Dec. 6, 2016, 10:42 p.m., Benjamin Mahler wrote: > > Thanks Ilya! > > > > Have you looked at other pidfile related libraries? Looks like BSD provides > > some functions for this (they're also available on Linux): > > https://www.freebsd.org/cgi/man.cgi?query=pidfile&sektion=3&manpath=FreeB