mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mesos Reviewbot Windows <revi...@mesos.apache.org>
Subject Re: Review Request 64765: Removed a stale TODO in the allocator.
Date Thu, 21 Dec 2017 04:21:40 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64765/#review194306
-----------------------------------------------------------



FAIL: Some Mesos libprocess-tests failed.

Reviews applied: `['64765']`

Failed command: `D:\DCOS\mesos\3rdparty\libprocess\src\tests\Debug\libprocess-tests.exe`

All the build artifacts available at: http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/64765

Relevant logs:

- [libprocess-tests-stdout.log](http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/64765/logs/libprocess-tests-stdout.log):

```
[ RUN      ] SSLVerifyIPAdd/SSLTest.RequireCertificate/0
D:\DCOS\mesos\mesos\3rdparty\libprocess\src\tests\ssl_tests.cpp(446): error: (await_subprocess(client.get(),
0)).failure(): 
[++++++++++] Subprocess output.
[==========] Running 1 test from 1 test case.

[----------] Global test environment set-up.

[----------] 1 test from SSLClientTest

[ RUN      ] SSLClientTest.client

[       OK ] SSLClientTest.client (8 ms)

[----------] 1 test from SSLClientTest (9 ms total)



[----------] Global test environment tear-down

[==========] 1 test from 1 test case ran. (9 ms total)

[  PASSED  ] 1 test.

[++++++++++]

[  FAILED  ] SSLVerifyIPAdd/SSLTest.RequireCertificate/0, where GetParam() = "false" (1083
ms)
[ RUN      ] SSLVerifyIPAdd/SSLTest.RequireCertificate/1
[       OK ] SSLVerifyIPAdd/SSLTest.RequireCertificate/1 (1071 ms)
[----------] 4 tests from SSLVerifyIPAdd/SSLTest (3171 ms total)

[----------] Global test environment tear-down
[==========] 223 tests from 35 test cases ran. (36579 ms total)
[  PASSED  ] 222 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] SSLVerifyIPAdd/SSLTest.RequireCertificate/0, where GetParam() = "false"

 1 FAILED TEST
  YOU HAVE 19 DISABLED TESTS

```

- [libprocess-tests-stderr.log](http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/64765/logs/libprocess-tests-stderr.log):

```
LIBPROCESS_SSL_VERIFY_CERT set to true
I1221 04:21:19.810196  6652 openssl.cpp:563] Using CA file: C:\Users\mesos\AppData\Local\Temp\0qptEg\cert.pem
WARNING: Logging before InitGoogleLogging() is written to STDERR
I1221 04:21:21.022442  7396 openssl.cpp:509] CA directory path unspecified! NOTE: Set CA directory
path with LIBPROCESS_SSL_CA_DIR=<dirpath>
I1221 04:21:21.023442  7396 openssl.cpp:514] Will not verify peer certificate!
NOTE: Set LIBPROCESS_SSL_VERIFY_CERT=1 to enable peer certificate verification
I1221 04:21:21.023442  7396 openssl.cpp:526] Will use IP address verification in subject alternative
name certificate extension.
I1221 04:21:21.023442  7396 openssl.cpp:534] LIBPROCESS_SSL_REQUIRE_CERT implies peer certificate
verification.
LIBPROCESS_SSL_VERIFY_CERT set to true
I1221 04:21:21.023442  7396 openssl.cpp:563] Using CA file: C:\Users\mesos\AppData\Local\Temp\r4xmr1\cert.pem
bject alternative name certificate extension.
I1221 04:21:18.947185  8496 openssl.cpp:534] LIBPROCESS_SSL_REQUIRE_CERT implies peer certificate
verification.
LIBPROCESS_SSL_VERIFY_CERT set to true
I1221 04:21:18.947185  8496 openssl.cpp:563] Using CA file: C:\Users\mesos\AppData\Local\Temp\sDeMmo\cert.pem
I1221 04:21:18.947185  8496 openssl.cpp:566] Using CA dir: C:\Users\mesos\AppData\Local\Temp\sDeMmo
I1221 04:21:19.277227  8496 openssl.cpp:509] CA directory path unspecified! NOTE: Set CA directory
path with LIBPROCESS_SSL_CA_DIR=<dirpath>
I1221 04:21:19.277227  8496 openssl.cpp:514] Will not verify peer certificate!
NOTE: Set LIBPROCESS_SSL_VERIFY_CERT=1 to enable peer certificate verification
I1221 04:21:19.277227  8496 openssl.cpp:534] LIBPROCESS_SSL_REQUIRE_CERT implies peer certificate
verification.
LIBPROCESS_SSL_VERIFY_CERT set to true
I1221 04:21:19.278228  8496 openssl.cpp:563] Using CA file: C:\Users\mesos\AppData\Local\Temp\0qptEg\cert.pem
I1221 04:21:20.490447  8496 openssl.cpp:509] CA directory path unspecified! NOTE: Set CA directory
path with LIBPROCESS_SSL_CA_DIR=<dirpath>
I1221 04:21:20.490447  8496 openssl.cpp:514] Will not verify peer certificate!
NOTE: Set LIBPROCESS_SSL_VERIFY_CERT=1 to enable peer certificate verification
I1221 04:21:20.490447  8496 openssl.cpp:526] Will use IP address verification in subject alternative
name certificate extension.
I1221 04:21:20.490447  8496 openssl.cpp:534] LIBPROCESS_SSL_REQUIRE_CERT implies peer certificate
verification.
LIBPROCESS_SSL_VERIFY_CERT set to true
I1221 04:21:20.490447  8496 openssl.cpp:563] Using CA file: C:\Users\mesos\AppData\Local\Temp\r4xmr1\cert.pem
I1221 04:21:21.135428  1468 process.cpp:887] Failed to accept socket: future discarded
```

- Mesos Reviewbot Windows


On Dec. 21, 2017, 3:49 a.m., Benjamin Mahler wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/64765/
> -----------------------------------------------------------
> 
> (Updated Dec. 21, 2017, 3:49 a.m.)
> 
> 
> Review request for mesos, Alexander Rukletsov and Meng Zhu.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> This TODO was added when we used to skip roles with satisfied
> quota during the first phase of allocation. We no longer do
> this since we now allocate reservations for these roles.
> 
> 
> Diffs
> -----
> 
>   src/master/allocator/mesos/hierarchical.cpp 7f7dae1950bebd22191189c2db0f6678d2e8fa3c

> 
> 
> Diff: https://reviews.apache.org/r/64765/diff/1/
> 
> 
> Testing
> -------
> 
> N/A
> 
> 
> Thanks,
> 
> Benjamin Mahler
> 
>


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