----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34142/#review90173 -----------------------------------------------------------
src/slave/containerizer/provisioners/appc.cpp (lines 143 - 151) <https://reviews.apache.org/r/34142/#comment143167> What if the candidate is over-specified (has more labels)? Should this still be a match? src/slave/containerizer/provisioners/appc.cpp (line 203) <https://reviews.apache.org/r/34142/#comment143168> Maybe generalize the Failure message to something like "Provisioner and Image type mismatch" src/slave/containerizer/provisioners/appc.cpp (lines 484 - 487) <https://reviews.apache.org/r/34142/#comment143174> Do we want to remove everything on a failure? This could make finding the failure point difficult. - Lily Chen On June 22, 2015, 4:57 p.m., Ian Downes wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34142/ > ----------------------------------------------------------- > > (Updated June 22, 2015, 4:57 p.m.) > > > Review request for mesos, Chi Zhang, Paul Brett, Timothy Chen, and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > Discovers image(s), fetches to the image store, then provisions using > a backend. > > > Diffs > ----- > > include/mesos/mesos.proto 8df1211165169c9595e0e6e85b5ddc404345ff70 > src/Makefile.am e7de0f3d1a5efeaef47d5074defe3b40db94f573 > src/slave/containerizer/mesos/containerizer.cpp > 8c102fb7d1f79ee768cb06de3a976ea12f958712 > src/slave/containerizer/provisioners/appc.hpp PRE-CREATION > src/slave/containerizer/provisioners/appc.cpp PRE-CREATION > src/slave/flags.hpp 7634e368c72e83932dcd992d78eaca146326606b > src/slave/flags.cpp cbf431eb0627bdaf07241cc0fc4630df06fb20e2 > > Diff: https://reviews.apache.org/r/34142/diff/ > > > Testing > ------- > > > Thanks, > > Ian Downes > >