----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38210/#review98476 -----------------------------------------------------------
specs/rpm/SOURCES/aurora.monit (line 13) <https://reviews.apache.org/r/38210/#comment154960> Start timeout 30s specs/rpm/SOURCES/aurora.monit (line 14) <https://reviews.apache.org/r/38210/#comment154961> 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) <https://reviews.apache.org/r/38210/#comment154956> Mind adding a start timeout of 30s? specs/rpm/SOURCES/thermos-observer.monit (line 2) <https://reviews.apache.org/r/38210/#comment154958> 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) <https://reviews.apache.org/r/38210/#comment154957> 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) <https://reviews.apache.org/r/38210/#comment154955> start_program = "/bin/bash -c '/bin/sleep 10 && /etc/init.d/thermos-observer start'" specs/rpm/aurora.spec (line 349) <https://reviews.apache.org/r/38210/#comment154959> 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 > >