-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46005/
-----------------------------------------------------------

Review request for mesos and Jie Yu.


Bugs: MESOS-5064
    https://issues.apache.org/jira/browse/MESOS-5064


Repository: mesos


Description
-------

Updated references to `flags.work_dir` in the agent code to use `.get()` now 
that the work directory is an `Option`. We enforce that 
`flags.work_dir.isSome()` in the agent initialization code and in 
'src/slave/main.cpp', so we don't have to perform this check elsewhere in the 
agent code.


Diffs
-----

  src/slave/containerizer/containerizer.cpp 
d0cae79834e451594d7675f00c5f7d2d2cd3a264 
  src/slave/containerizer/docker.cpp 9c24227171c88ee89b567ebc5200a9563dfff5be 
  src/slave/containerizer/external_containerizer.cpp 
cf4384cce44172a028c890f52f71ceb8ae109383 
  src/slave/containerizer/mesos/containerizer.cpp 
c25fa92d2a5fa9c828e77c3c0f8b1f795d1b8440 
  src/slave/containerizer/mesos/isolators/filesystem/linux.cpp 
9fc7c48f99155750fd3c18c7c102507e2726362b 
  src/slave/containerizer/mesos/isolators/filesystem/posix.cpp 
01c0ad6dbb6d509e62e769365586b3d23dcb240d 
  src/slave/containerizer/mesos/provisioner/provisioner.cpp 
8a4938ebe4e00779e88e7c538445e9ffd51202e2 
  src/slave/http.cpp a684ff504535e2b9b2064f8048f5e03a0efcb059 
  src/slave/slave.cpp f090c853b8affc4be5eecb4f616ec881fc2b60c3 
  src/slave/status_update_manager.cpp d616fd4e4f140666bb691752cc93f0bc6fcd6e6e 
  src/tests/container_logger_tests.cpp efadceafca5721bce4dbffadb35f54fd5365abb0 
  src/tests/containerizer/docker_containerizer_tests.cpp 
7accd32fba5eed196a82b1a171cb16d37b9e0539 
  src/tests/containerizer/external_containerizer_test.cpp 
226eac56725e908040abda306a2e5e3c96bce9d7 
  src/tests/containerizer/filesystem_isolator_tests.cpp 
51e60c2e8c6c8b76b51de0e7761ecbb0ca9c3304 
  src/tests/containerizer/isolator_tests.cpp 
7b4d47bd9e99b71269093d7c11559f3b74a3e22b 
  src/tests/containerizer/port_mapping_tests.cpp 
21ad1e1c53316a3bb6d914aa228ccf3658acdfbf 
  src/tests/containerizer/provisioner_appc_tests.cpp 
84fe52b6937c3b7d7628b17a2f045eec2f386b4d 
  src/tests/fetcher_cache_tests.cpp 9ffcd2375f1203bd3d7c5d0cc898e955d5cb124e 
  src/tests/gc_tests.cpp 4cb7c2f612984f7f5a9378a7f972f2438bbf28c5 
  src/tests/master_contender_detector_tests.cpp 
1426a173b1f458d67b748016aee311ac008b054a 
  src/tests/persistent_volume_tests.cpp 
d246f35046fff469b847c908de2b305ae629212f 
  src/tests/reservation_tests.cpp 2d7fb21e2fe153c2b62dfd60bbaccb350a157391 
  src/tests/slave_recovery_tests.cpp 79132344be3bcd2bda54357cd5e7e0c59a766fd8 
  src/tests/status_update_manager_tests.cpp 
f5313a124c59eba3da86a173817d0c2a6a943e17 

Diff: https://reviews.apache.org/r/46005/diff/


Testing
-------

`sudo make check`


Thanks,

Greg Mann

Reply via email to