-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63817/
-----------------------------------------------------------
(Updated Nov. 14, 2017, 3:56 p.m.)
Review request for mesos, Akash Gupta, Jie Yu, John Kordich, Joseph Wu, and
Michael Park.
Changes
-------
Add test results.
Bugs: MESOS-6714
https://issues.apache.org/jira/browse/MESOS-6714
Repository: mesos
Description
-------
These tests "just worked" once the isolators and command was fixed to be
Windows-compatible. Particularly, `/bin/echo --author` was replaced with
an equivalent PowerShell command, and `posix/cpu,posix/mem` was replaced
with `windows/cpu`.
Diffs
-----
src/tests/slave_tests.cpp 61dd940e4452a1dbc6c0361bfc917bdc2dd16be8
Diff: https://reviews.apache.org/r/63817/diff/1/
Testing (updated)
-------
This is the preliminary work I was doing to start fixing the agent recovery
path on Windows. However, since we don't do _any_ checkpointing currently, that
bug is much more involved, and I wanted to get these patches up sooner rather
than later as they enable a dozen more agent tests.
This is dependent on the isolator chain because of the cleanup work done in
`stout/windows/os.hpp` etc. that this code also touched.
Test results in non-administrative prompt:
# mesos-tests
```
[----------] Global test environment tear-down
[==========] 755 tests from 78 test cases ran. (156570 ms total)
[ PASSED ] 755 tests.
YOU HAVE 182 DISABLED TESTS
```
In comparison to `master`:
```
[----------] Global test environment tear-down
[==========] 747 tests from 76 test cases ran. (337529 ms total)
[ PASSED ] 747 tests.
YOU HAVE 187 DISABLED TESTS
```
# libprocess-tests
```
[----------] Global test environment tear-down
[==========] 158 tests from 29 test cases ran. (992 ms total)
[ PASSED ] 158 tests.
```
# stout-tests
```
[----------] Global test environment tear-down
[==========] 259 tests from 43 test cases ran. (5185 ms total)
[ PASSED ] 259 tests.
```
Thanks,
Andrew Schwartzmeyer