Re: Review Request 33490: MESOS-2633 Avoid adding duplicate tasks

2015-04-23 Thread Mesos ReviewBot
/apply-review.sh -n -r 33490 Error: 2015-04-23 19:43:56 URL:https://reviews.apache.org/r/33490/diff/raw/ [1294/1294] - 33490.patch [1] error: src/master/framework.cpp: does not exist in index Failed to apply patch - Mesos ReviewBot On April 23, 2015, 7:36 p.m., Marco Massenzio wrote

Re: Review Request 33491: Set the supplementary groups list when switching users.

2015-04-23 Thread Mesos ReviewBot
. - Mesos ReviewBot On April 23, 2015, 8:16 p.m., James Peach wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33491/ --- (Updated

Re: Review Request 33527: Updated documentation for ClangFormat.

2015-04-26 Thread Mesos ReviewBot
. - Mesos ReviewBot On April 26, 2015, 9:59 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33527

Re: Review Request 32509: Documented the scheduler Event/Call protobufs.

2015-04-22 Thread Mesos ReviewBot
, 32843, 32505, 32506, 32844, 32845, 33465, 32509] All tests passed. - Mesos ReviewBot On April 23, 2015, 2:55 a.m., Vinod Kone wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32509

Re: Review Request 32509: Documented the scheduler Event/Call protobufs.

2015-04-22 Thread Mesos ReviewBot
, 32843, 32505, 32506, 32844, 32845, 32509] All tests passed. - Mesos ReviewBot On April 22, 2015, 9:44 p.m., Vinod Kone wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32509

Re: Review Request 33644: MemoryTestHelper: Changed to do memset before mlock.

2015-04-28 Thread Mesos ReviewBot
. - Mesos ReviewBot On April 28, 2015, 11 p.m., Chi Zhang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33644/ --- (Updated

Re: Review Request 33781: Add license blobs to Java JNI cpp files

2015-05-02 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 2, 2015, 2:35 p.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33781/ --- (Updated

Re: Review Request 29889: Recover Docker containers when mesos slave is in a container

2015-05-02 Thread Mesos ReviewBot
/apply-review.sh -n -r 29327 Error: 2015-05-02 20:34:47 URL:https://reviews.apache.org/r/29327/diff/raw/ [2935/2935] - 29327.patch [1] error: patch failed: src/slave/containerizer/docker.cpp:499 error: src/slave/containerizer/docker.cpp: patch does not apply Failed to apply patch - Mesos ReviewBot

Re: Review Request 33752: MESOS-2537: Fix AC_ARG_ENABLED option processing

2015-05-01 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 1, 2015, 3:42 p.m., James Peach wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33752/ --- (Updated

Re: Review Request 33490: MESOS-2633 Avoid adding duplicate tasks

2015-04-30 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On April 30, 2015, 9:46 p.m., Marco Massenzio wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33490

Re: Review Request 33754: Update pthread autoconf macros for libprocess.

2015-05-01 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 1, 2015, 4:02 p.m., James Peach wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33754/ --- (Updated

Re: Review Request 33753: Update pthread and python autoconf macros for Mesos.

2015-05-01 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 1, 2015, 4:02 p.m., James Peach wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33753/ --- (Updated

Re: Review Request 33765: MESOS-2665: fix queuing discipline wrapper in linux/routing/queueing

2015-05-01 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 1, 2015, 7:24 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33765/ --- (Updated May

Re: Review Request 33631: Fixed a bug in port mapping isolator which will cause SIGABRT during slave recovery.

2015-04-28 Thread Mesos ReviewBot
. - Mesos ReviewBot On April 28, 2015, 5:55 p.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33631/ --- (Updated

Re: Review Request 33828: Fix for MESOS-2690. Issue with enable-optimize build.

2015-05-04 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 4, 2015, 11:48 p.m., Joris Van Remoortere wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33828

Re: Review Request 33876: Added usages() to resource monitor

2015-05-05 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On May 6, 2015, 2:16 a.m., Niklas Nielsen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33876

Re: Review Request 33792: Add InsensitiveHashMap.

2015-05-03 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 3, 2015, 9:37 a.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33792/ --- (Updated

Re: Review Request 33793: HTTP headers should be considered case-insensitive.

2015-05-03 Thread Mesos ReviewBot
distcheck Error: make dist-gzip am__post_remove_distdir='@:' make[1]: Entering directory `/home/jenkins/jenkins-slave/workspace/mesos-reviewbot' if test -d mesos-0.23.0; then find mesos-0.23.0 -type d ! -perm -200 -exec chmod u+w {} ';' rm -rf mesos-0.23.0 || { sleep 5 rm -rf mesos-0.23.0

Re: Review Request 33919: Integrated resources estimator with the slave.

2015-05-06 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On May 6, 2015, 10:35 p.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33919/ --- (Updated

Re: Review Request 33872: Renamed shutdown_tests.cpp to teardown_tests.cpp.

2015-05-06 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On May 6, 2015, 5:15 p.m., Vinod Kone wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33872

Re: Review Request 33865: Added RevocableInfo message to Resource protobuf.

2015-05-05 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 5, 2015, 9:13 p.m., Vinod Kone wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33865/ --- (Updated May

Re: Review Request 33849: mesos: use standard macros for compiler and vendor detection

2015-05-05 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 5, 2015, 3:53 p.m., James Peach wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33849/ --- (Updated

Re: Review Request 34018: Update existing lambdas to meet style guide

2015-05-09 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On May 9, 2015, 7:52 p.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34018

Re: Review Request 33514: Wired up --allocator flag in master.

2015-05-08 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On May 8, 2015, 8:44 p.m., Alexander Rukletsov wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33514

Re: Review Request 32398: Persisted the reservation state on the slave.

2015-05-11 Thread Mesos ReviewBot
, 29748, 32398] All tests passed. - Mesos ReviewBot On May 11, 2015, 10:42 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32398

Re: Review Request 34068: The test case of extend hashmap to support custom equality and hash

2015-05-11 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On May 12, 2015, 12:56 a.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34068

Re: Review Request 34016: Change the type of signaledWrapper to unique_ptr

2015-05-09 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 9, 2015, 5:07 p.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34016/ --- (Updated

Re: Review Request 33249: Send statusUpdate to scheduler on containerizer launch failure

2015-05-07 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 7, 2015, 4:17 p.m., Jay Buffington wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33249/ --- (Updated

Re: Review Request 33849: mesos: use standard macros for compiler and vendor detection

2015-05-05 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 5, 2015, 5:52 p.m., James Peach wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33849/ --- (Updated

Re: Review Request 32398: Persisted the reservation state on the slave.

2015-05-12 Thread Mesos ReviewBot
, 29748, 32398] All tests passed. - Mesos ReviewBot On May 12, 2015, 6:44 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32398

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

2015-05-12 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 12, 2015, 8:43 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 33155: Added tests for slave removal metrics.

2015-05-12 Thread Mesos ReviewBot
] All tests passed. - Mesos ReviewBot On May 12, 2015, 11:32 p.m., Ben Mahler wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33155

Re: Review Request 34129: Add 2 optional args public_ip and public_port for libprocess to advertise.

2015-05-12 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 12, 2015, 11:30 p.m., Anindya Sinha wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34129

Re: Review Request 30643: Optionally specify executor for mesos execute.

2015-05-12 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 13, 2015, 12:22 a.m., Ian Downes wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30643/ --- (Updated

Re: Review Request 33514: Wired up --allocator flag in master.

2015-05-14 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On May 14, 2015, 1:23 p.m., Alexander Rukletsov wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33514

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

2015-05-16 Thread Mesos ReviewBot
Failed to apply patch - Mesos ReviewBot On May 16, 2015, 8:31 p.m., Greg Mann wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34318

Re: Review Request 34317: Updated callers of os::getenv() in /src.

2015-05-16 Thread Mesos ReviewBot
Failed to apply patch - Mesos ReviewBot On May 16, 2015, 8:31 p.m., Greg Mann wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34317

Re: Review Request 34319: Update callers of os::getenv() in stout.

2015-05-16 Thread Mesos ReviewBot
Failed to apply patch - Mesos ReviewBot On May 16, 2015, 8:31 p.m., Greg Mann wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34319

Re: Review Request 33372: Added decorator documentation and described the semantic change in Mesos 0.23.0

2015-05-14 Thread Mesos ReviewBot
, 32948, 31017, 33372] All tests passed. - Mesos ReviewBot On May 14, 2015, 5:06 p.m., Niklas Nielsen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33372

Re: Review Request 33919: Integrated resources estimator with the slave.

2015-05-13 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On May 14, 2015, 12:07 a.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33919

Re: Review Request 34128: Enable different IP/Port for external access.

2015-05-12 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 12, 2015, 11:23 p.m., Anindya Sinha wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34128

Re: Review Request 34306: Added 'revocable_offers' field to FrameworkInfo.

2015-05-15 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On May 16, 2015, 12:24 a.m., Vinod Kone wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34306

Re: Review Request 34353: Added right ammount of spacing between structs

2015-05-18 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 18, 2015, 2:14 p.m., Alexander Rojas wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34353

Re: Review Request 30612: Added /master/frameworks/{framework}/tasks/{task} endpoint.

2015-05-18 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On May 18, 2015, 12:56 p.m., Alexander Rojas wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30612

Re: Review Request 34258: Removed os::dirname and os::basename.

2015-05-17 Thread Mesos ReviewBot
] All tests passed. - Mesos ReviewBot On May 17, 2015, 10:58 p.m., Till Toenshoff wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34258

Re: Review Request 34195: Refactoring to use BaseFlags common functionality

2015-05-18 Thread Mesos ReviewBot
/flags.hpp: patch does not apply error: patch failed: 3rdparty/libprocess/3rdparty/stout/tests/flags_tests.cpp:483 error: 3rdparty/libprocess/3rdparty/stout/tests/flags_tests.cpp: patch does not apply Failed to apply patch - Mesos ReviewBot On May 18, 2015, 5:28 p.m., Marco Massenzio wrote

Re: Review Request 34362: Include ExecutorInfos in master/state.json

2015-05-18 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 18, 2015, 5:41 p.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34362

Re: Review Request 34310: Use IDLE scheduling for revocable CPU in cgroups isolator.

2015-05-18 Thread Mesos ReviewBot
: ./support/apply-review.sh -n -r 34308 Error: 2015-05-18 20:30:01 URL:https://reviews.apache.org/r/34308/diff/raw/ [1838/1838] - 34308.patch [1] error: patch failed: include/mesos/resources.hpp:105 error: include/mesos/resources.hpp: patch does not apply Failed to apply patch - Mesos ReviewBot

Re: Review Request 34310: Use IDLE scheduling for revocable CPU in cgroups isolator.

2015-05-18 Thread Mesos ReviewBot
: ./support/apply-review.sh -n -r 34308 Error: 2015-05-18 21:03:45 URL:https://reviews.apache.org/r/34308/diff/raw/ [1126/1126] - 34308.patch [1] error: patch failed: include/mesos/resources.hpp:189 error: include/mesos/resources.hpp: patch does not apply Failed to apply patch - Mesos ReviewBot On May

Re: Review Request 34375: Removed use of namespace aliases.

2015-05-18 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 18, 2015, 11:13 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34375/ --- (Updated

Re: Review Request 34359: Support multiple reasons in status update message.

2015-05-18 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 18, 2015, 6:03 p.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34359

Re: Review Request 34129: Add 2 optional args public_ip and public_port for libprocess to advertise.

2015-05-18 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 18, 2015, 10:08 p.m., Anindya Sinha wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34129

Re: Review Request 34378: Fixed the dependency between 'summarize' and 'model'.

2015-05-18 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 18, 2015, 11:23 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34378/ --- (Updated

Re: Review Request 34387: Moved up Slave and Framework structs in master.hpp.

2015-05-18 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 19, 2015, 1:56 a.m., Ben Mahler wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34387/ --- (Updated

Re: Review Request 33919: Integrated resources estimator with the slave.

2015-05-12 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On May 13, 2015, 12:37 a.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33919

Re: Review Request 34152: Master flag validation now supports zookeeper

2015-05-13 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 13, 2015, 2:08 a.m., Tom Arnfeld wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34152/ --- (Updated

Re: Review Request 31444: Support chrooting in MesosContainerizer launch helper.

2015-05-12 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On May 13, 2015, 12:22 a.m., Ian Downes wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31444

Re: Review Request 33746: Improve logging of clone flags for linux_launcher.

2015-05-12 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 13, 2015, 12:23 a.m., Ian Downes wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33746/ --- (Updated

Re: Review Request 34018: Update existing lambdas to meet style guide

2015-05-17 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On May 17, 2015, 10:11 a.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34018

Re: Review Request 34442: Support manipulating scheduler affinity on Linux.

2015-05-19 Thread Mesos ReviewBot
: src/tests/sched_tests.cpp: patch does not apply Failed to apply patch - Mesos ReviewBot On May 19, 2015, 11:32 p.m., Joris Van Remoortere wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r

Re: Review Request 34317: Updated callers of os::getenv() in /src.

2015-05-17 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On May 17, 2015, 4:54 a.m., Greg Mann wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34317

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

2015-05-17 Thread Mesos ReviewBot
not apply Failed to apply patch - Mesos ReviewBot On May 17, 2015, 1:32 p.m., Greg Mann wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34318

Re: Review Request 34431: Add htb queueing discipline

2015-05-20 Thread Mesos ReviewBot
/tests/routing_tests.cpp: patch does not apply Failed to apply patch - Mesos ReviewBot On May 20, 2015, 4:02 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34431

Re: Review Request 34420: Updated the committer doc regarding checking JIRA before committing.

2015-05-19 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 19, 2015, 5:40 p.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34420/ --- (Updated May 19

Re: Review Request 34416: Add 'path::basename' and 'path::dirname' to stout's user guide

2015-05-19 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 19, 2015, 4:13 p.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34416

Re: Review Request 34483: Moved the repository url constant to reviewboardrc.

2015-05-20 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 20, 2015, 6:04 p.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34483/ --- (Updated May 20

Re: Review Request 34488: Allowed post-reviews.py to configure tracking branch.

2015-05-20 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On May 20, 2015, 6:56 p.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34488

Re: Review Request 34488: Allowed post-reviews.py to configure tracking branch.

2015-05-20 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On May 21, 2015, 12:18 a.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34488

Re: Review Request 34432: Remove duplicate constant string references to mesos-containerizer

2015-05-20 Thread Mesos ReviewBot
: src/tests/routing_tests.cpp: patch does not apply Failed to apply patch - Mesos ReviewBot On May 20, 2015, 9:57 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34432

Re: Review Request 34068: The test case of extend hashmap to support custom equality and hash

2015-05-20 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On May 20, 2015, 7:10 p.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34068

Re: Review Request 33275: Fix capture by reference of temporaries in Stout.

2015-05-20 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On May 20, 2015, 10:36 p.m., Joris Van Remoortere wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33275

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

2015-06-03 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On June 3, 2015, 7:40 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34863

Re: Review Request 34911: Added a test for launching an executor on revocable resources.

2015-06-02 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 1, 2015, 11:15 p.m., Vinod Kone wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34911

Re: Review Request 34721: Added QoS kill executor correction test.

2015-06-04 Thread Mesos ReviewBot
, 34719, 34720, 34721] All tests passed. - Mesos ReviewBot On June 4, 2015, 9:25 p.m., Niklas Nielsen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34721

Re: Review Request 35102: Remove common/lock. Use synchronized instead.

2015-06-04 Thread Mesos ReviewBot
to apply patch - Mesos ReviewBot On June 4, 2015, 10:55 p.m., Joris Van Remoortere wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35102

Re: Review Request 35102: Remove common/lock. Use synchronized instead.

2015-06-04 Thread Mesos ReviewBot
, 35093, 35094, 35095, 35096, 35097, 35098, 35099, 35100, 35101, 35102] Failed command: make -j3 distcheck Error: make dist-gzip am__post_remove_distdir='@:' make[1]: Entering directory `/home/jenkins/jenkins-slave/workspace/mesos-reviewbot' if test -d mesos-0.23.0; then find mesos-0.23.0 -type d

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

2015-06-03 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 3, 2015, 10:53 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 34687: (De)Serializing MasterInfo PB to JSON

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

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

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

Re: Review Request 33159: Pump updateFramework through Allocator from Master.

2015-06-05 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 5, 2015, 8:53 a.m., Joris Van Remoortere wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33159

Re: Review Request 34970: Cleaned up and generalized NoExecutorScheduler to be more configurable.

2015-06-05 Thread Mesos ReviewBot
patch - Mesos ReviewBot On June 5, 2015, 5:52 a.m., Ben Mahler wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34970

Re: Review Request 35102: Remove common/lock. Use synchronized instead.

2015-06-05 Thread Mesos ReviewBot
, 35092, 35093, 35094, 35095, 35096, 35097, 35098, 35099, 35100, 35101, 35102] All tests passed. - Mesos ReviewBot On June 5, 2015, 7:42 a.m., Joris Van Remoortere wrote: --- This is an automatically generated e-mail. To reply, visit

Re: Review Request 35207: Included doxygen documentation in docs/home.md.

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

Re: Review Request 35206: Added mainpage to doxygen documentation.

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

Re: Review Request 35194: Exposing Resources along with ResourceStatistics

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

Re: Review Request 35247: Fixed race between EXPECT_CALL(resourceOffers, _) and driver.start() in fetcher_cache_tests.cpp.

2015-06-09 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 9, 2015, 9:32 a.m., Bernd Mathiske wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35247

Re: Review Request 35234: libprocess: consistent handling of --enable options

2015-06-08 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On June 9, 2015, 12:04 a.m., James Peach wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35234

Re: Review Request 35239: Update the JSON model for Resources to display their revocablility attribute.

2015-06-08 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 9, 2015, 12:38 a.m., Jiang Yan Xu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35239/ --- (Updated

Re: Review Request 35229: Report per-container metrics for network bandwidth throttling to the slave.

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

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

2015-06-08 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 9, 2015, 12:17 a.m., Bartek Plotka wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35157

Re: Review Request 35129: Refactor Future::Data to use ResultT. Remove dynamic allocation.

2015-06-05 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 5, 2015, 1:16 p.m., Joris Van Remoortere wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35129

Re: Review Request 35152: Document and consolidate qdisc handles

2015-06-05 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 5, 2015, 8:38 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35152

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

2015-06-05 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 5, 2015, 11:53 p.m., Bartek Plotka wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35157

Re: Review Request 35152: Document and consolidate qdisc handles

2015-06-05 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 5, 2015, 10:35 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35152

Re: Review Request 35165: Extend fq_codel to allow parent and handle to be specified at runtime.

2015-06-05 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On June 6, 2015, 12:02 a.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35165

Re: Review Request 35182: Fix handling large docker image info on inspect.

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

Re: Review Request 33057: Added secret check to CRAM-MD5 authenticatee.

2015-06-08 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 8, 2015, 9:42 a.m., Till Toenshoff wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33057

Re: Review Request 33296: Added a flag which controls libprocess firewall initialzation.

2015-06-08 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 8, 2015, 12:11 p.m., Alexander Rojas wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33296

Re: Review Request 30032: Added support for cache control in libprocess when dealing with static files.

2015-06-08 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 8, 2015, 3:14 p.m., Alexander Rojas wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30032

Re: Review Request 34980: Pass callback to the resource estimator to retrieve ResourceUsage from resource monitor on demand.

2015-06-03 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 3, 2015, 5:56 a.m., Bartek Plotka wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34980

  1   2   3   4   5   6   7   8   9   10   >