Build failed in Jenkins: Mesos-Reviewbot #23847

[Apache Jenkins Server <je...@builds.apache.org>]

See <https://builds.apache.org/job/Mesos-Reviewbot/23847/display/redirect?page=changes>

Changes:

[bbannier] Reformated verify-reviews.py with yapf.

[bbannier] Fixed some lint issues in verify-reviews.py.

[bbannier] Fixed verify-reviews.py to not abort review verification prematurely.

------------------------------------------
[...truncated 42.55 MB...]
rm -rf authentication/executor/.libs authentication/executor/_libs
rm -f slave/containerizer/mesos/isolators/network/*.o
rm -f slave/containerizer/mesos/provisioner/appc/.deps/.dirstamp
rm -rf authentication/http/.libs authentication/http/_libs
rm -f slave/containerizer/mesos/isolators/network/*.lo
rm -f slave/containerizer/mesos/provisioner/appc/.dirstamp
rm -rf authorizer/.libs authorizer/_libs
rm -f slave/containerizer/mesos/isolators/network/cni/*.o
rm -f slave/containerizer/mesos/provisioner/backends/.deps/.dirstamp
rm -rf authorizer/local/.libs authorizer/local/_libs
rm -f slave/containerizer/mesos/isolators/network/cni/*.lo
rm -f slave/containerizer/mesos/provisioner/backends/.dirstamp
rm -f slave/containerizer/mesos/provisioner/docker/.deps/.dirstamp
rm -f slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/*.o
rm -rf checks/.libs checks/_libs
rm -f slave/containerizer/mesos/provisioner/docker/.dirstamp
rm -f slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/*.lo
rm -f slave/qos_controllers/.deps/.dirstamp
rm -f slave/containerizer/mesos/isolators/posix/*.o
rm -f slave/qos_controllers/.dirstamp
rm -f slave/containerizer/mesos/isolators/posix/*.lo
rm -rf common/.libs common/_libs
rm -f slave/resource_estimators/.deps/.dirstamp
rm -f slave/containerizer/mesos/isolators/volume/*.o
rm -f slave/resource_estimators/.dirstamp
rm -f slave/containerizer/mesos/isolators/volume/*.lo
rm -f state/.deps/.dirstamp
rm -f slave/containerizer/mesos/isolators/xfs/*.o
rm -f state/.dirstamp
rm -rf csi/.libs csi/_libs
rm -f slave/containerizer/mesos/isolators/xfs/*.lo
rm -f status_update_manager/.deps/.dirstamp
rm -f slave/containerizer/mesos/provisioner/*.o
rm -f status_update_manager/.dirstamp
rm -rf docker/.libs docker/_libs
rm -f slave/containerizer/mesos/provisioner/*.lo
rm -f tests/.deps/.dirstamp
rm -f slave/containerizer/mesos/provisioner/appc/*.o
rm -f tests/.dirstamp
rm -f slave/containerizer/mesos/provisioner/appc/*.lo
rm -f tests/common/.deps/.dirstamp
rm -rf examples/.libs examples/_libs
rm -f tests/common/.dirstamp
rm -f slave/containerizer/mesos/provisioner/backends/*.o
rm -f tests/containerizer/.deps/.dirstamp
rm -f slave/containerizer/mesos/provisioner/backends/*.lo
rm -rf exec/.libs exec/_libs
rm -f tests/containerizer/.dirstamp
rm -f slave/containerizer/mesos/provisioner/docker/*.o
rm -rf executor/.libs executor/_libs
rm -f uri/.deps/.dirstamp
rm -f slave/containerizer/mesos/provisioner/docker/*.lo
rm -f uri/.dirstamp
rm -f slave/qos_controllers/*.o
rm -rf files/.libs files/_libs
rm -f uri/fetchers/.deps/.dirstamp
rm -f slave/qos_controllers/*.lo
rm -f uri/fetchers/.dirstamp
rm -rf hdfs/.libs hdfs/_libs
rm -f usage/.deps/.dirstamp
rm -f slave/resource_estimators/*.o
rm -f usage/.dirstamp
rm -f slave/resource_estimators/*.lo
rm -rf hook/.libs hook/_libs
rm -f v1/.deps/.dirstamp
rm -f state/*.o
rm -f v1/.dirstamp
rm -rf internal/.libs internal/_libs
rm -f state/*.lo
rm -f version/.deps/.dirstamp
rm -f status_update_manager/*.o
rm -rf java/jni/.libs java/jni/_libs
rm -f version/.dirstamp
rm -f status_update_manager/*.lo
rm -f watcher/.deps/.dirstamp
rm -f tests/*.o
rm -rf jvm/.libs jvm/_libs
rm -f watcher/.dirstamp
rm -f zookeeper/.deps/.dirstamp
rm -rf jvm/org/apache/.libs jvm/org/apache/_libs
rm -f zookeeper/.dirstamp
rm -rf linux/.libs linux/_libs
rm -rf linux/routing/.libs linux/routing/_libs
rm -rf linux/routing/diagnosis/.libs linux/routing/diagnosis/_libs
rm -rf linux/routing/filter/.libs linux/routing/filter/_libs
rm -rf linux/routing/link/.libs linux/routing/link/_libs
rm -rf linux/routing/queueing/.libs linux/routing/queueing/_libs
rm -rf linux/seccomp/.libs linux/seccomp/_libs
rm -rf local/.libs local/_libs
rm -rf log/.libs log/_libs
rm -rf log/tool/.libs log/tool/_libs
rm -rf logging/.libs logging/_libs
rm -rf master/.libs master/_libs
rm -rf master/allocator/.libs master/allocator/_libs
rm -rf master/allocator/mesos/.libs master/allocator/mesos/_libs
rm -rf master/allocator/sorter/drf/.libs master/allocator/sorter/drf/_libs
rm -rf master/allocator/sorter/random/.libs master/allocator/sorter/random/_libs
rm -rf master/contender/.libs master/contender/_libs
rm -rf master/detector/.libs master/detector/_libs
rm -rf messages/.libs messages/_libs
rm -rf module/.libs module/_libs
rm -rf oci/.libs oci/_libs
rm -rf posix/.libs posix/_libs
rm -rf resource_provider/.libs resource_provider/_libs
rm -rf resource_provider/storage/.libs resource_provider/storage/_libs
rm -rf sched/.libs sched/_libs
rm -rf scheduler/.libs scheduler/_libs
rm -rf secret/.libs secret/_libs
rm -rf slave/.libs slave/_libs
rm -rf slave/container_loggers/.libs slave/container_loggers/_libs
rm -rf slave/containerizer/.libs slave/containerizer/_libs
rm -rf slave/containerizer/mesos/.libs slave/containerizer/mesos/_libs
rm -rf slave/containerizer/mesos/io/.libs slave/containerizer/mesos/io/_libs
rm -f tests/common/*.o
rm -rf slave/containerizer/mesos/isolators/.libs slave/containerizer/mesos/isolators/_libs
rm -f tests/containerizer/*.o
rm -rf slave/containerizer/mesos/isolators/appc/.libs slave/containerizer/mesos/isolators/appc/_libs
rm -rf slave/containerizer/mesos/isolators/cgroups/.libs slave/containerizer/mesos/isolators/cgroups/_libs
rm -rf slave/containerizer/mesos/isolators/cgroups/subsystems/.libs slave/containerizer/mesos/isolators/cgroups/subsystems/_libs
rm -rf slave/containerizer/mesos/isolators/docker/.libs slave/containerizer/mesos/isolators/docker/_libs
rm -rf slave/containerizer/mesos/isolators/docker/volume/.libs slave/containerizer/mesos/isolators/docker/volume/_libs
rm -rf slave/containerizer/mesos/isolators/filesystem/.libs slave/containerizer/mesos/isolators/filesystem/_libs
rm -rf slave/containerizer/mesos/isolators/gpu/.libs slave/containerizer/mesos/isolators/gpu/_libs
rm -rf slave/containerizer/mesos/isolators/linux/.libs slave/containerizer/mesos/isolators/linux/_libs
rm -rf slave/containerizer/mesos/isolators/namespaces/.libs slave/containerizer/mesos/isolators/namespaces/_libs
rm -rf slave/containerizer/mesos/isolators/network/.libs slave/containerizer/mesos/isolators/network/_libs
rm -rf slave/containerizer/mesos/isolators/network/cni/.libs slave/containerizer/mesos/isolators/network/cni/_libs
rm -rf slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/.libs slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/_libs
rm -f uri/*.o
rm -rf slave/containerizer/mesos/isolators/posix/.libs slave/containerizer/mesos/isolators/posix/_libs
rm -f uri/*.lo
rm -f uri/fetchers/*.o
rm -f uri/fetchers/*.lo
rm -rf slave/containerizer/mesos/isolators/volume/.libs slave/containerizer/mesos/isolators/volume/_libs
rm -f usage/*.o
rm -f usage/*.lo
rm -rf slave/containerizer/mesos/isolators/xfs/.libs slave/containerizer/mesos/isolators/xfs/_libs
rm -rf slave/containerizer/mesos/provisioner/.libs slave/containerizer/mesos/provisioner/_libs
rm -f v1/*.o
rm -f v1/*.lo
rm -f version/*.o
rm -rf slave/containerizer/mesos/provisioner/appc/.libs slave/containerizer/mesos/provisioner/appc/_libs
rm -f version/*.lo
rm -f watcher/*.o
rm -f watcher/*.lo
rm -rf slave/containerizer/mesos/provisioner/backends/.libs slave/containerizer/mesos/provisioner/backends/_libs
rm -f zookeeper/*.o
rm -f zookeeper/*.lo
rm -rf slave/containerizer/mesos/provisioner/docker/.libs slave/containerizer/mesos/provisioner/docker/_libs
rm -rf slave/qos_controllers/.libs slave/qos_controllers/_libs
rm -rf slave/resource_estimators/.libs slave/resource_estimators/_libs
rm -rf state/.libs state/_libs
rm -rf status_update_manager/.libs status_update_manager/_libs
rm -rf uri/.libs uri/_libs
rm -rf uri/fetchers/.libs uri/fetchers/_libs
rm -rf usage/.libs usage/_libs
rm -rf v1/.libs v1/_libs
rm -rf version/.libs version/_libs
rm -rf watcher/.libs watcher/_libs
rm -rf zookeeper/.libs zookeeper/_libs
rm -rf ../include/csi/.deps ../include/mesos/.deps ../include/mesos/agent/.deps ../include/mesos/allocator/.deps ../include/mesos/appc/.deps ../include/mesos/authentication/.deps ../include/mesos/authorizer/.deps ../include/mesos/docker/.deps ../include/mesos/executor/.deps ../include/mesos/fetcher/.deps ../include/mesos/maintenance/.deps ../include/mesos/master/.deps ../include/mesos/module/.deps ../include/mesos/oci/.deps ../include/mesos/quota/.deps ../include/mesos/resource_provider/.deps ../include/mesos/scheduler/.deps ../include/mesos/seccomp/.deps ../include/mesos/slave/.deps ../include/mesos/state/.deps ../include/mesos/uri/.deps ../include/mesos/v1/.deps ../include/mesos/v1/agent/.deps ../include/mesos/v1/allocator/.deps ../include/mesos/v1/executor/.deps ../include/mesos/v1/maintenance/.deps ../include/mesos/v1/master/.deps ../include/mesos/v1/quota/.deps ../include/mesos/v1/resource_provider/.deps ../include/mesos/v1/scheduler/.deps appc/.deps authentication/cram_md5/.deps authentication/executor/.deps authentication/http/.deps authorizer/.deps authorizer/local/.deps checks/.deps cli/.deps common/.deps csi/.deps docker/.deps examples/.deps exec/.deps executor/.deps files/.deps hdfs/.deps hook/.deps internal/.deps java/jni/.deps jvm/.deps jvm/org/apache/.deps launcher/.deps linux/.deps linux/routing/.deps linux/routing/diagnosis/.deps linux/routing/filter/.deps linux/routing/link/.deps linux/routing/queueing/.deps linux/seccomp/.deps local/.deps log/.deps log/tool/.deps logging/.deps master/.deps master/allocator/.deps master/allocator/mesos/.deps master/allocator/sorter/drf/.deps master/allocator/sorter/random/.deps master/contender/.deps master/detector/.deps messages/.deps module/.deps oci/.deps posix/.deps resource_provider/.deps resource_provider/storage/.deps sched/.deps scheduler/.deps secret/.deps slave/.deps slave/container_loggers/.deps slave/containerizer/.deps slave/containerizer/mesos/.deps slave/containerizer/mesos/io/.deps slave/containerizer/mesos/isolators/.deps slave/containerizer/mesos/isolators/appc/.deps slave/containerizer/mesos/isolators/cgroups/.deps slave/containerizer/mesos/isolators/cgroups/subsystems/.deps slave/containerizer/mesos/isolators/docker/.deps slave/containerizer/mesos/isolators/docker/volume/.deps slave/containerizer/mesos/isolators/filesystem/.deps slave/containerizer/mesos/isolators/gpu/.deps slave/containerizer/mesos/isolators/linux/.deps slave/containerizer/mesos/isolators/namespaces/.deps slave/containerizer/mesos/isolators/network/.deps slave/containerizer/mesos/isolators/network/cni/.deps slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/.deps slave/containerizer/mesos/isolators/posix/.deps slave/containerizer/mesos/isolators/volume/.deps slave/containerizer/mesos/isolators/xfs/.deps slave/containerizer/mesos/provisioner/.deps slave/containerizer/mesos/provisioner/appc/.deps slave/containerizer/mesos/provisioner/backends/.deps slave/containerizer/mesos/provisioner/docker/.deps slave/qos_controllers/.deps slave/resource_estimators/.deps state/.deps status_update_manager/.deps tests/.deps tests/common/.deps tests/containerizer/.deps uri/.deps uri/fetchers/.deps usage/.deps v1/.deps version/.deps watcher/.deps zookeeper/.deps
rm -f Makefile
make[2]: Leaving directory '/tmp/SRC/build/mesos-1.8.0/_build/sub/src'
rm -f config.status config.cache config.log configure.lineno config.status.lineno
rm -f Makefile
make[1]: Leaving directory '/tmp/SRC/build/mesos-1.8.0/_build/sub'
if test -d "mesos-1.8.0"; then find "mesos-1.8.0" -type d ! -perm -200 -exec chmod u+w {} ';' && rm -rf "mesos-1.8.0" || { sleep 5 && rm -rf "mesos-1.8.0"; }; else :; fi
=============================================
mesos-1.8.0 archives ready for distribution: 
mesos-1.8.0.tar.gz
=============================================
Untagged: mesos/mesos-build:ubuntu-16.04
Untagged: mesos/mesos-build@sha256:fa967cbcfb44f55708a3cbc87f245c6d29dd891464db558af56a03ee321526bb
Deleted: sha256:e895c0531b9a9a288fabe479a49f7059aed83645351ac99ec2ea2616822c9f97
Deleted: sha256:09119b0b314a69ba6ec6251f2e89f4199fe1e874c84f9abf07dcbd23dbc3f1c1
Deleted: sha256:562fe6af5a3883058be9c784a839392215ed4185a21d21f1d99df0d17f3ae6e7
Deleted: sha256:0dcdfbe322a4f332f73ac70905d557300ec6dae3bd19586758772c750c7b4a19
Deleted: sha256:c66652d605f01094e2db53d62505dbd524e076d5aa69b89d5e620003803eb149
Deleted: sha256:a2cf79bfb9593c05fd7142ddb49afef77ea4ad5d2464e841f09fe62ffee396e0
Deleted: sha256:f7a904214b390f39d98573882f631dc908df8b2b540cf04e1062f8182c1efffd
Deleted: sha256:ce1f6fcaa83dfce189d76e08f184085732eab4eeb2562d2399953958405c5bec
Deleted: sha256:cce92fda689ab9033f0b8db214bc63edd1ae3e05831a0f3a9418976d7dc7ccdd
Deleted: sha256:d22094bbd65447c59a42c580eaa3a44cee9cd855f00905f59409be21bcefc745
Deleted: sha256:b8976847450013f3eb5e9a81a5778f73ed7bef67e6393049712ef17102b4b7b7
Deleted: sha256:b8c891f0ffec910a12757d733b178e3f62d81dbbde2b31d3b754071c416108ed
Checking if review: 69955 needs verification
Latest diff timestamp: 2019-02-12 05:27:22
Latest review timestamp: 2019-02-13 07:22:36
Checking if review: 69971 needs verification
Latest diff timestamp: 2019-02-13 05:53:54
Latest review timestamp: 2019-02-13 16:02:59
Checking if review: 69342 needs verification
Skipping blocking review 69342
Checking if review: 69895 needs verification
Skipping blocking review 69895
Checking if review: 69896 needs verification
Skipping blocking review 69896
Checking if review: 69897 needs verification
Skipping blocking review 69897
Checking if review: 69904 needs verification
Skipping blocking review 69904
Checking if review: 69960 needs verification
Skipping blocking review 69960
Checking if review: 69962 needs verification
Skipping blocking review 69962
Checking if review: 69963 needs verification
Skipping blocking review 69963
Checking if review: 69967 needs verification
Skipping blocking review 69967
Checking if review: 69961 needs verification
Skipping blocking review 69961
Checking if review: 69980 needs verification
Latest diff timestamp: 2019-02-13 23:19:01
Latest review timestamp: 2019-02-14 07:03:09
Checking if review: 69977 needs verification
Skipping blocking review 69977
Checking if review: 69615 needs verification
Latest diff timestamp: 2019-02-08 06:44:52
Latest review timestamp: 2019-01-23 02:30:12
Verifying review 69615
Applying review 69615
ccd1970d1be47306d15cffc68edadfe3926d265a

Traceback (most recent call last):
  File "<https://builds.apache.org/job/Mesos-Reviewbot/ws/support/verify-reviews.py",> line 185, in apply_reviews
    apply_review(review_request["id"])
  File "<https://builds.apache.org/job/Mesos-Reviewbot/ws/support/verify-reviews.py",> line 155, in apply_review
    (sys.executable, review_id), SCRIPT_PATH)
  File "<https://builds.apache.org/job/Mesos-Reviewbot/ws/support/verify-reviews.py",> line 117, in shell
    command, stderr=subprocess.STDOUT, cwd=working_dir, shell=True)
  File "/usr/lib/python3.5/subprocess.py", line 626, in check_output
    **kwargs).stdout
  File "/usr/lib/python3.5/subprocess.py", line 708, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command 'cd .. && /usr/bin/python3 support/apply-reviews.py -n -r 69615' returned non-zero exit status 1

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<https://builds.apache.org/job/Mesos-Reviewbot/ws/support/verify-reviews.py",> line 378, in <module>
    main()
  File "<https://builds.apache.org/job/Mesos-Reviewbot/ws/support/verify-reviews.py",> line 371, in main
    verify_review(review_request)
  File "<https://builds.apache.org/job/Mesos-Reviewbot/ws/support/verify-reviews.py",> line 222, in verify_review
    apply_reviews(review_request, reviews)
  File "<https://builds.apache.org/job/Mesos-Reviewbot/ws/support/verify-reviews.py",> line 187, in apply_reviews
    if "patch does not apply" in err.output:
TypeError: a bytes-like object is required, not 'str'
Build step 'Execute shell' marked build as failure