aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Aurora ReviewBot" <wfar...@apache.org>
Subject Re: Review Request 34967: Use TaskStatus Reason to set memory limit message.
Date Wed, 03 Jun 2015 01:03:05 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34967/#review86337
-----------------------------------------------------------


Master (54d561e) is red with this patch.
  ./build-support/jenkins/build.sh

: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
:compileTestJavaNote: /home/jenkins/jenkins-slave/workspace/AuroraBot/src/test/java/org/apache/aurora/scheduler/UserTaskLauncherTest.java
uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.

:processTestResources
:testClasses
:checkstyleTest[ant:checkstyle] /home/jenkins/jenkins-slave/workspace/AuroraBot/src/test/java/org/apache/aurora/scheduler/UserTaskLauncherTest.java:203:
'block' child have incorrect indentation level 6, expected level should be 8.
[ant:checkstyle] /home/jenkins/jenkins-slave/workspace/AuroraBot/src/test/java/org/apache/aurora/scheduler/UserTaskLauncherTest.java:203:
'latch' have incorrect indentation level 6, expected level should be 8.
[ant:checkstyle] /home/jenkins/jenkins-slave/workspace/AuroraBot/src/test/java/org/apache/aurora/scheduler/UserTaskLauncherTest.java:203:
'method call' child have incorrect indentation level 6, expected level should be 8.
[ant:checkstyle] /home/jenkins/jenkins-slave/workspace/AuroraBot/src/test/java/org/apache/aurora/scheduler/UserTaskLauncherTest.java:204:
'block' child have incorrect indentation level 6, expected level should be 8.
[ant:checkstyle] /home/jenkins/jenkins-slave/workspace/AuroraBot/src/test/java/org/apache/aurora/scheduler/UserTaskLauncherTest.java:204:
'return' have incorrect indentation level 6, expected level should be 8.
[ant:checkstyle] /home/jenkins/jenkins-slave/workspace/AuroraBot/src/test/java/org/apache/aurora/scheduler/UserTaskLauncherTest.java:205:
'block rcurly' have incorrect indentation level 4, expected level should be 6.
 FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':checkstyleTest'.
> Checkstyle rule violations were found. See the report at: file:///home/jenkins/jenkins-slave/workspace/AuroraBot/dist/reports/checkstyle/test.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 53.047 secs


I will refresh this build result if you post a review containing "@ReviewBot retry"

- Aurora ReviewBot


On June 3, 2015, 12:58 a.m., Zameer Manji wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/34967/
> -----------------------------------------------------------
> 
> (Updated June 3, 2015, 12:58 a.m.)
> 
> 
> Review request for Aurora and Maxim Khutornenko.
> 
> 
> Bugs: AURORA-1341
>     https://issues.apache.org/jira/browse/AURORA-1341
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Use TaskStatus Reason to set memory limit message instead of checking the contents of
the message field. Future versions of Mesos can change the diagnostic information in the message
field causing Aurora to display no information when a task fails.
> 
> 
> Diffs
> -----
> 
>   src/main/java/org/apache/aurora/scheduler/UserTaskLauncher.java 5af691d0ac959cf4b5d01752daf996803e91ed16

>   src/test/java/org/apache/aurora/scheduler/UserTaskLauncherTest.java fe2fc954350b42487151fc820ebad22a41aeb039

> 
> Diff: https://reviews.apache.org/r/34967/diff/
> 
> 
> Testing
> -------
> 
> ./gradlew test -Pq
> 
> 
> Thanks,
> 
> Zameer Manji
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message