----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69726/#review211936 -----------------------------------------------------------
src/tests/api_tests.cpp Line 4830 (original), 4830 (patched) <https://reviews.apache.org/r/69726/#comment297500> Could you explain what in particular is broken here (e.g., by linking a ticket)? Resource provider managers use in memory storage for their state which should be fine here (e.g., no agent restart). The message you are quoting in the commit message is just for user information. Right now it is hard to get a grasp for what needs to be done for this test to be enabled again. - Benjamin Bannier On Jan. 11, 2019, 11:46 p.m., Benno Evers wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69726/ > ----------------------------------------------------------- > > (Updated Jan. 11, 2019, 11:46 p.m.) > > > Review request for mesos, Gastón Kleiman and Greg Mann. > > > Repository: mesos > > > Description > ------- > > This test was broken on Windows due to the following: > > W0111 20:44:47.175993 16980 slave.cpp:8949] Persisting resource > provider manager state is not supported on Windows > > > Diffs > ----- > > src/tests/api_tests.cpp c597243e2e210e83a4ab7441fbcfa3198b43d849 > > > Diff: https://reviews.apache.org/r/69726/diff/1/ > > > Testing > ------- > > > Thanks, > > Benno Evers > >
