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