----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46003/#review128278 -----------------------------------------------------------
src/local/local.cpp (line 353) <https://reviews.apache.org/r/46003/#comment191706> Instead of using CHECK_SOME(), let's check the error and use EXIT(EXIT_FAILURE) instead so that it fails gracefully. Also, I think we should following the same pattern using a single top level temp directory with numerical sub-directories (e.g., 0, 1, 2) for slaves. src/slave/flags.cpp (line 181) <https://reviews.apache.org/r/46003/#comment191709> not suitable for whe work directory in production. src/tests/containerizer/mesos_containerizer_tests.cpp (line 117) <https://reviews.apache.org/r/46003/#comment191737> Can you return Error instead? src/tests/containerizer/mesos_containerizer_tests.cpp (line 178) <https://reviews.apache.org/r/46003/#comment191738> Not yours, but can you follow up with a patch to use 'UUID::random().toString()' for all such cases in this file? Thanks! src/tests/containerizer/mesos_containerizer_tests.cpp (line 338) <https://reviews.apache.org/r/46003/#comment191735> Why this change? - Jie Yu On April 11, 2016, 9:06 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46003/ > ----------------------------------------------------------- > > (Updated April 11, 2016, 9:06 p.m.) > > > Review request for mesos and Jie Yu. > > > Bugs: MESOS-5064 > https://issues.apache.org/jira/browse/MESOS-5064 > > > Repository: mesos > > > Description > ------- > > The default value for the agent's `--work_dir` flag was removed, the > type of the parameter was changed to `Option<string>`, and code was > added to explicitly set the work directory when necessary. > > > Diffs > ----- > > src/local/local.cpp df72ac52110b75d63df1076496b48e63d06d42ce > src/slave/flags.hpp 300db49100d989d6c0409766b1341cb956ea1631 > src/slave/flags.cpp dd7bc9a48dfd8481336a2d2ec0beecd19a342644 > src/slave/main.cpp 70df4f384b09a1fb078cd893efe52a5e3b116f48 > src/slave/slave.hpp 76f3aff394e5cecc54dcb3065cb872e238bc228e > src/slave/slave.cpp f090c853b8affc4be5eecb4f616ec881fc2b60c3 > src/tests/containerizer/mesos_containerizer_tests.cpp > 09742ff21513dc2570684d384b257868dd57a9ce > > Diff: https://reviews.apache.org/r/46003/diff/ > > > Testing > ------- > > `sudo make check` > > > Thanks, > > Greg Mann > >