mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrew Schwartzmeyer <>
Subject Re: Review Request 63779: Windows: Fixed `SlaveReregisterTaskExecutorIds` test.
Date Thu, 30 Nov 2017 23:41:36 GMT

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

(Updated Nov. 30, 2017, 3:41 p.m.)

Review request for mesos, Akash Gupta, Alexander Rukletsov, Jie Yu, John Kordich, Joseph Wu,
and Li Li.


Rebased and added test results.

Repository: mesos


Similar to the check tests, with the `cpu` hard cap enabled, the task in
this test failed with an OOM error because PowerShell requires more
resources than were being allocated. The `defaultTaskResourcesString`
was moved into the `MesosTests` base class so that any test creating a
task can use these default resources, instead of a hard-coded string.
Additionally, the resources allocated by default for an agent were
increased so that multiple tasks can be run simultaneously (as this test

Diffs (updated)

  src/tests/check_tests.cpp f14b81aa06f6e0acbc2c3d4c65b0443d09127e78 
  src/tests/master_slave_reconciliation_tests.cpp d2501f5c140290fd4de7e7a286b51f7ec05ae6e4

  src/tests/mesos.hpp c871878fb1e2ea2716da6232ea0ed766399068b6 



Testing (updated)

`make check` passed on CentOS 7

`ctest -C Debug` passed on Windows 10


Andrew Schwartzmeyer

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