----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55037/#review162010 -----------------------------------------------------------
This helper method belongs in stout, as it is precisely something which may give different results on different hosts. (i.e. `os::var`) Also, there are a couple places that could also use the function: * src/slave/containerizer/docker.cpp * src/slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/port_mapper.cpp - Joseph Wu On Jan. 15, 2017, 2:17 a.m., Alex Clemmer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55037/ > ----------------------------------------------------------- > > (Updated Jan. 15, 2017, 2:17 a.m.) > > > Review request for mesos, Andrew Schwartzmeyer, Daniel Pravat, John Kordich, > and Joseph Wu. > > > Bugs: MESOS-6839 > https://issues.apache.org/jira/browse/MESOS-6839 > > > Repository: mesos > > > Description > ------- > > Currently, when the containerizer launches a process with the POSIX > launcher, it will check if the `$PATH` environment variable (or `%PATH%` > in the case of Windows) is set in the `LaunchInfo`. If it is not, we > supply a default path. Unfortunately, this default path is specific to > POSIX. In many of our tests, this causes many of our tests to be unable > to find Windows-standard executables like `ping`, and subsequently fail. > > This commit will introduce a function, `defaultPath` that returns a > sensible default path for both POSIX and Windows. Since the Windows > implementation of this depends on the configuration of the host running > the containerizer (rather than, say, the one creating the `TaskInfo`), > we choose to implement this in `launch.cpp` instead of Stout, where a > user could mistakenly call it and expect the same output on all hosts. > > > Diffs > ----- > > src/slave/containerizer/mesos/isolators/network/cni/cni.cpp > ea91c71fdfac48a2fc1d31a0ee088a73244be367 > src/slave/containerizer/mesos/launch.cpp > e482ab8bdfc358f695b87cda72ca59fb64cd8c4d > src/slave/containerizer/mesos/utils.hpp > a54106dc4893bb222f42ede936ac9029e817faf9 > > Diff: https://reviews.apache.org/r/55037/diff/ > > > Testing > ------- > > > Thanks, > > Alex Clemmer > >