> On April 4, 2016, 1:31 p.m., Aurora ReviewBot wrote: > > Master (9847b02) is green with this patch. > > ./build-support/jenkins/build.sh > > > > I will refresh this build result if you post a review containing > > "@ReviewBot retry" > > Ashwin Murthy wrote: > Bill, will you be merging? Thanks!
My mistake, for some reason i had convinced myself there was another reviewer. Thanks for the nudge! This will be on master shortly. - Bill ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45511/#review126933 ----------------------------------------------------------- On April 4, 2016, 12:43 p.m., Ashwin Murthy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45511/ > ----------------------------------------------------------- > > (Updated April 4, 2016, 12:43 p.m.) > > > Review request for Aurora and Bill Farner. > > > Repository: aurora > > > Description > ------- > > AURORA-1493: create ELB-friendly endpoint to detect leading scheduler. The > fix is to add a new endpoint - "/leaderhealth" which returns http status code > 200 (OK) if the instance is the leader. If the instance is not the leader but > a leading exists, returns 500 (Internal server error). If there is no leader > at all, returns 503 (Service unavailable) > > > Diffs > ----- > > RELEASE-NOTES.md 6fc3afeb5a9e2f2c2ba944fbc6d611d3494eb779 > docs/reference/scheduler-endpoints.md PRE-CREATION > src/main/java/org/apache/aurora/scheduler/http/JettyServerModule.java > df649ff4ffdc741afdbc4850c2dbf98bd3e218f1 > src/main/java/org/apache/aurora/scheduler/http/LeaderHealth.java > PRE-CREATION > src/main/resources/scheduler/assets/index.html > 861d5518890182737c6e545d6c409004c23cd18d > src/test/java/org/apache/aurora/scheduler/http/LeaderHealthTest.java > PRE-CREATION > src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh > e1c12bbd4382c31e576439f6693d82d5661029b9 > > Diff: https://reviews.apache.org/r/45511/diff/ > > > Testing > ------- > > Added new unit test > > > Thanks, > > Ashwin Murthy > >