----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39525/#review103489 -----------------------------------------------------------
Master (5609fc2) is red with this patch. ./build-support/jenkins/build.sh :commons:processResources :commons:classes :commons:jar :compileJavaNote: 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] /home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/java/org/apache/aurora/scheduler/http/api/GsonMessageBodyHandler.java:24:8: error: Unused import - java.nio.ByteBuffer. [ant:checkstyle] /home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/java/org/apache/aurora/scheduler/http/api/GsonMessageBodyHandler.java:43:8: error: Unused import - com.google.common.io.BaseEncoding. [ant:checkstyle] /home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/java/org/apache/aurora/scheduler/http/api/GsonMessageBodyHandler.java:55:8: error: Unused import - com.google.gson.TypeAdapter. [ant:checkstyle] /home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/java/org/apache/aurora/scheduler/http/api/GsonMessageBodyHandler.java:56:8: error: Unused import - com.google.gson.stream.JsonReader. [ant:checkstyle] /home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/java/org/apache/aurora/scheduler/http/api/GsonMessageBodyHandler.java:57:8: error: Unused import - com.google.gson.stream.JsonWriter. 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.xml * 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 42.2 secs I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On Oct. 21, 2015, 10:29 p.m., Kevin Sweeney wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39525/ > ----------------------------------------------------------- > > (Updated Oct. 21, 2015, 10:29 p.m.) > > > Review request for Aurora, Maxim Khutornenko and Zameer Manji. > > > Repository: aurora > > > Description > ------- > > This change ignores all SessionKeys. Since the scheduler was doing > "requireNonNull" on SessionKey before the client needs to continue to set > SessionKey to a non-null value for this release. In the next release the > parameter can be dropped completely. > > Apologies for the large diff, thankfully it's mostly red. Coverage in > SchedulerThriftInterface is very high, most of the deleted tests deal with > branches related to authentication or authorization failures, as these are > now enforced in a different layer. > > > Diffs > ----- > > NEWS 7622f9db706c1764876f4174855343414dc59366 > src/main/java/org/apache/aurora/auth/CapabilityValidator.java > 198cdf3a63195c6b125d6a2e5d726075fd41f809 > src/main/java/org/apache/aurora/auth/SessionValidator.java > b688a0f0f1a4549dc4fe81f5cc50af1167856e87 > src/main/java/org/apache/aurora/auth/UnsecureAuthModule.java > 476dd9a89ce317e3bd242fd70a55cc919aff8418 > src/main/java/org/apache/aurora/auth/UnsecureSessionContext.java > 47f81aa4851c753779918dfeb137041073127cd4 > src/main/java/org/apache/aurora/scheduler/app/MoreModules.java > b79ccc849834c9a7f7ac654e4302f2ba4e13ff66 > src/main/java/org/apache/aurora/scheduler/http/JettyServerModule.java > ee024a34f1fdde49791ad6573fe5c0ae3953a7c7 > > src/main/java/org/apache/aurora/scheduler/http/api/GsonMessageBodyHandler.java > 2ec7da611db871749a79e2e93c1a2b988a1ca578 > src/main/java/org/apache/aurora/scheduler/thrift/AuditMessages.java > 3a29a628de8f1c1c3ed609b494656ac95db9efcf > > src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java > 304437e95dd1692b65bf5d0a6697ab62127e6833 > src/main/java/org/apache/aurora/scheduler/thrift/aop/AopModule.java > e951f8e55353ff3c8d7d0e86016a2a004137c299 > > src/main/java/org/apache/aurora/scheduler/thrift/aop/LoggingInterceptor.java > d110b21d4193eb37a10ee8ba2eca82d208496c38 > > src/main/java/org/apache/aurora/scheduler/thrift/aop/UserCapabilityInterceptor.java > dd8d272751034ef7daa720e12dd4ed850532da39 > src/main/java/org/apache/aurora/scheduler/thrift/auth/Requires.java > 9b3431899cddfc7f949fc8cb12f6e4739b4989d0 > src/main/java/org/apache/aurora/scheduler/thrift/auth/ThriftAuthModule.java > e1a21f632f2bee5834cbbde9d6902d71b6a090d9 > src/test/java/org/apache/aurora/auth/UnsecureSessionContextTest.java > c3c34656d781bc54282206caf1211d1385fd68c8 > src/test/java/org/apache/aurora/scheduler/http/api/ApiBetaTest.java > 8af7c46e46d2269903118deb2a2a2b61bcc1d8d1 > src/test/java/org/apache/aurora/scheduler/thrift/AuditMessagesTest.java > 0f8748d2981d48236f602778d2c1848b4c572445 > > src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java > f537f7a39496f69953e6d86cb15359fe3a705480 > src/test/java/org/apache/aurora/scheduler/thrift/ThriftIT.java > f06481bd237db0ab84850fe987d6e1fcb194f31b > src/test/java/org/apache/aurora/scheduler/thrift/aop/AopModuleTest.java > b940e82644365fae613d577ba5b8779077f115d4 > > src/test/java/org/apache/aurora/scheduler/thrift/aop/LoggingInterceptorTest.java > 1bb3eae224de287d0ed83534f1c933ddd72298bb > > Diff: https://reviews.apache.org/r/39525/diff/ > > > Testing > ------- > > ./graldew -Pq clean build > > > Thanks, > > Kevin Sweeney > >