----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37847/#review96774 -----------------------------------------------------------
Master (06ddaad) is red with this patch. ./build-support/jenkins/build.sh :processJmhResources UP-TO-DATE :jmhClasses :checkstyleJmh :jsHint :checkstyleMain :compileTestJava :processTestResources :testClasses :checkstyleTest :findbugsJmh :findbugsMain :findbugsTest :licenseJmh UP-TO-DATE :licenseMain UP-TO-DATE :licenseTest UP-TO-DATE :license UP-TO-DATE :pmdMain :test :jacocoTestReport Coverage report generated: file:///home/jenkins/jenkins-slave/workspace/AuroraBot/dist/reports/jacoco/test/html/index.html :analyzeReport Instruction coverage of 0.8962987319920261 exceeds minimum coverage of 0.89. Branch coverage is 0.823905109489051, but must be greater than 0.835 :analyzeReport FAILED FAILURE: Build failed with an exception. * What went wrong: Execution failed for task ':analyzeReport'. > Test coverage missing for org/apache/aurora/scheduler/http/Agents Test coverage missing for org/apache/aurora/scheduler/http/Agents$1 Test coverage missing for org/apache/aurora/scheduler/http/Agents$Agent * Try: Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. BUILD FAILED Total time: 4 mins 12.543 secs I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On Aug. 27, 2015, 6:55 p.m., Kevin Sweeney wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37847/ > ----------------------------------------------------------- > > (Updated Aug. 27, 2015, 6:55 p.m.) > > > Review request for Aurora, Joshua Cohen and Bill Farner. > > > Bugs: AURORA-1449 and AURORA-1450 > https://issues.apache.org/jira/browse/AURORA-1449 > https://issues.apache.org/jira/browse/AURORA-1450 > > > Repository: aurora > > > Description > ------- > > Change the UI to refer to Mesos Agents instead of Mesos Slaves. > > This change also moves the /slaves endpoint to /agents and adds a > compatibility redirect. > > > Diffs > ----- > > src/main/java/org/apache/aurora/scheduler/http/JettyServerModule.java > 05af127a91b6c0e8db089cb828c38e556b07d45f > src/main/java/org/apache/aurora/scheduler/http/Slaves.java > a9fef25595d616f0fbbb7c5ad6de39184a60002a > src/main/resources/org/apache/aurora/scheduler/http/slaves.st > 854a9d4a3dc0be6bdf0ea0ad6194ffce48039919 > src/main/resources/org/apache/aurora/scheduler/http/utilization.st > 4a8a07222d65681e77c90566be9dce9283c7be65 > src/main/resources/scheduler/assets/index.html > d0bd897524a4c763d11a601898b663fec672e979 > src/main/resources/scheduler/assets/js/filters.js > bf6aef86fd50ba38b64f5d9f481d6915c38881f7 > > Diff: https://reviews.apache.org/r/37847/diff/ > > > Testing > ------- > > Tested /slaves and /agents in vagrant. > > > Thanks, > > Kevin Sweeney > >