mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alex Clemmer <>
Subject Re: Review Request 55037: Added default PATH value in `launch.cpp` for Windows case.
Date Sun, 15 Jan 2017 09:00:28 GMT

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

(Updated Jan. 15, 2017, 9 a.m.)

Review request for mesos, Andrew Schwartzmeyer, Daniel Pravat, John Kordich, and Joseph Wu.


Address Till's comments.

Bugs: MESOS-6839

Repository: mesos


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, `default_path` 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 (updated)

  src/slave/containerizer/mesos/containerizer.hpp 9fcad933e1ec3b52d4dc366ba80d282deea04c0b

  src/slave/containerizer/mesos/isolators/network/cni/cni.cpp ea91c71fdfac48a2fc1d31a0ee088a73244be367

  src/slave/containerizer/mesos/launch.cpp e482ab8bdfc358f695b87cda72ca59fb64cd8c4d 




Alex Clemmer

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