Re: Review Request 34976: Added installation instructions for Ubuntu 14.04 and OSX

2015-06-03 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 3, 2015, 6:44 a.m., Marco Massenzio wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34976

Re: Review Request 34984: Added help for files

2015-06-03 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 3, 2015, 6:11 a.m., Aditi Dixit wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34984/ --- (Updated

Re: Review Request 34748: Defined protobuf for usage returned by Resource Monitor. Reused ResourceUsage. Pass ResourceUsage to ResourceEstimator

2015-06-02 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On June 2, 2015, 8:16 p.m., Bartek Plotka wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34748

Re: Review Request 34863: Add tests for new qdisc statistics functions.

2015-06-02 Thread Mesos ReviewBot
to apply patch - Mesos ReviewBot On June 2, 2015, 5:31 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34863

Re: Review Request 34962: Moved NoopResourceEstimator to a separate file.

2015-06-02 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 2, 2015, 11:15 p.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34962/ --- (Updated June

Re: Review Request 34958: Moved ingress root handle to a proper location.

2015-06-02 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On June 2, 2015, 10:06 p.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34958

Re: Review Request 34894: Add new message for Traffic Control statistics

2015-06-02 Thread Mesos ReviewBot
to apply patch - Mesos ReviewBot On June 2, 2015, 8:52 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34894

Re: Review Request 34687: (De)Serializing MasterInfo PB to JSON

2015-06-02 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 2, 2015, 9:52 p.m., Marco Massenzio wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34687

Re: Review Request 35253: Enable configure to detect libevent and openssl on Mac OS X.

2015-06-09 Thread Mesos ReviewBot
, 29530, 29531, 29532, 29533, 29534, 29535, 29536, 29538, 29539, 29540, 29569, 28763, 35253] All tests passed. - Mesos ReviewBot On June 9, 2015, 2:52 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply

Re: Review Request 35291: Fixed some typos

2015-06-10 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 10, 2015, 10:20 a.m., Aditi Dixit wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35291/ --- (Updated

Re: Review Request 35260: Refactored the ResourceMonitor to get statistics from the Slave.

2015-06-09 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 9, 2015, 6:54 p.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35260/ --- (Updated

Re: Review Request 35262: Logged framework capabilities during (re-)registration.

2015-06-09 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 9, 2015, 7:05 p.m., Ben Mahler wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35262/ --- (Updated

Re: Review Request 34318: Update callers of os::getenv() in libprocess.

2015-06-04 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On June 4, 2015, 6:14 a.m., Greg Mann wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34318

Re: Review Request 35028: Added a fixed resource estimator.

2015-06-04 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 4, 2015, 5:56 p.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35028/ --- (Updated June 4

Re: Review Request 35264: Added a slave integration test in MonitorTest.

2015-06-09 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On June 9, 2015, 7:51 p.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35264

Re: Review Request 35273: Removed unnecessary accessors in filter code.

2015-06-09 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On June 9, 2015, 10:59 p.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35273

Re: Review Request 35281: Added QoS module loader to ::create() factory.

2015-06-09 Thread Mesos ReviewBot
] All tests passed. - Mesos ReviewBot On June 10, 2015, 1:09 a.m., Niklas Nielsen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35281

Re: Review Request 30952: Adding scheduler validations to master

2015-06-09 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 10, 2015, 12:08 a.m., Isabel Jimenez wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30952

Re: Review Request 33339: Add a Java example framework to test persistent volumes.

2015-06-07 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 7, 2015, 8:10 a.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9

Re: Review Request 35438: Fixed fetcher cache test race for resource offers when starting tasks and changed corresponding CHECK to EXPECT.

2015-06-22 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 22, 2015, 2:57 p.m., Bernd Mathiske wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35438

Re: Review Request 35770: Consistent 'Test' suffix in mesos test names.

2015-06-23 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 23, 2015, 1:21 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35770

Re: Review Request 35785: Getting Started: Fixed inconsistent style for CentOS 7.1.

2015-06-23 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 23, 2015, 1:23 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35785/ --- (Updated

Re: Review Request 35777: Made post-reviews.py handle bad (or not) ReviewBoard URLs.

2015-06-23 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 23, 2015, 8:19 a.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35777/ --- (Updated

Re: Review Request 35775: Removed libgen.h include.

2015-06-23 Thread Mesos ReviewBot
, 34258, 35773, 35774, 35775] All tests passed. - Mesos ReviewBot On June 23, 2015, 8:27 a.m., Till Toenshoff wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35775

Re: Review Request 35743: flags: fixed const'ness of load

2015-06-22 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 22, 2015, 9:27 p.m., Jojy Varghese wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35743

Re: Review Request 35738: Add slave metric to count container launch failures.

2015-06-22 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 23, 2015, 12:42 a.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35738/ --- (Updated

Re: Review Request 35771: Fixed post-reviews.py hanging bug.

2015-06-23 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 23, 2015, 7:17 a.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35771/ --- (Updated

Re: Review Request 35797: Updated FrameworkInfo to update capabilities (Part 1)

2015-06-23 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 24, 2015, 2:44 a.m., Aditi Dixit wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35797/ --- (Updated

Re: Review Request 33339: Add a Java example framework to test persistent volumes.

2015-06-21 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 21, 2015, 9:57 a.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9

Re: Review Request 35721: Set the owner of persistent volumes to frameworkInfo.user .

2015-06-21 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 22, 2015, 1:56 a.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35721

Re: Review Request 35335: Fixed JSON output error TaskInfo executor_id in slave/http.cpp

2015-06-20 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 18, 2015, 11:11 p.m., Brendan Chang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35335

Re: Review Request 29507: Added Configurable Slave Ping Timeouts

2015-06-22 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 22, 2015, 10:03 a.m., Adam B wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29507/ --- (Updated June

Re: Review Request 35863: Fix a typo in docs/docker-containerizer.md file

2015-06-25 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 25, 2015, 5:47 a.m., Zhiwei Chen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35863/ --- (Updated

Re: Review Request 35702: [WIP] Added /reserve HTTP endpoint to the master.

2015-06-25 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 26, 2015, 4:44 a.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35702

Re: Review Request 35874: Added template parameters and constructors to hashset which reflex the state of hashmap

2015-06-25 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 25, 2015, 1:45 p.m., Alexander Rojas wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35874

Re: Review Request 35861: Correct include list errors for stout

2015-06-25 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 25, 2015, 6:32 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35861/ --- (Updated

Re: Review Request 35841: Fix usage of ATOMIC_FLAG_INIT

2015-06-25 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 25, 2015, 5:27 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35841/ --- (Updated

Re: Review Request 35861: Correct include list errors for stout

2015-06-25 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 25, 2015, 9:58 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35861/ --- (Updated

Re: Review Request 35841: Fix usage of ATOMIC_FLAG_INIT

2015-06-24 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 24, 2015, 8:54 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35841/ --- (Updated

Re: Review Request 35843: Making markup doxygen-friendly

2015-06-24 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 24, 2015, 7:57 p.m., Artem Harutyunyan wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35843

Re: Review Request 34835: Add constexpr to C++ whitelist

2015-06-24 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 24, 2015, 9:14 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34835/ --- (Updated

Re: Review Request 35858: Added Message call support to the master and the C++ scheduler library.

2015-06-24 Thread Mesos ReviewBot
] All tests passed. - Mesos ReviewBot On June 25, 2015, 12:31 a.m., Vinod Kone wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35858

Re: Review Request 35861: Correct include list errors for stout

2015-06-24 Thread Mesos ReviewBot
not apply error: 3rdparty/libprocess/3rdparty/stout/include/stout/os/execenv.hpp: does not exist in index Failed to apply patch - Mesos ReviewBot On June 25, 2015, 1:05 a.m., Paul Brett wrote: --- This is an automatically generated e-mail

Re: Review Request 35849: Added a precision test for Resources.

2015-06-24 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 24, 2015, 10:10 p.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35849/ --- (Updated June

Re: Review Request 35568: Remove html from libprocess Developer Guide.

2015-06-24 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 24, 2015, 2:56 p.m., Joerg Schad wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35568

Re: Review Request 35861: Correct include list errors for stout

2015-06-24 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 25, 2015, 4:29 a.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35861/ --- (Updated

Re: Review Request 35816: Fixed the incorrect CHECK_EQ in updateAllocation.

2015-06-24 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 24, 2015, 3:35 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35816/ --- (Updated

Re: Review Request 35815: Testing JSON paylodad in ZooKeeper

2015-06-25 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 25, 2015, 7:53 a.m., Marco Massenzio wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35815

Re: Review Request 35738: Add slave metric to count container launch failures.

2015-06-22 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 22, 2015, 7:26 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35738/ --- (Updated

Re: Review Request 35033: Define potentially missing MS_* mount flags.

2015-06-22 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 22, 2015, 4:51 p.m., Ian Downes wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35033/ --- (Updated

Re: Review Request 34908: Rename --docker_sandbox_directory flag for general use.

2015-06-22 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 22, 2015, 4:55 p.m., Ian Downes wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34908/ --- (Updated

Re: Review Request 35586: Updated LinuxLauncher to receive list of namespaces.

2015-06-22 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 23, 2015, 2:48 a.m., Kapil Arya wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35586

Re: Review Request 35763: Document workaround for LIBNL_CFLAGS path

2015-06-22 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 23, 2015, 2:39 a.m., Roger Ignazio wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35763

Re: Review Request 35765: Updated network-monitoring docs to reflect correct libnl version.

2015-06-22 Thread Mesos ReviewBot
] All tests passed. - Mesos ReviewBot On June 23, 2015, 3:09 a.m., Kapil Arya wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35765

Re: Review Request 35756: Fixed a race condition in hook tests for remove-executor hook.

2015-06-22 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 23, 2015, 12:53 a.m., Kapil Arya wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35756/ --- (Updated

Re: Review Request 35682: Sped up Allocator::updateSlave() and Allocator::updateAllocation().

2015-06-20 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On June 20, 2015, 1:08 a.m., Vinod Kone wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35682

Re: Review Request 35687: Added capabilities to state.json

2015-06-20 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 20, 2015, 7:14 a.m., Aditi Dixit wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35687/ --- (Updated

Re: Review Request 35686: Change the if statement to a CHECK at the end of _runTask.

2015-06-20 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 20, 2015, 4:42 a.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35686/ --- (Updated

Re: Review Request 35766: Added --with-libnl=DIR configure flag.

2015-06-23 Thread Mesos ReviewBot
, 35766] All tests passed. - Mesos ReviewBot On June 23, 2015, 6:18 a.m., Kapil Arya wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35766

Re: Review Request 35728: Fix failing test: SlaveTest.ROOT_RunTaskWithCommandInfoWithUser.

2015-06-22 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 22, 2015, 11:34 a.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35728

Re: Review Request 35717: Add reservations support to master's state.json

2015-06-22 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 22, 2015, 11:37 a.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35717

Re: Review Request 35769: Consistent 'Test' suffix in libprocess test names.

2015-06-23 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 23, 2015, 7:06 a.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35769/ --- (Updated

Re: Review Request 35772: Made post-reviews.py print parent diff only if one exists.

2015-06-23 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 23, 2015, 7:21 a.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35772/ --- (Updated

Re: Review Request 32982: Added reservation user guide.

2015-06-26 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 26, 2015, 4:04 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32982/ --- (Updated

Re: Review Request 35927: Added TaskStatus::Reason to Termination Message.

2015-06-26 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 26, 2015, 4:26 p.m., Joerg Schad wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35927/ --- (Updated

Re: Review Request 32982: Added reservation user guide.

2015-06-26 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 26, 2015, 6:12 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32982/ --- (Updated

Re: Review Request 32982: Added reservation user guide.

2015-06-26 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 26, 2015, 8:43 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32982/ --- (Updated

Re: Review Request 35699: Added an invariant CHECK_EQ for available resources in HierarchicalAllocator::updateAllocation.

2015-06-20 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 20, 2015, 7:52 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35699/ --- (Updated

Re: Review Request 35702: [WIP] Add /reserve HTTP endpoint to the master.

2015-06-20 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 20, 2015, 8:19 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35702/ --- (Updated

Re: Review Request 35697: mesos: Fixed test names.

2015-06-20 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 20, 2015, 7:14 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35697/ --- (Updated

Re: Review Request 35701: Minor formatting cleanup in the Master.

2015-06-20 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 20, 2015, 8 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35701/ --- (Updated

Re: Review Request 35287: Rename OptionT::get(const T _t) to getOrElse() and refactor original functions (stout)

2015-06-20 Thread Mesos ReviewBot
: patch does not apply Failed to apply patch - Mesos ReviewBot On June 20, 2015, 4:48 p.m., Mark Wang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35287

Re: Review Request 35567: Added 'executor_environment_variables' flag to slave.

2015-06-20 Thread Mesos ReviewBot
to apply patch - Mesos ReviewBot On June 20, 2015, 7 p.m., Benjamin Hindman wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35567

Re: Review Request 35696: libprocess: Fixed test names.

2015-06-20 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 20, 2015, 7:14 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35696/ --- (Updated

Re: Review Request 35611: Added initial doxygen documentation for stout flags.

2015-06-20 Thread Mesos ReviewBot
: patch does not apply Failed to apply patch - Mesos ReviewBot On June 20, 2015, 3:19 p.m., Benjamin Hindman wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35611

Re: Review Request 35703: Set refuse seconds on the correct filter in reservation test.

2015-06-20 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 20, 2015, 8:51 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35703/ --- (Updated

Re: Review Request 33339: Add a Java example framework to test persistent volumes.

2015-06-21 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 21, 2015, 5:37 a.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9

Re: Review Request 33241: docs: Add documentation on observability metrics.

2015-06-24 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 24, 2015, 4:28 p.m., Ricardo Cervera-Navarro wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33241

Re: Review Request 35797: Updated FrameworkInfo to update capabilities (Part 1)

2015-06-23 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 23, 2015, 8:08 p.m., Aditi Dixit wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35797/ --- (Updated

Re: Review Request 35756: Fixed a race condition in hook tests for remove-executor hook.

2015-06-23 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 23, 2015, 9:47 p.m., Kapil Arya wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35756/ --- (Updated

Re: Review Request 35766: Added --with-nl=DIR configure flag.

2015-06-23 Thread Mesos ReviewBot
, 35766] All tests passed. - Mesos ReviewBot On June 23, 2015, 10:49 p.m., Kapil Arya wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35766

Re: Review Request 35939: New Conflict HTTP response type

2015-06-26 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 26, 2015, 6:58 p.m., Isabel Jimenez wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35939

Re: Review Request 32982: Added reservation user guide.

2015-06-26 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 26, 2015, 7:53 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32982/ --- (Updated

Re: Review Request 35799: Support mounting relative paths with docker.

2015-06-26 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 26, 2015, 11:26 p.m., Timothy Chen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35799

Re: Review Request 35935: Added a simulation benchmark for reconciliation.

2015-06-26 Thread Mesos ReviewBot
] Failed command: ./support/apply-review.sh -n -r 35911 Error: 2015-06-27 01:16:30 URL:https://reviews.apache.org/r/35911/diff/raw/ [11665/11665] - 35911.patch [1] error: patch failed: src/sched/sched.cpp:915 error: src/sched/sched.cpp: patch does not apply Failed to apply patch - Mesos ReviewBot

Re: Review Request 35958: Updated SlaveTest.PingTimeoutNoPings test to use custom timeout values.

2015-06-26 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 27, 2015, 12:46 a.m., Adam B wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35958

Re: Review Request 35702: Added /reserve HTTP endpoint to the master.

2015-06-26 Thread Mesos ReviewBot
] All tests passed. - Mesos ReviewBot On June 26, 2015, 10:56 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35702

Re: Review Request 34835: Add constexpr to C++ whitelist

2015-06-26 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 26, 2015, 10:36 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34835/ --- (Updated

Re: Review Request 35361: Added doc for standalone to HA

2015-06-11 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 11, 2015, 3:22 p.m., Michael Schenck wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35361

Re: Review Request 35281: Added QoS module loader to ::create() factory.

2015-06-11 Thread Mesos ReviewBot
] All tests passed. - Mesos ReviewBot On June 11, 2015, 6:55 p.m., Niklas Nielsen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35281

Re: Review Request 35157: Added unit tests for fetching ResourceUsage in both QoS Controller and Resource Estimator .

2015-06-11 Thread Mesos ReviewBot
: ./support/apply-review.sh -n -r 35157 Error: 2015-06-11 19:41:24 URL:https://reviews.apache.org/r/35157/diff/raw/ [18101/18101] - 35157.patch [1] error: patch failed: src/tests/oversubscription_tests.cpp:63 error: src/tests/oversubscription_tests.cpp: patch does not apply Failed to apply patch - Mesos

Re: Review Request 35350: Fixes markdown of the Doxygen Styleguide to provide better rendering.

2015-06-11 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 11, 2015, 11:59 a.m., Alexander Rojas wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35350

Re: Review Request 35305: Added a SlaveTest to caputure the case when containerizer usage fails.

2015-06-10 Thread Mesos ReviewBot
] All tests passed. - Mesos ReviewBot On June 10, 2015, 6:30 p.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35305

Re: Review Request 35367: Changed Resourcs JSON model() to combine non-revocable resources and ignore revocable resources.

2015-06-11 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 11, 2015, 7:56 p.m., Jiang Yan Xu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35367/ --- (Updated

Re: Review Request 35313: Added slave metrics for revocable resources.

2015-06-10 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 10, 2015, 7:50 p.m., Ben Mahler wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35313

Re: Review Request 35321: Implemented the fixed resource estimator.

2015-06-10 Thread Mesos ReviewBot
, 35320, 35321] All tests passed. - Mesos ReviewBot On June 10, 2015, 7:49 p.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35321

Re: Review Request 35363: Improvements on libprocess/README.md.

2015-06-11 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 11, 2015, 8:40 p.m., Joerg Schad wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35363/ --- (Updated

Re: Review Request 35330: Capped number of parallel inspect instances on a docker ps call.

2015-06-11 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 11, 2015, 9:39 p.m., Lily Chen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35330/ --- (Updated

Re: Review Request 35455: Update 'Getting Started' documentation.

2015-06-15 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 15, 2015, 12:20 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35455

Re: Review Request 34392: Added a method to Path which returns the modification time of the represented path.

2015-06-15 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 15, 2015, 11:34 a.m., Alexander Rojas wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34392

<    1   2   3   4   5   6   7   8   9   10   >