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 52588: Enable per task volume mounts via scheduler API
Date Thu, 06 Oct 2016 01:12:38 GMT

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



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

:commons:jar
:compileJava/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] /home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/java/org/apache/aurora/scheduler/configuration/ConfigurationManager.java:242:
error: Line is longer than 100 characters (found 107).
[ant:checkstyle] /home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/java/org/apache/aurora/scheduler/storage/db/migration/V009_CreateContainerVolumesTable.java:1:
error: Line does not match expected header line of '^\/\*\*$'.
[ant:checkstyle] /home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/java/org/apache/aurora/scheduler/storage/db/typehandlers/VolumeModeTypeHandler.java:1:
error: Missing a header - not enough lines in file.
[ant:checkstyle] /home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/java/org/apache/aurora/scheduler/storage/db/views/DbContainer.java:31:
error: File contains a sequence of empty lines.
 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 23.586 secs


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

- Aurora ReviewBot


On Oct. 6, 2016, 1:07 a.m., Zameer Manji wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52588/
> -----------------------------------------------------------
> 
> (Updated Oct. 6, 2016, 1:07 a.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, Santhosh Kumar Shanmugham, and Stephan Erb.
> 
> 
> Bugs: AURORA-1107
>     https://issues.apache.org/jira/browse/AURORA-1107
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> This allows users to specify volume mounts for tasks using the unified
> containerizer if the operator permits them. This is analogous to enabling docker
> parameters per task and using the `--volume` parameter.
> 
> This does not include the needed DSL changes or an e2e test which will be in a
> subsequent diff.
> 
> 
> Diffs
> -----
> 
>   api/src/main/thrift/org/apache/aurora/gen/api.thrift 0c74665f750571ccfdeda5a0b71cccbe169716a7

>   src/main/java/org/apache/aurora/scheduler/app/AppModule.java c6c2a6d4ec48681378210d1fcb7909fd088d2afb

>   src/main/java/org/apache/aurora/scheduler/base/TaskTestUtil.java 3bd22a0d47cf0793c749b008ccbdf58033c75a11

>   src/main/java/org/apache/aurora/scheduler/configuration/ConfigurationManager.java 701f79c900e1d5794741475333805d1c12af58dd

>   src/main/java/org/apache/aurora/scheduler/mesos/MesosTaskFactory.java 9038c36e4ae05ade273a1c218b7a7e8af218ae39

>   src/main/java/org/apache/aurora/scheduler/storage/db/DbStorage.java acb44984b4aa38bbe1cac8f805754d55dbe39f3a

>   src/main/java/org/apache/aurora/scheduler/storage/db/TaskConfigManager.java e137e57a0b111b9320e6902aa586d7f24e0d5c58

>   src/main/java/org/apache/aurora/scheduler/storage/db/TaskConfigMapper.java 151306a567448c45223e90c8a29ac6ecbc9f6653

>   src/main/java/org/apache/aurora/scheduler/storage/db/migration/V009_CreateContainerVolumesTable.java
PRE-CREATION 
>   src/main/java/org/apache/aurora/scheduler/storage/db/typehandlers/TypeHandlers.java
e30c387239e868daedf496dd886df4d8c00f1fc3 
>   src/main/java/org/apache/aurora/scheduler/storage/db/typehandlers/VolumeModeTypeHandler.java
PRE-CREATION 
>   src/main/java/org/apache/aurora/scheduler/storage/db/views/DbContainer.java 8d4d7eca3a17a3b96bbbdec9271503e7e71b3aff

>   src/main/resources/org/apache/aurora/scheduler/storage/db/TaskConfigMapper.xml 3fce25f5e6d180a20dc3ace83f666d47bf32c0c5

>   src/main/resources/org/apache/aurora/scheduler/storage/db/schema.sql e943c647af4d0bdb1c733813d019d91942fbbc95

>   src/test/java/org/apache/aurora/scheduler/configuration/ConfigurationManagerTest.java
db9f276c1ad26b33ff66f679787ee798bbb69c80 
>   src/test/java/org/apache/aurora/scheduler/mesos/MesosTaskFactoryImplTest.java bd052abd65109544be92d3e292f1c2b238c86135

>   src/test/java/org/apache/aurora/scheduler/storage/AbstractTaskStoreTest.java 3d07292c83c229cc85d3ad624a7a7a65ef0dcd51

>   src/test/java/org/apache/aurora/scheduler/thrift/Fixtures.java 95b371627d23f7ed3472561f17880bf3c4259b96

>   src/test/java/org/apache/aurora/scheduler/thrift/ThriftIT.java 3caad32a30a02a1dd33893266c0dc817633a0f37

> 
> Diff: https://reviews.apache.org/r/52588/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Zameer Manji
> 
>


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