See <https://builds.apache.org/job/AuroraBot/635/changes>

Changes:

[maxim] Adding PMD rule to check @Timed annotation placement.

[maxim] Adding JMH framework support for scheduler performance analysis.

[maxim] More logging in MaintenanceController.

[maxim] Improving logging experience in admin drain_hosts.

[maxim] Suppressing redundant client command error messaging.

------------------------------------------
[...truncated 2290 lines...]
config.status: creating test/nodejs/Makefile
config.status: creating test/php/Makefile
config.status: creating test/perl/Makefile
config.status: creating test/py/Makefile
config.status: creating test/py.twisted/Makefile
config.status: creating test/py.tornado/Makefile
config.status: creating test/rb/Makefile
config.status: creating tutorial/Makefile
config.status: creating tutorial/cpp/Makefile
config.status: creating tutorial/go/Makefile
config.status: creating tutorial/java/Makefile
config.status: creating tutorial/js/Makefile
config.status: creating tutorial/py/Makefile
config.status: creating tutorial/py.twisted/Makefile
config.status: creating tutorial/py.tornado/Makefile
config.status: creating tutorial/rb/Makefile
config.status: creating config.h
config.status: creating lib/cpp/src/thrift/config.h
config.status: executing depfiles commands
config.status: executing libtool commands

thrift 0.9.1

Building C++ Library ......... : no
Building C (GLib) Library .... : no
Building Java Library ........ : no
Building C# Library .......... : no
Building Python Library ...... : no
Building Ruby Library ........ : no
Building Haskell Library ..... : no
Building Perl Library ........ : no
Building PHP Library ......... : no
Building Erlang Library ...... : no
Building Go Library .......... : no
Building D Library ........... : no

If something is missing that you think should be present,
please skim the output of configure to find the missing
component.  Details are present in config.log.
make[1]: Entering directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1'>
make  all-recursive
make[2]: Entering directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1'>
Making all in compiler/cpp
make[3]: Entering directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1/compiler/cpp'>
make  all-am
make[4]: Entering directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1/compiler/cpp'>
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall 
-Wno-sign-compare -Wno-unused -g -O2 -c -o libparse_a-thrifty.o `test -f 
'thrifty.cc' || echo './'`thrifty.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall 
-Wno-sign-compare -Wno-unused -g -O2 -c -o libparse_a-thriftl.o `test -f 
'thriftl.cc' || echo './'`thriftl.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-main.o `test -f 'src/main.cc' || echo './'`src/main.cc
gcc -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src   -g -O2 -c 
-o thrift-md5.o `test -f 'src/md5.c' || echo './'`src/md5.c
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_generator.o `test -f 'src/generate/t_generator.cc' || echo 
'./'`src/generate/t_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-parse.o `test -f 'src/parse/parse.cc' || echo 
'./'`src/parse/parse.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_c_glib_generator.o `test -f 
'src/generate/t_c_glib_generator.cc' || echo 
'./'`src/generate/t_c_glib_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_cpp_generator.o `test -f 'src/generate/t_cpp_generator.cc' 
|| echo './'`src/generate/t_cpp_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_java_generator.o `test -f 'src/generate/t_java_generator.cc' 
|| echo './'`src/generate/t_java_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_as3_generator.o `test -f 'src/generate/t_as3_generator.cc' 
|| echo './'`src/generate/t_as3_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_csharp_generator.o `test -f 
'src/generate/t_csharp_generator.cc' || echo 
'./'`src/generate/t_csharp_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_py_generator.o `test -f 'src/generate/t_py_generator.cc' || 
echo './'`src/generate/t_py_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_rb_generator.o `test -f 'src/generate/t_rb_generator.cc' || 
echo './'`src/generate/t_rb_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_perl_generator.o `test -f 'src/generate/t_perl_generator.cc' 
|| echo './'`src/generate/t_perl_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_php_generator.o `test -f 'src/generate/t_php_generator.cc' 
|| echo './'`src/generate/t_php_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_erl_generator.o `test -f 'src/generate/t_erl_generator.cc' 
|| echo './'`src/generate/t_erl_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_cocoa_generator.o `test -f 
'src/generate/t_cocoa_generator.cc' || echo 
'./'`src/generate/t_cocoa_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_st_generator.o `test -f 'src/generate/t_st_generator.cc' || 
echo './'`src/generate/t_st_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_ocaml_generator.o `test -f 
'src/generate/t_ocaml_generator.cc' || echo 
'./'`src/generate/t_ocaml_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_hs_generator.o `test -f 'src/generate/t_hs_generator.cc' || 
echo './'`src/generate/t_hs_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_xsd_generator.o `test -f 'src/generate/t_xsd_generator.cc' 
|| echo './'`src/generate/t_xsd_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_html_generator.o `test -f 'src/generate/t_html_generator.cc' 
|| echo './'`src/generate/t_html_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_js_generator.o `test -f 'src/generate/t_js_generator.cc' || 
echo './'`src/generate/t_js_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_javame_generator.o `test -f 
'src/generate/t_javame_generator.cc' || echo 
'./'`src/generate/t_javame_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_delphi_generator.o `test -f 
'src/generate/t_delphi_generator.cc' || echo 
'./'`src/generate/t_delphi_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_go_generator.o `test -f 'src/generate/t_go_generator.cc' || 
echo './'`src/generate/t_go_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_gv_generator.o `test -f 'src/generate/t_gv_generator.cc' || 
echo './'`src/generate/t_gv_generator.cc
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src/thrift  -I./src  -Wall -g 
-O2 -c -o thrift-t_d_generator.o `test -f 'src/generate/t_d_generator.cc' || 
echo './'`src/generate/t_d_generator.cc
rm -f libparse.a
ar cru libparse.a libparse_a-thrifty.o libparse_a-thriftl.o 
ranlib libparse.a
/bin/bash ../../libtool --tag=CXX   --mode=link g++ -Wall -g -O2   -o thrift 
thrift-main.o thrift-md5.o thrift-t_generator.o thrift-parse.o 
thrift-t_c_glib_generator.o thrift-t_cpp_generator.o thrift-t_java_generator.o 
thrift-t_as3_generator.o thrift-t_csharp_generator.o thrift-t_py_generator.o 
thrift-t_rb_generator.o thrift-t_perl_generator.o thrift-t_php_generator.o 
thrift-t_erl_generator.o thrift-t_cocoa_generator.o thrift-t_st_generator.o 
thrift-t_ocaml_generator.o thrift-t_hs_generator.o thrift-t_xsd_generator.o 
thrift-t_html_generator.o thrift-t_js_generator.o thrift-t_javame_generator.o 
thrift-t_delphi_generator.o thrift-t_go_generator.o thrift-t_gv_generator.o 
thrift-t_d_generator.o -lfl libparse.a 
libtool: link: g++ -Wall -g -O2 -o thrift thrift-main.o thrift-md5.o 
thrift-t_generator.o thrift-parse.o thrift-t_c_glib_generator.o 
thrift-t_cpp_generator.o thrift-t_java_generator.o thrift-t_as3_generator.o 
thrift-t_csharp_generator.o thrift-t_py_generator.o thrift-t_rb_generator.o 
thrift-t_perl_generator.o thrift-t_php_generator.o thrift-t_erl_generator.o 
thrift-t_cocoa_generator.o thrift-t_st_generator.o thrift-t_ocaml_generator.o 
thrift-t_hs_generator.o thrift-t_xsd_generator.o thrift-t_html_generator.o 
thrift-t_js_generator.o thrift-t_javame_generator.o thrift-t_delphi_generator.o 
thrift-t_go_generator.o thrift-t_gv_generator.o thrift-t_d_generator.o  -lfl 
libparse.a
make[4]: Leaving directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1/compiler/cpp'>
make[3]: Leaving directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1/compiler/cpp'>
Making all in lib
make[3]: Entering directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1/lib'>
make[4]: Entering directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1/lib'>
make[4]: Nothing to be done for `all-am'.
make[4]: Leaving directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1/lib'>
make[3]: Leaving directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1/lib'>
Making all in tutorial
make[3]: Entering directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1/tutorial'>
make[4]: Entering directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1/tutorial'>
../compiler/cpp/thrift --gen html -r ../tutorial/tutorial.thrift
make[4]: Leaving directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1/tutorial'>
make[3]: Leaving directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1/tutorial'>
Making all in tutorial
make[3]: Entering directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1/tutorial'>
make[4]: Entering directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1/tutorial'>
../compiler/cpp/thrift --gen html -r ../tutorial/tutorial.thrift
make[4]: Leaving directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1/tutorial'>
make[3]: Leaving directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1/tutorial'>
make[3]: Entering directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1'>
make[3]: Leaving directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1'>
make[2]: Leaving directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1'>
make[1]: Leaving directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift/thrift-0.9.1'>
make: Leaving directory 
`<https://builds.apache.org/job/AuroraBot/ws/build-support/thrift'>
:api:classesThriftNote: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.

:api:checkPython
:api:generateThriftEntitiesJava
:api:classesThriftEntities
:api:compileJava UP-TO-DATE
:api:generateThriftResources
:api:processResources UP-TO-DATE
:api:classes
:api:jar
:compileJavaNote: Writing 
<https://builds.apache.org/job/AuroraBot/ws/dist/classes/main/com/twitter/common/args/apt/cmdline.arg.info.txt.2>

:processResources
:classes
:jar
:assemble
:compileJmhJava
Download 
https://repo1.maven.org/maven2/org/openjdk/jmh/jmh-core/1.1.1/jmh-core-1.1.1.pom
Download 
https://repo1.maven.org/maven2/org/openjdk/jmh/jmh-parent/1.1.1/jmh-parent-1.1.1.pom
Download 
https://repo1.maven.org/maven2/org/openjdk/jmh/jmh-generator-annprocess/1.1.1/jmh-generator-annprocess-1.1.1.pom
Download 
https://repo1.maven.org/maven2/net/sf/jopt-simple/jopt-simple/4.6/jopt-simple-4.6.pom
Download 
https://repo1.maven.org/maven2/org/openjdk/jmh/jmh-core/1.1.1/jmh-core-1.1.1.jar
Download 
https://repo1.maven.org/maven2/org/openjdk/jmh/jmh-generator-annprocess/1.1.1/jmh-generator-annprocess-1.1.1.jar
Download 
https://repo1.maven.org/maven2/net/sf/jopt-simple/jopt-simple/4.6/jopt-simple-4.6.jar
Download 
https://repo1.maven.org/maven2/org/hamcrest/hamcrest-core/1.2/hamcrest-core-1.2.jar
warning: Supported source version 'RELEASE_6' from annotation processor 
'org.openjdk.jmh.generators.BenchmarkProcessor' less than -source '1.7'
1 warning
:processJmhResources UP-TO-DATE
:jmhClasses
:checkstyleJmh[ant:checkstyle] 
<https://builds.apache.org/job/AuroraBot/ws/src/jmh/java/org/apache/aurora/benchmark/Offers.java>:29:
 Wrong order for 'com.twitter.common.quantity.Amount' import. Order should be: 
java, javax, scala, com, net, org.         Each group should be separated by a 
single blank line.
[ant:checkstyle] 
<https://builds.apache.org/job/AuroraBot/ws/src/jmh/java/org/apache/aurora/benchmark/Offers.java>:55:27:
 'cpu' hides a field.
[ant:checkstyle] 
<https://builds.apache.org/job/AuroraBot/ws/src/jmh/java/org/apache/aurora/benchmark/Offers.java>:60:39:
 'ram' hides a field.
[ant:checkstyle] 
<https://builds.apache.org/job/AuroraBot/ws/src/jmh/java/org/apache/aurora/benchmark/Offers.java>:65:40:
 'disk' hides a field.
[ant:checkstyle] 
<https://builds.apache.org/job/AuroraBot/ws/src/jmh/java/org/apache/aurora/benchmark/Offers.java>:70:26:
 'ports' hides a field.
[ant:checkstyle] 
<https://builds.apache.org/job/AuroraBot/ws/src/jmh/java/org/apache/aurora/benchmark/Offers.java>:75:48:
 'maintenanceMode' hides a field.
[ant:checkstyle] 
<https://builds.apache.org/job/AuroraBot/ws/src/jmh/java/org/apache/aurora/benchmark/SchedulingBenchmarks.java>:56:
 Wrong order for 'com.twitter.common.quantity.Amount' import. Order should be: 
java, javax, scala, com, net, org.         Each group should be separated by a 
single blank line.
[ant:checkstyle] 
<https://builds.apache.org/job/AuroraBot/ws/src/jmh/java/org/apache/aurora/benchmark/SchedulingBenchmarks.java>:66:17:
 'abstract' modifier out of order with the JLS suggestions.
[ant:checkstyle] 
<https://builds.apache.org/job/AuroraBot/ws/src/jmh/java/org/apache/aurora/benchmark/SchedulingBenchmarks.java>:67:13:
 Variable 'storage' must be private and have accessor methods.
[ant:checkstyle] 
<https://builds.apache.org/job/AuroraBot/ws/src/jmh/java/org/apache/aurora/benchmark/SchedulingBenchmarks.java>:68:19:
 Variable 'taskScheduler' must be private and have accessor methods.
[ant:checkstyle] 
<https://builds.apache.org/job/AuroraBot/ws/src/jmh/java/org/apache/aurora/benchmark/SchedulingBenchmarks.java>:69:16:
 Variable 'offerQueue' must be private and have accessor methods.
[ant:checkstyle] 
<https://builds.apache.org/job/AuroraBot/ws/src/jmh/java/org/apache/aurora/benchmark/SchedulingBenchmarks.java>:70:20:
 Variable 'task' must be private and have accessor methods.
[ant:checkstyle] 
<https://builds.apache.org/job/AuroraBot/ws/src/jmh/java/org/apache/aurora/benchmark/SchedulingBenchmarks.java>:71:20:
 Variable 'offers' must be private and have accessor methods.
[ant:checkstyle] 
<https://builds.apache.org/job/AuroraBot/ws/src/jmh/java/org/apache/aurora/benchmark/SchedulingBenchmarks.java>:138:27:
 'task' hides a field.
[ant:checkstyle] 
<https://builds.apache.org/job/AuroraBot/ws/src/jmh/java/org/apache/aurora/benchmark/Tasks.java>:34:
 Wrong order for 'com.twitter.common.quantity.Amount' import. Order should be: 
java, javax, scala, com, net, org.         Each group should be separated by a 
single blank line.
[ant:checkstyle] 
<https://builds.apache.org/job/AuroraBot/ws/src/jmh/java/org/apache/aurora/benchmark/Tasks.java>:56:36:
 'taskIdFormat' hides a field.
[ant:checkstyle] 
<https://builds.apache.org/job/AuroraBot/ws/src/jmh/java/org/apache/aurora/benchmark/Tasks.java>:61:27:
 'cpu' hides a field.
[ant:checkstyle] 
<https://builds.apache.org/job/AuroraBot/ws/src/jmh/java/org/apache/aurora/benchmark/Tasks.java>:66:39:
 'ram' hides a field.
[ant:checkstyle] 
<https://builds.apache.org/job/AuroraBot/ws/src/jmh/java/org/apache/aurora/benchmark/Tasks.java>:71:40:
 'disk' hides a field.
[ant:checkstyle] 
<https://builds.apache.org/job/AuroraBot/ws/src/jmh/java/org/apache/aurora/benchmark/Tasks.java>:76:46:
 'scheduleStatus' hides a field.
[ant:checkstyle] 
<https://builds.apache.org/job/AuroraBot/ws/src/jmh/java/org/apache/aurora/benchmark/Tasks.java>:109:
 File contains a sequence of empty lines.
 FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':checkstyleJmh'.
> Checkstyle rule violations were found. See the report at: 
> file://<https://builds.apache.org/job/AuroraBot/ws/dist/reports/checkstyle/jmh.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: 2 mins 6.995 secs
Request: https://reviews.apache.org/api/repositories/?name=aurora
Request: 
https://reviews.apache.org/api/review-requests/?status=pending&repository=133
Found 13 review requests to inspect
Inspecting review 28731: Implemented TaskScheduler benchmarks.
Request: https://reviews.apache.org/api/review-requests/28731/reviews/
Request: https://reviews.apache.org/api/review-requests/28731/diffs/
Request: https://reviews.apache.org/api/review-requests/28731/diffs/
Applying diff 52924
Request: https://reviews.apache.org/api/review-requests/28731/diffs/3/
Running build command.
Request: https://reviews.apache.org/api/review-requests/28731/diffs/3/files/
Replying to review 28731:
Master (07bc3ab) is green with this patch.
  ./build-support/jenkins/build.sh

However, it appears that it might lack test coverage.

I will refresh this build result if you post a review containing "@ReviewBot 
retry"
Request: https://reviews.apache.org/api/review-requests/28731/reviews/
{u'stat': u'ok', u'review': {u'body_top': u'Master (07bc3ab) is green with this 
patch.\n  ./build-support/jenkins/build.sh\n\nHowever, it appears that it might 
lack test coverage.\n\nI will refresh this build result if you post a review 
containing "@ReviewBot retry"', u'ship_it': False, u'links': {u'diff_comments': 
{u'href': 
u'https://reviews.apache.org/api/review-requests/28731/reviews/64792/diff-comments/',
 u'method': u'GET'}, u'file_attachment_comments': {u'href': 
u'https://reviews.apache.org/api/review-requests/28731/reviews/64792/file-attachment-comments/',
 u'method': u'GET'}, u'self': {u'href': 
u'https://reviews.apache.org/api/review-requests/28731/reviews/64792/', 
u'method': u'GET'}, u'update': {u'href': 
u'https://reviews.apache.org/api/review-requests/28731/reviews/64792/', 
u'method': u'PUT'}, u'screenshot_comments': {u'href': 
u'https://reviews.apache.org/api/review-requests/28731/reviews/64792/screenshot-comments/',
 u'method': u'GET'}, u'user': {u'href': 
u'https://reviews.apache.org/api/users/aurorabot/', u'method': u'GET', 
u'title': u'aurorabot'}, u'replies': {u'href': 
u'https://reviews.apache.org/api/review-requests/28731/reviews/64792/replies/', 
u'method': u'GET'}, u'delete': {u'href': 
u'https://reviews.apache.org/api/review-requests/28731/reviews/64792/', 
u'method': u'DELETE'}}, u'timestamp': u'2014-12-11T20:53:39Z', u'id': 64792, 
u'text_type': u'plain', u'body_bottom': u'', u'extra_data': {}, u'public': 
True}}
Inspecting review 28872: Improving quota check message in the client.
Request: https://reviews.apache.org/api/review-requests/28872/reviews/
Request: https://reviews.apache.org/api/review-requests/28872/diffs/
Inspecting review 28198: Add an example on how to build with Docker.
Request: https://reviews.apache.org/api/review-requests/28198/reviews/
Request: https://reviews.apache.org/api/review-requests/28198/diffs/
Inspecting review 28920: Add support for docker containers to aurora
Request: https://reviews.apache.org/api/review-requests/28920/reviews/
Request: https://reviews.apache.org/api/review-requests/28920/diffs/
Inspecting review 28913: Remove SANDBOX_DELETED state.
Request: https://reviews.apache.org/api/review-requests/28913/reviews/
Request: https://reviews.apache.org/api/review-requests/28913/diffs/
Inspecting review 28617: Implemented offer filtering for tasks with static 
vetoes.
Request: https://reviews.apache.org/api/review-requests/28617/reviews/
Request: https://reviews.apache.org/api/review-requests/28617/diffs/
Inspecting review 28879: Make abstract decorators effective in CommandHook class
Request: https://reviews.apache.org/api/review-requests/28879/reviews/
Request: https://reviews.apache.org/api/review-requests/28879/diffs/
Inspecting review 28486: Move sandbox initialization into try block.
Request: https://reviews.apache.org/api/review-requests/28486/reviews/
Request: https://reviews.apache.org/api/review-requests/28486/diffs/
Inspecting review 27492: AURORA-617: Switch pants 3rdparty to use 
python_requirements
Request: https://reviews.apache.org/api/review-requests/27492/reviews/
Request: https://reviews.apache.org/api/review-requests/27492/diffs/
Inspecting review 27182: Add a test for the thermos resource module
Request: https://reviews.apache.org/api/review-requests/27182/reviews/
Request: https://reviews.apache.org/api/review-requests/27182/diffs/
Inspecting review 27778: Fix false positive client test.
Request: https://reviews.apache.org/api/review-requests/27778/reviews/
Request: https://reviews.apache.org/api/review-requests/27778/diffs/
Inspecting review 26753: Start removing clientv1.
Request: https://reviews.apache.org/api/review-requests/26753/reviews/
Request: https://reviews.apache.org/api/review-requests/26753/diffs/
Inspecting review 27080: Improve status command output ordering.
Request: https://reviews.apache.org/api/review-requests/27080/reviews/
Traceback (most recent call last):
  File "./build-support/jenkins/review_feedback.py", line 202, in <module>
    main()
  File "./build-support/jenkins/review_feedback.py", line 158, in main
    if not _needs_reply(server, request):
  File "./build-support/jenkins/review_feedback.py", line 88, in _needs_reply
    reviews = 
server.get_resource(request['links']['reviews']['href'])['reviews']
  File "./build-support/jenkins/review_feedback.py", line 51, in get_resource
    return json.loads(self.get_resource_data(href, args=args, data=data))
  File "./build-support/jenkins/review_feedback.py", line 44, in 
get_resource_data
    result = urllib2.urlopen(request, data=data)
  File "/usr/lib/python2.7/urllib2.py", line 127, in urlopen
    return _opener.open(url, data, timeout)
  File "/usr/lib/python2.7/urllib2.py", line 404, in open
    response = self._open(req, data)
  File "/usr/lib/python2.7/urllib2.py", line 422, in _open
    '_open', req)
  File "/usr/lib/python2.7/urllib2.py", line 382, in _call_chain
    result = func(*args)
  File "/usr/lib/python2.7/urllib2.py", line 1222, in https_open
    return self.do_open(httplib.HTTPSConnection, req)
  File "/usr/lib/python2.7/urllib2.py", line 1184, in do_open
    raise URLError(err)
urllib2.URLError: <urlopen error [Errno -2] Name or service not known>
Build step 'Execute shell' marked build as failure

Reply via email to