----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60173/#review178265 -----------------------------------------------------------
Master (58ee6e4) is red with this patch. ./build-support/jenkins/build.sh :commons:classes :commons:jar :compileJava Download https://repo1.maven.org/maven2/com/h2database/h2/1.4.196/h2-1.4.196.pom Download https://repo1.maven.org/maven2/com/h2database/h2/1.4.196/h2-1.4.196.jar /home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/java/org/apache/aurora/scheduler/storage/log/WriteAheadStorage.java:74: Note: Wrote forwarder org.apache.aurora.scheduler.storage.log.WriteAheadStorageForwarder @Forward({ ^ Note: Writing file:/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/classes/main/org/apache/aurora/common/args/apt/cmdline.arg.info.txt.2 Note: Writing file:/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/classes/main/META-INF/compiler/resource-mappings/org.apache.aurora.common.args.apt.CmdLineProcessor :generateBuildProperties :processResources :classes :jar :startScripts :distTar :distZip :assemble :compileJmhJavaNote: /home/jenkins/jenkins-slave/workspace/AuroraBot/src/jmh/java/org/apache/aurora/benchmark/fakes/FakeSchedulerDriver.java uses or overrides a deprecated API. Note: Recompile with -Xlint:deprecation for details. :processJmhResources UP-TO-DATE :jmhClasses :checkstyleJmh :jsHint :checkstyleMain[ant:checkstyle] [ERROR] /home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/java/org/apache/aurora/scheduler/thrift/aop/AopModule.java:18:8: Unused import - com.google.common.annotations.VisibleForTesting. [UnusedImports] FAILED FAILURE: Build failed with an exception. * What went wrong: Execution failed for task ':checkstyleMain'. > Checkstyle rule violations were found. See the report at: > file:///home/jenkins/jenkins-slave/workspace/AuroraBot/dist/reports/checkstyle/main.html * 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: 1 mins 8.273 secs I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On June 19, 2017, 4:38 p.m., Jordan Ly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60173/ > ----------------------------------------------------------- > > (Updated June 19, 2017, 4:38 p.m.) > > > Review request for Aurora, David McLaughlin, Santhosh Kumar Shanmugham, > Stephan Erb, and Zameer Manji. > > > Repository: aurora > > > Description > ------- > > Allow for custom Thrift method interceptors to be injected via Guice modules. > Cluster operators might use this feature to inject interceptors that only > allow certain roles to call certain endpoints, or to dynamically check if a > job should be able to use a constraint. > > > Diffs > ----- > > RELEASE-NOTES.md e032f7927a68b00401ea8f073ff52b5def74f3ce > docs/reference/scheduler-configuration.md > 3d53c5a552e06f62a7572591fb0c92ccae42c54b > src/main/java/org/apache/aurora/scheduler/thrift/aop/AopModule.java > f59ee1a0514a6dc52573c0b932cba755e0a10e18 > > > Diff: https://reviews.apache.org/r/60173/diff/1/ > > > Testing > ------- > > Injected a custom module on a Vagrant box. > > Unit + integration tests pass. > > > Thanks, > > Jordan Ly > >
