Re: Review Request 49642: Updating release docs to remind about rc argument usage.

2016-07-06 Thread Stephan Erb

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


Ship it!




Ship It!

- Stephan Erb


On July 5, 2016, 7:09 p.m., Maxim Khutornenko wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49642/
> ---
> 
> (Updated July 5, 2016, 7:09 p.m.)
> 
> 
> Review request for Aurora and Jake Farrell.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Updating release docs to remind about rc argument usage.
> 
> 
> Diffs
> -
> 
>   docs/development/committers-guide.md 
> e76e487aaa290656b6076e7473895542b114f4c8 
> 
> Diff: https://reviews.apache.org/r/49642/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Maxim Khutornenko
> 
>



Re: Review Request 49642: Updating release docs to remind about rc argument usage.

2016-07-05 Thread Aurora ReviewBot

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



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

1015 tests completed, 31 failed, 2 skipped
:test FAILED
:jacocoTestReport
Coverage report generated: 
file:///home/jenkins/jenkins-slave/workspace/AuroraBot/dist/reports/jacoco/test/html/index.html
:analyzeReport
Instruction coverage is 0.8706879069872243, but must be greater than 0.89
Branch coverage is 0.7762777242044359, but must be greater than 0.835
:analyzeReport FAILED

FAILURE: Build completed with 2 failures.

1: Task 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/index.html

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug 
option to get more log output.
==

2: Task failed with an exception.
---
* What went wrong:
Execution failed for task ':analyzeReport'.
> Test coverage missing for 
> org/apache/aurora/scheduler/http/LeaderRedirectFilter
  Test coverage missing for org/apache/aurora/scheduler/http/HttpStatsFilter
  Test coverage missing for org/apache/aurora/scheduler/http/HttpStatsFilter$1
  Test coverage missing for 
org/apache/aurora/scheduler/http/HttpStatsFilter$ResponseWithStatus
  Test coverage missing for 
org/apache/aurora/scheduler/http/api/security/ShiroKerberosAuthenticationFilter
  Test coverage missing for 
org/apache/aurora/scheduler/http/api/security/ShiroKerberosPermissiveAuthenticationFilter
  Test coverage missing for org/apache/aurora/scheduler/http/api/ApiBeta

* 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: 5 mins 19.286 secs


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

- Aurora ReviewBot


On July 5, 2016, 5:09 p.m., Maxim Khutornenko wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49642/
> ---
> 
> (Updated July 5, 2016, 5:09 p.m.)
> 
> 
> Review request for Aurora and Jake Farrell.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Updating release docs to remind about rc argument usage.
> 
> 
> Diffs
> -
> 
>   docs/development/committers-guide.md 
> e76e487aaa290656b6076e7473895542b114f4c8 
> 
> Diff: https://reviews.apache.org/r/49642/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Maxim Khutornenko
> 
>