mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrew Schwartzmeyer <>
Subject Re: Review Request 65409: Fixed `SlaveRecoveryTest.ReconcileTasksMissingFromSlave`.
Date Mon, 29 Jan 2018 21:45:20 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Jan. 29, 2018, 1:45 p.m.)

Review request for mesos, Akash Gupta, Gaston Kleiman, Jie Yu, and Joseph Wu.

Bugs: MESOS-6713

Repository: mesos


Because it is not possible to delete a file (or a folder recursively)
with open handles on Windows, we have to explicitly `reset()` the agent
before removing the framework meta directory. Otherwise, the task status
update manager will be destructed too late, and so an open handle for
`task.updates` will cause the `os::rmdir` to fail.

This is safe because we previously destructed the agent anyway, just
later in the test when it was reassigned.


  src/tests/slave_recovery_tests.cpp 6dcbedb94a2c7b3d65770fc89920c0cad1298d1f 


Testing (updated)

make check on CentOS 7, all passed
ctest on Windows, all passed including new SlaveRecoveryTests


Andrew Schwartzmeyer

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message