----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62047/#review184737 -----------------------------------------------------------
src/slave/slave.cpp Line 7457 (original), 7471 (patched) <https://reviews.apache.org/r/62047/#comment260935> It's not absoluate and relative, it's real and virtual :) Can you clarify here and in other locations that we do both because the old one needs to remain for backwards compatibility, but we want to remove exposing the real path (maybe with a ticket to do so) in 2.x? src/slave/slave.cpp Lines 7466-7470 (original), 7484-7491 (patched) <https://reviews.apache.org/r/62047/#comment260937> What was going on here originally with the two different calls? src/slave/slave.cpp Lines 7487-7488 (patched) <https://reviews.apache.org/r/62047/#comment260936> For the virtual path, do we want to attach one directory up from latest and let users specify "latest"? Otherwise, we're still not supporting the ability to look at previous executor runs? - Benjamin Mahler On Sept. 1, 2017, 10:54 p.m., Zhitao Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62047/ > ----------------------------------------------------------- > > (Updated Sept. 1, 2017, 10:54 p.m.) > > > Review request for mesos, Benjamin Mahler and Jason Lai. > > > Bugs: MESOS-7899 > https://issues.apache.org/jira/browse/MESOS-7899 > > > Repository: mesos > > > Description > ------- > > This patch added support to attach a virtual path > `/frameworks/{framework-id}/executors/{executor-id}/latest` to `Files` class > in agent, which can be used to navigate an executor's latest sandbox without > needing to know `work_dir` and `slave_id`. > > > Diffs > ----- > > src/slave/paths.hpp 51b481fc0870f1e95448f85ee2fd485fceea1919 > src/slave/paths.cpp 1a6943f03493079ab291616d3e36e0fd8809cf33 > src/slave/slave.cpp df920ec07cd59c7ba6baccfc1c20ed3809f187d6 > > > Diff: https://reviews.apache.org/r/62047/diff/1/ > > > Testing > ------- > > Ran a master/agent pair, launch a task with `mesos-execute` and use > `files/browse` endpoint to browse the virtual path. > > > Thanks, > > Zhitao Li > >
