-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54493/
-----------------------------------------------------------
(Updated Dec. 9, 2016, 7:56 p.m.)
Review request for mesos, Andrew Schwartzmeyer, Daniel Pravat, and Joseph Wu.
Bugs: MESOS-6717
https://issues.apache.org/jira/browse/MESOS-6717
Repository: mesos
Description
-------
This commit marks the resolution of all known issues blocking
MESOS-6717.
The issue this commit specifically will address the problem that all
Agent tests will fail on platforms that don't support authentication
(i.e., platforms where CMake does not pass `-DHAS_AUTHENTICATION` into
the compiler). In particular, passing the `--authentication` flag into
the Master on platforms that do not support authentication will cause
the Master to return an error, which in turn causes the test harness to
ungracefully exit. Since Windows in particular needs to be able to run
Agent tests that pass, but does not yet support authentication, it is
important to address this issue.
Our resolution here involves:
1. Exposing `HAS_AUTHENTICATION` as part of the CMake configuration
phase, so that users can pass something like
`-DHAS_AUTHENTICATION=0` to opt out.
2. Setting the default for the CMake `HAS_AUTHENTICATION` flag to be
`TRUE`, since this is the default path.
3. Causing CMake to error out if `HAS_AUTHENTICATION` is set to `TRUE`
on Windows, where it is not yet supported.
Diffs (updated)
-----
cmake/CompilationConfigure.cmake 6bd07e0050f9e66b245683e97dc44b51fcc37ea0
docs/windows.md 0301527d7315b427c8a6ed14fef72c73908103f9
src/Makefile.am 5736349e9b898f0e46ccfdaa73d437e2bc4974dc
src/tests/mesos.cpp 8fd8bcb033f47e2538aa36cd373c892a882afdfd
support/windows-build.bat b3b6d0bbe877cf3794a5c616374ffce8c1b1552c
Diff: https://reviews.apache.org/r/54493/diff/
Testing
-------
Thanks,
Alex Clemmer
