Re: Review Request 38210: [aurora-packaging] Adds Monit supervisor support for EL6

[Stephan Erb]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38210/#review172215
---



We are only officially supporing Centos-7 as of today. It relies on systemd and 
thus would not require monit. Please feel free to re-open if you disagree.

- Stephan Erb


On Sept. 9, 2015, 8:56 a.m., Steve Salevan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38210/
> ---
> 
> (Updated Sept. 9, 2015, 8:56 a.m.)
> 
> 
> Review request for Aurora, Joe Smith, Kevin Sweeney, and Bill Farner.
> 
> 
> Bugs: AURORA-1480
> https://issues.apache.org/jira/browse/AURORA-1480
> 
> 
> Repository: aurora-packaging
> 
> 
> Description
> ---
> 
> As requested by AURORA-1480, this RB adds Monit as a runtime requirement for 
> the aurora and aurora-thermos packages alongside some basic Monit scripts to 
> properly supervise these processes.
> 
> Let me know what you think and thanks!
> 
> 
> Diffs
> -
> 
>   specs/rpm/SOURCES/aurora.monit PRE-CREATION 
>   specs/rpm/SOURCES/thermos-observer.monit PRE-CREATION 
>   specs/rpm/aurora.spec cf9c2715106a27fcb71447b5453ad3b847d2235a 
> 
> 
> Diff: https://reviews.apache.org/r/38210/diff/1/
> 
> 
> Testing
> ---
> 
> Tested Monit scripts against a local machine
> 
> 
> Thanks,
> 
> Steve Salevan
> 
>

Re: Review Request 38210: [aurora-packaging] Adds Monit supervisor support for EL6

[Joe Smith]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38210/#review98476
---



specs/rpm/SOURCES/aurora.monit (line 13)


Start timeout 30s



specs/rpm/SOURCES/aurora.monit (line 14)


if failed port 8081 send "GET /health HTTP/1.0\r\nUser-Agent: 
monit\r\n\r\n" expect "OK\n" with timeout 2 seconds for 5 cycles then restart



specs/rpm/SOURCES/thermos-observer.monit (line 1)


Mind adding a start timeout of 30s?



specs/rpm/SOURCES/thermos-observer.monit (line 2)


What's the default poll interval? I saw some references to it being set in 
~/.monitrc, but that seems to be off what we actually want. Can we set it to 
60s?



specs/rpm/SOURCES/thermos-observer.monit (line 14)


In practice, we've had ~good luck with:

if failed port 1338 protocol http and request "/health" with timeout 2 
seconds for 10 cycles then restart



specs/rpm/SOURCES/thermos-observer.monit (line 15)


start_program = "/bin/bash -c '/bin/sleep 10 && 
/etc/init.d/thermos-observer start'"



specs/rpm/aurora.spec (line 349)


This should constitute a -2, right?


- Joe Smith


On Sept. 8, 2015, 11:56 p.m., Steve Salevan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38210/
> ---
> 
> (Updated Sept. 8, 2015, 11:56 p.m.)
> 
> 
> Review request for Aurora, Joe Smith, Kevin Sweeney, and Bill Farner.
> 
> 
> Bugs: AURORA-1480
> https://issues.apache.org/jira/browse/AURORA-1480
> 
> 
> Repository: aurora-packaging
> 
> 
> Description
> ---
> 
> As requested by AURORA-1480, this RB adds Monit as a runtime requirement for 
> the aurora and aurora-thermos packages alongside some basic Monit scripts to 
> properly supervise these processes.
> 
> Let me know what you think and thanks!
> 
> 
> Diffs
> -
> 
>   specs/rpm/SOURCES/aurora.monit PRE-CREATION 
>   specs/rpm/SOURCES/thermos-observer.monit PRE-CREATION 
>   specs/rpm/aurora.spec cf9c2715106a27fcb71447b5453ad3b847d2235a 
> 
> Diff: https://reviews.apache.org/r/38210/diff/
> 
> 
> Testing
> ---
> 
> Tested Monit scripts against a local machine
> 
> 
> Thanks,
> 
> Steve Salevan
> 
>