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

Changes:

[chhsiao] Removed unnecessary failure handling in agent HTTP API handlers.

[chhsiao] Performed RP-specific validations when adding/updating RP configs.

[chhsiao] Removed `ROOT` requirements for `AgentResourceProviderConfigApiTest`.

[chhsiao] Set master/agent flags in `AgentResourceProviderConfigApiTest` 
fixture.

[chhsiao] Set master/agent flags in `StorageLocalResourceProviderTest` fixture.

[chhsiao] Added unit tests for adding/updating invalid resource provider 
configs.

[chhsiao] Moved the container ID prefix generation to `LocalResourceProvider`.

[chhsiao] Cleaned up residual containers when removing resource provider 
configs.

[chhsiao] Tested container cleanup in 
`AgentResourceProviderConfigApiTest.Remove`.

[chhsiao] Added MESOS-9228 to the 1.7.1 CHANGELOG.

------------------------------------------
[...truncated 39.83 MB...]
rm -f slave/containerizer/mesos/provisioner/backends/.deps/.dirstamp
rm -f slave/containerizer/mesos/isolators/network/*.o
rm -f slave/containerizer/mesos/provisioner/backends/.dirstamp
rm -f slave/containerizer/mesos/isolators/network/*.lo
rm -f slave/containerizer/mesos/provisioner/docker/.deps/.dirstamp
rm -f slave/containerizer/mesos/isolators/network/cni/*.o
rm -f slave/containerizer/mesos/provisioner/docker/.dirstamp
rm -f slave/containerizer/mesos/isolators/network/cni/*.lo
rm -f slave/qos_controllers/.deps/.dirstamp
rm -f slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/*.o
rm -f slave/qos_controllers/.dirstamp
rm -f slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/*.lo
rm -f slave/resource_estimators/.deps/.dirstamp
rm -f slave/containerizer/mesos/isolators/posix/*.o
rm -f slave/resource_estimators/.dirstamp
rm -f slave/containerizer/mesos/isolators/posix/*.lo
rm -f state/.deps/.dirstamp
rm -f slave/containerizer/mesos/isolators/volume/*.o
rm -f state/.dirstamp
rm -f slave/containerizer/mesos/isolators/volume/*.lo
rm -f status_update_manager/.deps/.dirstamp
rm -f status_update_manager/.dirstamp
rm -f slave/containerizer/mesos/isolators/xfs/*.o
rm -f tests/.deps/.dirstamp
rm -f slave/containerizer/mesos/isolators/xfs/*.lo
rm -f tests/.dirstamp
rm -f slave/containerizer/mesos/provisioner/*.o
rm -f tests/common/.deps/.dirstamp
rm -f slave/containerizer/mesos/provisioner/*.lo
rm -f tests/common/.dirstamp
rm -f slave/containerizer/mesos/provisioner/appc/*.o
rm -f tests/containerizer/.deps/.dirstamp
rm -f slave/containerizer/mesos/provisioner/appc/*.lo
rm -f tests/containerizer/.dirstamp
rm -f slave/containerizer/mesos/provisioner/backends/*.o
rm -f uri/.deps/.dirstamp
rm -f slave/containerizer/mesos/provisioner/backends/*.lo
rm -f uri/.dirstamp
rm -f slave/containerizer/mesos/provisioner/docker/*.o
rm -f uri/fetchers/.deps/.dirstamp
rm -f slave/containerizer/mesos/provisioner/docker/*.lo
rm -rf ../include/csi/.libs ../include/csi/_libs
rm -f uri/fetchers/.dirstamp
rm -f slave/qos_controllers/*.o
rm -f usage/.deps/.dirstamp
rm -rf ../include/mesos/.libs ../include/mesos/_libs
rm -f slave/qos_controllers/*.lo
rm -f usage/.dirstamp
rm -f slave/resource_estimators/*.o
rm -rf ../include/mesos/agent/.libs ../include/mesos/agent/_libs
rm -f v1/.deps/.dirstamp
rm -f slave/resource_estimators/*.lo
rm -rf ../include/mesos/allocator/.libs ../include/mesos/allocator/_libs
rm -f v1/.dirstamp
rm -f state/*.o
rm -rf ../include/mesos/appc/.libs ../include/mesos/appc/_libs
rm -f version/.deps/.dirstamp
rm -f state/*.lo
rm -rf ../include/mesos/authentication/.libs 
../include/mesos/authentication/_libs
rm -f version/.dirstamp
rm -f status_update_manager/*.o
rm -rf ../include/mesos/authorizer/.libs ../include/mesos/authorizer/_libs
rm -f watcher/.deps/.dirstamp
rm -f status_update_manager/*.lo
rm -f watcher/.dirstamp
rm -rf ../include/mesos/docker/.libs ../include/mesos/docker/_libs
rm -f tests/*.o
rm -f zookeeper/.deps/.dirstamp
rm -rf ../include/mesos/executor/.libs ../include/mesos/executor/_libs
rm -f zookeeper/.dirstamp
rm -rf ../include/mesos/fetcher/.libs ../include/mesos/fetcher/_libs
rm -rf ../include/mesos/maintenance/.libs ../include/mesos/maintenance/_libs
rm -rf ../include/mesos/master/.libs ../include/mesos/master/_libs
rm -rf ../include/mesos/module/.libs ../include/mesos/module/_libs
rm -rf ../include/mesos/oci/.libs ../include/mesos/oci/_libs
rm -rf ../include/mesos/quota/.libs ../include/mesos/quota/_libs
rm -rf ../include/mesos/resource_provider/.libs 
../include/mesos/resource_provider/_libs
rm -rf ../include/mesos/scheduler/.libs ../include/mesos/scheduler/_libs
rm -rf ../include/mesos/slave/.libs ../include/mesos/slave/_libs
rm -rf ../include/mesos/state/.libs ../include/mesos/state/_libs
rm -rf ../include/mesos/uri/.libs ../include/mesos/uri/_libs
rm -rf ../include/mesos/v1/.libs ../include/mesos/v1/_libs
rm -rf ../include/mesos/v1/agent/.libs ../include/mesos/v1/agent/_libs
rm -rf ../include/mesos/v1/allocator/.libs ../include/mesos/v1/allocator/_libs
rm -rf ../include/mesos/v1/executor/.libs ../include/mesos/v1/executor/_libs
rm -rf ../include/mesos/v1/maintenance/.libs 
../include/mesos/v1/maintenance/_libs
rm -rf ../include/mesos/v1/master/.libs ../include/mesos/v1/master/_libs
rm -rf ../include/mesos/v1/quota/.libs ../include/mesos/v1/quota/_libs
rm -rf ../include/mesos/v1/resource_provider/.libs 
../include/mesos/v1/resource_provider/_libs
rm -rf ../include/mesos/v1/scheduler/.libs ../include/mesos/v1/scheduler/_libs
rm -rf appc/.libs appc/_libs
rm -rf authentication/cram_md5/.libs authentication/cram_md5/_libs
rm -rf authentication/executor/.libs authentication/executor/_libs
rm -rf authentication/http/.libs authentication/http/_libs
rm -rf authorizer/.libs authorizer/_libs
rm -rf authorizer/local/.libs authorizer/local/_libs
rm -rf checks/.libs checks/_libs
rm -rf common/.libs common/_libs
rm -rf csi/.libs csi/_libs
rm -rf docker/.libs docker/_libs
rm -rf examples/.libs examples/_libs
rm -rf exec/.libs exec/_libs
rm -rf executor/.libs executor/_libs
rm -rf files/.libs files/_libs
rm -rf hdfs/.libs hdfs/_libs
rm -rf hook/.libs hook/_libs
rm -rf internal/.libs internal/_libs
rm -rf java/jni/.libs java/jni/_libs
rm -rf jvm/.libs jvm/_libs
rm -rf jvm/org/apache/.libs jvm/org/apache/_libs
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 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 -f tests/common/*.o
rm -rf secret/.libs secret/_libs
rm -rf slave/.libs slave/_libs
rm -f tests/containerizer/*.o
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 uri/*.o
rm -rf slave/containerizer/mesos/isolators/.libs 
slave/containerizer/mesos/isolators/_libs
rm -f uri/*.lo
rm -f uri/fetchers/*.o
rm -rf slave/containerizer/mesos/isolators/appc/.libs 
slave/containerizer/mesos/isolators/appc/_libs
rm -f uri/fetchers/*.lo
rm -rf slave/containerizer/mesos/isolators/cgroups/.libs 
slave/containerizer/mesos/isolators/cgroups/_libs
rm -f usage/*.o
rm -f usage/*.lo
rm -f v1/*.o
rm -rf slave/containerizer/mesos/isolators/cgroups/subsystems/.libs 
slave/containerizer/mesos/isolators/cgroups/subsystems/_libs
rm -f v1/*.lo
rm -f version/*.o
rm -f version/*.lo
rm -f watcher/*.o
rm -rf slave/containerizer/mesos/isolators/docker/.libs 
slave/containerizer/mesos/isolators/docker/_libs
rm -f watcher/*.lo
rm -rf slave/containerizer/mesos/isolators/docker/volume/.libs 
slave/containerizer/mesos/isolators/docker/volume/_libs
rm -f zookeeper/*.o
rm -f zookeeper/*.lo
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 -rf slave/containerizer/mesos/isolators/posix/.libs 
slave/containerizer/mesos/isolators/posix/_libs
rm -rf slave/containerizer/mesos/isolators/volume/.libs 
slave/containerizer/mesos/isolators/volume/_libs
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 -rf slave/containerizer/mesos/provisioner/appc/.libs 
slave/containerizer/mesos/provisioner/appc/_libs
rm -rf slave/containerizer/mesos/provisioner/backends/.libs 
slave/containerizer/mesos/provisioner/backends/_libs
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/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 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:4c182f919163e3131eff1144df46d5aa4ce69810c17ea68b62d58f2f32813308
Deleted: sha256:03de0cadaa0ab780958449ad6e0336f9a35490801f73ee874a3dde89cfca7292
Deleted: sha256:d5b4fcc7a86dfa26a914be57012f83ee207d8a17694876309128924237215d90
Deleted: sha256:555296da9bbb164d853789b913bbc7506cb330397fdd0f87adda431d7f40bddb
Deleted: sha256:869ed3753aa81aadb245ecafca08cdd6580241e537f7b1e20e89701d21333ded
Deleted: sha256:eb101916dbdb53eedc8ef17346d59644d06348fdfb8c82538566b6b846731203
Deleted: sha256:6508e1a91fee84110e915fe36271cf7e458608bf46c1965495bde852bde41a42
Deleted: sha256:65326b3892a34181bde892271355ea2c717e6a2d297fc41150e9fbdcbc6fd8de
Deleted: sha256:f6355466cd156bf11358159f0c1ff82453795cc5b554d21aea00449be0e690ad
Deleted: sha256:77e6ddba346d8ad1e436256f6373dede5af4002006981b7d4116c561c759cefa
Deleted: sha256:8db758ab2fdb54da0aec53aeac876934337e6170f5a8c8872b3d4171e3d465b7
Deleted: sha256:a7fc6b405fe8ef71edfa6163d1dc9f1cb1df426049eefaa7d388e9df21a061ad
Deleted: sha256:5a3e35538f7f2e2727c8ac92f08c30002b9e8a77737de0dab91244344d59f69b
Deleted: sha256:ff986b10a018b48074e6d3a68b39aad8ccc002cdad912d4148c0f92b3729323e

09-22-18_07:24:57 - Running 
<https://builds.apache.org/job/Mesos-Reviewbot/ws/support/verify-reviews.py>
Checking if review 68731 needs verification
Skipping blocking review 68731
Checking if review 68138 needs verification
Patch never verified, needs verification
Verifying review 68138
Applying review 68138
python support/apply-reviews.py -n -r 68138
Traceback (most recent call last):
  File 
"<https://builds.apache.org/job/Mesos-Reviewbot/ws/support/verify-reviews.py";,> 
line 156, in verify_review
    apply_reviews(review_request, reviews, handler)
  File 
"<https://builds.apache.org/job/Mesos-Reviewbot/ws/support/verify-reviews.py";,> 
line 124, in apply_reviews
    apply_review(review_request["id"])
  File 
"<https://builds.apache.org/job/Mesos-Reviewbot/ws/support/verify-reviews.py";,> 
line 97, in apply_review
    shell("python support/apply-reviews.py -n -r %s" % review_id)
  File 
"<https://builds.apache.org/job/Mesos-Reviewbot/ws/support/verify-reviews.py";,> 
line 91, in shell
    command, stderr=subprocess.STDOUT, 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 'python support/apply-reviews.py -n -r 
68138' 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 292, in <module>
    main()
  File 
"<https://builds.apache.org/job/Mesos-Reviewbot/ws/support/verify-reviews.py";,> 
line 286, in main
    verify_review(review_request, handler)
  File 
"<https://builds.apache.org/job/Mesos-Reviewbot/ws/support/verify-reviews.py";,> 
line 217, in verify_review
    output += "\nFull log: "
TypeError: can't concat bytes to str
Build step 'Execute shell' marked build as failure

Reply via email to