Re: Review Request 45850: Factor out a discovery package.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45850/#review127529 --- Ship it! Master (11d5a72) is green with this patch. ./build-support/jenkins/build.sh I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On April 7, 2016, 12:43 a.m., John Sirois wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45850/ > --- > > (Updated April 7, 2016, 12:43 a.m.) > > > Review request for Aurora and Bill Farner. > > > Bugs: AURORA-1468 > https://issues.apache.org/jira/browse/AURORA-1468 > > > Repository: aurora > > > Description > --- > > This will be the home for the Curator implementations. > > In the process, lift the `ServerSetMonitor` to a top-level class and add > a test. Also tighten up the `ServiceDiscoveryModule` and make > requirements and exports clear. > > src/main/java/org/apache/aurora/scheduler/app/SchedulerMain.java > | 1 + > > src/main/java/org/apache/aurora/scheduler/discovery/CommonsServerGroupMonitor.java >| 59 ++ > src/main/java/org/apache/aurora/scheduler/{app => > discovery}/ServiceDiscoveryModule.java | 61 +++--- > src/test/java/org/apache/aurora/scheduler/app/SchedulerIT.java > | 1 + > > src/test/java/org/apache/aurora/scheduler/discovery/CommonsServerGroupMonitorTest.java >| 137 ++ > 5 files changed, 216 insertions(+), 43 deletions(-) > > > Diffs > - > > src/main/java/org/apache/aurora/scheduler/app/SchedulerMain.java > 11f6ad14eb334ce3e6390ca5754add1c0998d64e > src/main/java/org/apache/aurora/scheduler/app/ServiceDiscoveryModule.java > 73695cd125078f02737762c25abe8639066b4948 > > src/main/java/org/apache/aurora/scheduler/discovery/CommonsServerGroupMonitor.java > PRE-CREATION > src/test/java/org/apache/aurora/scheduler/app/SchedulerIT.java > 5b7775005eb813493de3cb79ca3415c196d806c5 > > src/test/java/org/apache/aurora/scheduler/discovery/CommonsServerGroupMonitorTest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/45850/diff/ > > > Testing > --- > > Locally green: > ``` > ./gradlew -Pq build > ./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh > ``` > > > Thanks, > > John Sirois > >
Re: Review Request 45850: Factor out a discovery package.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45850/#review127527 --- Ship it! Ship It! - Bill Farner On April 6, 2016, 5:43 p.m., John Sirois wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45850/ > --- > > (Updated April 6, 2016, 5:43 p.m.) > > > Review request for Aurora and Bill Farner. > > > Bugs: AURORA-1468 > https://issues.apache.org/jira/browse/AURORA-1468 > > > Repository: aurora > > > Description > --- > > This will be the home for the Curator implementations. > > In the process, lift the `ServerSetMonitor` to a top-level class and add > a test. Also tighten up the `ServiceDiscoveryModule` and make > requirements and exports clear. > > src/main/java/org/apache/aurora/scheduler/app/SchedulerMain.java > | 1 + > > src/main/java/org/apache/aurora/scheduler/discovery/CommonsServerGroupMonitor.java >| 59 ++ > src/main/java/org/apache/aurora/scheduler/{app => > discovery}/ServiceDiscoveryModule.java | 61 +++--- > src/test/java/org/apache/aurora/scheduler/app/SchedulerIT.java > | 1 + > > src/test/java/org/apache/aurora/scheduler/discovery/CommonsServerGroupMonitorTest.java >| 137 ++ > 5 files changed, 216 insertions(+), 43 deletions(-) > > > Diffs > - > > src/main/java/org/apache/aurora/scheduler/app/SchedulerMain.java > 11f6ad14eb334ce3e6390ca5754add1c0998d64e > src/main/java/org/apache/aurora/scheduler/app/ServiceDiscoveryModule.java > 73695cd125078f02737762c25abe8639066b4948 > > src/main/java/org/apache/aurora/scheduler/discovery/CommonsServerGroupMonitor.java > PRE-CREATION > src/test/java/org/apache/aurora/scheduler/app/SchedulerIT.java > 5b7775005eb813493de3cb79ca3415c196d806c5 > > src/test/java/org/apache/aurora/scheduler/discovery/CommonsServerGroupMonitorTest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/45850/diff/ > > > Testing > --- > > Locally green: > ``` > ./gradlew -Pq build > ./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh > ``` > > > Thanks, > > John Sirois > >
Review Request 45850: Factor out a discovery package.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45850/ --- Review request for Aurora and Bill Farner. Bugs: AURORA-1468 https://issues.apache.org/jira/browse/AURORA-1468 Repository: aurora Description --- This will be the home for the Curator implementations. In the process, lift the `ServerSetMonitor` to a top-level class and add a test. Also tighten up the `ServiceDiscoveryModule` and make requirements and exports clear. src/main/java/org/apache/aurora/scheduler/app/SchedulerMain.java | 1 + src/main/java/org/apache/aurora/scheduler/discovery/CommonsServerGroupMonitor.java | 59 ++ src/main/java/org/apache/aurora/scheduler/{app => discovery}/ServiceDiscoveryModule.java | 61 +++--- src/test/java/org/apache/aurora/scheduler/app/SchedulerIT.java | 1 + src/test/java/org/apache/aurora/scheduler/discovery/CommonsServerGroupMonitorTest.java | 137 ++ 5 files changed, 216 insertions(+), 43 deletions(-) Diffs - src/main/java/org/apache/aurora/scheduler/app/SchedulerMain.java 11f6ad14eb334ce3e6390ca5754add1c0998d64e src/main/java/org/apache/aurora/scheduler/app/ServiceDiscoveryModule.java 73695cd125078f02737762c25abe8639066b4948 src/main/java/org/apache/aurora/scheduler/discovery/CommonsServerGroupMonitor.java PRE-CREATION src/test/java/org/apache/aurora/scheduler/app/SchedulerIT.java 5b7775005eb813493de3cb79ca3415c196d806c5 src/test/java/org/apache/aurora/scheduler/discovery/CommonsServerGroupMonitorTest.java PRE-CREATION Diff: https://reviews.apache.org/r/45850/diff/ Testing --- Locally green: ``` ./gradlew -Pq build ./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh ``` Thanks, John Sirois