----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38137/#review97944 -----------------------------------------------------------
Bad patch! Reviews applied: [38141, 38137] Failed command: ./support/apply-review.sh -n -r 38137 Error: 2015-09-07 22:18:35 URL:https://reviews.apache.org/r/38137/diff/raw/ [86583/86583] -> "38137.patch" [1] error: patch failed: src/Makefile.am:483 error: src/Makefile.am: patch does not apply error: patch failed: src/slave/containerizer/provisioners/appc.cpp:33 error: src/slave/containerizer/provisioners/appc.cpp: patch does not apply error: patch failed: src/slave/containerizer/provisioners/appc/paths.hpp:47 error: src/slave/containerizer/provisioners/appc/paths.hpp: patch does not apply error: patch failed: src/slave/containerizer/provisioners/appc/paths.cpp:85 error: src/slave/containerizer/provisioners/appc/paths.cpp: patch does not apply error: src/slave/containerizer/provisioners/paths.hpp: already exists in index error: src/slave/containerizer/provisioners/paths.cpp: already exists in index Failed to apply patch - Mesos ReviewBot On Sept. 7, 2015, 9:58 p.m., Timothy Chen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38137/ > ----------------------------------------------------------- > > (Updated Sept. 7, 2015, 9:58 p.m.) > > > Review request for mesos, Jojy Varghese, Till Toenshoff, and Jiang Yan Xu. > > > Repository: mesos > > > Description > ------- > > Joining all the commits around provisioner and local store into one review so > it's easier to review, as patches > are changing code on top of each other. > > All the commits are going to committed together. > > > Diffs > ----- > > src/Makefile.am 5fdca0f > src/messages/docker_provisioner.hpp PRE-CREATION > src/messages/docker_provisioner.proto PRE-CREATION > src/slave/containerizer/provisioner.hpp 9e0e0b8 > src/slave/containerizer/provisioner.cpp 95894c0 > src/slave/containerizer/provisioners/appc.cpp fc5ee19 > src/slave/containerizer/provisioners/appc/paths.hpp fb3a1a7 > src/slave/containerizer/provisioners/appc/paths.cpp e6be851 > src/slave/containerizer/provisioners/docker.hpp PRE-CREATION > src/slave/containerizer/provisioners/docker.cpp PRE-CREATION > src/slave/containerizer/provisioners/docker/local_store.hpp PRE-CREATION > src/slave/containerizer/provisioners/docker/local_store.cpp PRE-CREATION > src/slave/containerizer/provisioners/docker/paths.hpp PRE-CREATION > src/slave/containerizer/provisioners/docker/paths.cpp PRE-CREATION > src/slave/containerizer/provisioners/docker/reference_store.hpp > PRE-CREATION > src/slave/containerizer/provisioners/docker/reference_store.cpp > PRE-CREATION > src/slave/containerizer/provisioners/docker/store.hpp PRE-CREATION > src/slave/containerizer/provisioners/paths.hpp PRE-CREATION > src/slave/containerizer/provisioners/paths.cpp PRE-CREATION > src/slave/flags.hpp b8335aa > src/slave/flags.cpp 7539441 > src/tests/containerizer/docker_provisioner_tests.cpp PRE-CREATION > > Diff: https://reviews.apache.org/r/38137/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Timothy Chen > >
