-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66537/#review202335
-----------------------------------------------------------
Master (a3d596e) is red with this patch.
./build-support/jenkins/build.sh
Pass 2: Analyzing classes (320 / 333) - 96% complete
Pass 2: Analyzing classes (321 / 333) - 96% complete
Pass 2: Analyzing classes (322 / 333) - 96% complete
Pass 2: Analyzing classes (323 / 333) - 96% complete
Pass 2: Analyzing classes (324 / 333) - 97% complete
Pass 2: Analyzing classes (325 / 333) - 97% complete
Pass 2: Analyzing classes (326 / 333) - 97% complete
Pass 2: Analyzing classes (327 / 333) - 98% complete
Pass 2: Analyzing classes (328 / 333) - 98% complete
Pass 2: Analyzing classes (329 / 333) - 98% complete
Pass 2: Analyzing classes (330 / 333) - 99% complete
Pass 2: Analyzing classes (331 / 333) - 99% complete
Pass 2: Analyzing classes (332 / 333) - 99% complete
Pass 2: Analyzing classes (333 / 333) - 100% complete
Done with analysis
:test
org.apache.aurora.scheduler.events.WebhookTest >
testTaskChangedWithOldStateError FAILED
java.lang.AssertionError at WebhookTest.java:251
I0503 01:42:21.319 [ShutdownHook, SchedulerMain] Stopping scheduler services.
1087 tests completed, 1 failed, 1 skipped
:test FAILED
:jacocoTestReport
Coverage report generated:
file:///home/jenkins/jenkins-slave/workspace/AuroraBot/dist/reports/jacoco/test/html/index.html
:jacocoTestCoverageVerification
FAILURE: Build failed with an exception.
* What went wrong:
Execution failed for task ':test'.
> There were failing tests. See the report at:
> file:///home/jenkins/jenkins-slave/workspace/AuroraBot/dist/reports/tests/test/index.html
* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug
option to get more log output.
* Get more help at https://help.gradle.org
BUILD FAILED in 8m 46s
45 actionable tasks: 36 executed, 9 up-to-date
I will refresh this build result if you post a review containing "@ReviewBot
retry"
- Aurora ReviewBot
On May 3, 2018, 1:06 a.m., Steve Salevan wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/66537/
> -----------------------------------------------------------
>
> (Updated May 3, 2018, 1:06 a.m.)
>
>
> Review request for Aurora, Renan DelValle and Stephan Erb.
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Hey there!
>
> Over here at Spine we've added a few enhancements to Aurora's Docker
> functionality to support several of our use cases. We'd like to submit these
> back up to the upstream to support the great work y'all are doing in this
> space.
>
> Here's what's included in this RB:
>
> * Support for the force_pull flag to help ensure container freshness
> * Overrides for a Docker --entrypoint which can be specified on the job
> * Support for alternative Docker networks (defaults to HOST)
> * Support for user Docker networks
>
> We currently use Aurora to schedule Docker containers without the use of
> Thermos, so we've added support for server-side templating of common Thermos
> variables into Docker executor's parameters for this purpose.
>
> This change modifies Aurora's api.thrift with several new optional fields,
> and all added code handles their absence gracefully, so no backfills have
> been added. We've threaded these schema changes through to the Python Aurora
> client alongside support for the Mesos Fetcher URIs already supported
> server-side.
>
> Let me know what you think and thanks!
>
>
> Diffs
> -----
>
> .gitignore 9ce74ebbbc57b77d912eaa573a8fb18ed4aa3c15
> api/src/main/thrift/org/apache/aurora/gen/api.thrift
> ef754e32172e7490a47a13e7b526f243ffa3efeb
>
> src/main/java/org/apache/aurora/scheduler/configuration/ConfigurationManager.java
> 4073229b74d0e0e7fd31552bd96894ceb8a0971a
> src/main/java/org/apache/aurora/scheduler/mesos/MesosTaskFactory.java
> bcb2bbf882f43d813dd26c746d806e78bae6bcf3
> src/main/python/apache/aurora/config/schema/base.py
> a629bcd1261e5959da0a8458a55545d4e2c2a7a5
> src/main/python/apache/aurora/config/thrift.py
> 6d2dde6e964daa68bf6f0e5bbbffecc5bd8c0431
> src/test/java/org/apache/aurora/scheduler/base/TaskTestUtil.java
> 778148a7c033cba9004954cabc33a2b1d003dccf
>
> src/test/java/org/apache/aurora/scheduler/configuration/ConfigurationManagerTest.java
> 749ffeac6cb851f32bba7606390203d7a046a0e6
>
> src/test/java/org/apache/aurora/scheduler/mesos/MesosTaskFactoryImplTest.java
> 686087ef858b8a5a8e956d82a7bd692f7be28b12
> src/test/java/org/apache/aurora/scheduler/thrift/ThriftIT.java
> 40851c419e4d62e6545959eebc0ce144fdecc697
> src/test/python/apache/aurora/client/cli/test_inspect.py
> e4f43d0573c7862adc9bc679f4cea40cc76eac38
> src/test/python/apache/aurora/config/test_thrift.py
> 8e1d0e177959af12b97bdd1cd47845b72bc12fe1
>
> src/test/resources/org/apache/aurora/scheduler/storage/durability/goldens/current/saveCronJob
> 88e1c36a1aa2d192b95963f7aa36e243a447e4af
>
> src/test/resources/org/apache/aurora/scheduler/storage/durability/goldens/current/saveJobUpdate
> 32fdcdacde58345cdd6c4b449b82c0c90c2b2aae
>
> src/test/resources/org/apache/aurora/scheduler/storage/durability/goldens/current/saveTasks
> 4323031ec6bd128576c2a43ebc11f04a9f046e2f
>
>
> Diff: https://reviews.apache.org/r/66537/diff/4/
>
>
> Testing
> -------
>
> Unit tests for all new functionality have been added and run cleanly
> alongside checkstyle/static analysis tooling.
>
> Operational/functional testing completed within Spine infrastructure.
>
>
> Thanks,
>
> Steve Salevan
>
>
