-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67657/#review205126
-----------------------------------------------------------
Master (3140448) is red with this patch.
./build-support/jenkins/build.sh
[189] ./src/main/resources/source-sans-pro.css 1.05 kB {0} [built]
[221] ./src/main/js/index.js 3.43 kB {0} [built]
+ 264 hidden modules
:processResources
:classes
:jar
:moreStartScripts
:startScripts
:distTar
:distZip
:assemble
:compileTestJavaNote: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
:processTestResources
:testClasses
: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 NO-SOURCE
:jmhClasses
:checkstyleJmh
:checkstyleMain[ant:checkstyle] [ERROR]
/home/jenkins/jenkins-slave/workspace/AuroraBot/src/main/java/org/apache/aurora/scheduler/sla/SlaManager.java:36:8:
Unused import - com.google.gson.GsonBuilder. [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.
* Get more help at https://help.gradle.org
BUILD FAILED in 5m 47s
32 actionable tasks: 26 executed, 6 up-to-date
I will refresh this build result if you post a review containing "@ReviewBot
retry"
- Aurora ReviewBot
On June 20, 2018, 10:14 p.m., Santhosh Kumar Shanmugham wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/67657/
> -----------------------------------------------------------
>
> (Updated June 20, 2018, 10:14 p.m.)
>
>
> Review request for Aurora, David McLaughlin, Franck Cuny, and Jordan Ly.
>
>
> Repository: aurora
>
>
> Description
> -------
>
> With the introduction of `timeoutSecs` for HostMaintenanceRequest
> and the `CoordinatorSlaPolicy`, it will be beneficial to expose the
> time remaining until forced maintenance to the Coordinator. Send
> the time remaining until force task maintenance as an extra query
> param to the Coordinator.
>
>
> Diffs
> -----
>
> docs/features/sla-requirements.md 555b174d2324b0b1b596a3da72b0a5a67fcca153
>
> src/main/java/org/apache/aurora/scheduler/maintenance/MaintenanceController.java
> 626a68263d6118f138cd6012fd49e033b09b75f0
> src/main/java/org/apache/aurora/scheduler/sla/SlaManager.java
> 9c5caf4af8e8c8bad408100af2bc4fe045603340
>
> src/test/java/org/apache/aurora/scheduler/maintenance/MaintenanceControllerImplTest.java
> c9390df25f7eacbab14a508b1926a05aac8112d6
> src/test/java/org/apache/aurora/scheduler/sla/SlaManagerTest.java
> 759a1bca4814b2cf70a20eac26aaabfcef682332
>
>
> Diff: https://reviews.apache.org/r/67657/diff/2/
>
>
> Testing
> -------
>
> ./gradlew test
>
> **Tested on Vagrant**
>
> ***Logs from Coordinator***
> Request received for {'task': ['devcluster/vagrant/test/coordinator/0']}
> {
> "forceMaintenanceCountdownMs": "604755646",
> "task": "devcluster/vagrant/test/coordinator/0",
> "taskConfig": {
> "assignedTask": {
> "assignedPorts": {},
> "instanceId": 0,
> "slaveHost": "192.168.33.7",
> "slaveId": "f0336813-864b-4c8f-914c-80f8cef3b61d-S0",
> "task": {
> ...<SNIPPED>
> }
> Responded: True
>
>
> Thanks,
>
> Santhosh Kumar Shanmugham
>
>
