----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63389/ -----------------------------------------------------------
(Updated Nov. 3, 2017, 3:28 a.m.) Review request for mesos, Jie Yu, Joseph Wu, and Jan Schlicht. Summary (updated) ----------------- WIP: Added a mock resource provider manager. Bugs: MESOS-8144 https://issues.apache.org/jira/browse/MESOS-8144 Repository: mesos Description ------- This patch adds a `ResourceProviderManager*` parameter to the slave constructor, so we can pass in a mock resource provider manager. The mock manager can be used to test against either the agent or the resource provider. The declaration of `ResourceProviderManagerProcess` is moved to a separated `manager_process.hpp` header file so we can mock the internal process. Diffs (updated) ----- src/Makefile.am 1c97b1fd8151f87c4e9e6d62884b0ef7d582c312 src/local/local.cpp 99c1b087d75a1759ed61a4e6178e15919bee1e77 src/resource_provider/manager.hpp 3b70e75c6b6721864ae0ee9c4a593b5035d8388f src/resource_provider/manager.cpp 11f890156f0fd099f8a97b07cdc458a0726ee78e src/resource_provider/manager_process.hpp PRE-CREATION src/slave/main.cpp 94431b036524ef4db16a594c80f45b64a45728f7 src/slave/slave.hpp df1b0205124555dcb6a0efa5c237f5e77fa2bdf7 src/slave/slave.cpp 337083dbe60bba2d3773b785bdceeaf0b8fcd070 src/tests/cluster.hpp 65634121b7fe076a7cd9a1c5aa6101a96b1c009d src/tests/cluster.cpp d657da6d84e5ea28e0dad180dce069de569c5d38 src/tests/mesos.hpp e25defeb55608136e77363aa48cf820092a13a59 src/tests/mesos.cpp 9185b5bf2175be5b0f6b6a03a04e9e5445bf22fd src/tests/mock_slave.hpp 57189cee20511d145ae6b47a4dc2c66a14638dad src/tests/mock_slave.cpp db24f9e5b71f558d2f811f0da8a9cc9c7c2dd341 src/tests/resource_provider/mock_manager.hpp PRE-CREATION src/tests/resource_provider/mock_manager.cpp PRE-CREATION Diff: https://reviews.apache.org/r/63389/diff/5/ Changes: https://reviews.apache.org/r/63389/diff/4-5/ Testing ------- make Thanks, Chun-Hung Hsiao