Re: Review Request 37114: MESOS-3187, support docker host command line option

2015-08-07 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 7, 2015, 5:04 a.m., Vaibhav Khanduja wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37114

Re: Review Request 37179: Enhance the bootstrap script

2015-08-06 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 6, 2015, 9:37 a.m., Zhiwei Chen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37179/ --- (Updated

Re: Review Request 37065: Fixed MemIsolatorTest failure on OSX.

2015-08-06 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 6, 2015, 5:54 a.m., Artem Harutyunyan wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37065

Re: Review Request 37189: Added std::hash template specializations.

2015-08-07 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On Aug. 7, 2015, 10:28 a.m., Jan Schlicht wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37189

Re: Review Request 37208: Fix the spell error in help message of slave component.

2015-08-07 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 7, 2015, 8:12 a.m., Yong Qiao Wang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37208

Re: Review Request 37065: Fixed MemIsolatorTest failure on OSX.

2015-08-06 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 6, 2015, 5:34 p.m., Artem Harutyunyan wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37065

Re: Review Request 37082: Tests for subscribe/failover functionality for http based framework

2015-08-06 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On Aug. 6, 2015, 4:26 p.m., Anand Mazumdar wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37082

Re: Review Request 37192: More basic call validation tests for http api

2015-08-06 Thread Mesos ReviewBot
] All tests passed. - Mesos ReviewBot On Aug. 6, 2015, 5:06 p.m., Anand Mazumdar wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37192

Re: Review Request 37075: Protobuf definitions instructing the fetcher cache about checksums and their validation.

2015-08-04 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 4, 2015, 1:41 p.m., Bernd Mathiske wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37075

Re: Review Request 36910: Patch configure.ac to include $LIBS in the CRAM-MD5 check

2015-08-04 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 4, 2015, 1:49 p.m., Chris Heller wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36910/ --- (Updated

Re: Review Request 36819: Use setup.py in python cli package.

2015-08-07 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 7, 2015, 3:08 p.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36819

Re: Review Request 36811: Don't check protobuf jar when --disable-java flag.

2015-08-07 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On Aug. 7, 2015, 3:21 p.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36811

Re: Review Request 36816: Support HTTP checks in Mesos health check program

2015-08-07 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 7, 2015, 3:35 p.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36816

Re: Review Request 35711: Disallow special characters in role name.

2015-08-07 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.24.0; then find mesos-0.24.0 -type d ! -perm -200 -exec chmod u+w {} ';' rm -rf mesos-0.24.0 || { sleep 5 rm -rf mesos-0.24.0

Re: Review Request 37228: Updated slave task label decorator hook to pass in ExecutorInfo.

2015-08-07 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 7, 2015, 6:51 p.m., Kapil Arya wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37228/ --- (Updated

Re: Review Request 36431: Show stdout/stderr when using mesos-execute.

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

Re: Review Request 37189: Added std::hash template specializations.

2015-08-07 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On Aug. 7, 2015, 11:52 a.m., Jan Schlicht wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37189

Re: Review Request 36847: Added HTTP Delete Method.

2015-08-13 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 13, 2015, 4:19 p.m., Joerg Schad wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36847/ --- (Updated

Re: Review Request 37415: Fix flaky ExamplesTest.JavaFramework

2015-08-13 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On Aug. 13, 2015, 4:49 p.m., Greg Mann wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37415

Re: Review Request 37257: Add SUPPRESS call interface to the scheduler

2015-08-13 Thread Mesos ReviewBot
/apply-review.sh -n -r 37257 Error: 2015-08-13 23:39:06 URL:https://reviews.apache.org/r/37257/diff/raw/ [5394/5394] - 37257.patch [1] error: patch failed: src/tests/scheduler_tests.cpp:903 error: src/tests/scheduler_tests.cpp: patch does not apply Failed to apply patch - Mesos ReviewBot On Aug

Re: Review Request 37443: Add labels to FrameworkInfo (v2)

2015-08-13 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 13, 2015, 8:20 p.m., James DeFelice wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37443

Re: Review Request 37442: Factor out the token extraction rules in prepartion for extending them to cope with multiple versions.

2015-08-13 Thread Mesos ReviewBot
, 37442] All tests passed. - Mesos ReviewBot On Aug. 13, 2015, 9:59 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37442

Re: Review Request 37457: Add --ip_discovery_command to Agent.

2015-08-13 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On Aug. 14, 2015, 12:21 a.m., Marco Massenzio wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37457

Re: Review Request 37445: Fix typos in style guide.

2015-08-13 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 13, 2015, 9:58 p.m., Neil Conway wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37445/ --- (Updated

Re: Review Request 37468: Removed allocation types to mesos::master namespace

2015-08-13 Thread Mesos ReviewBot
) not allowed Failed to commit patch - Mesos ReviewBot On Aug. 14, 2015, 2:01 a.m., José Guilherme Vanz wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37468

Re: Review Request 37475: Fixed a wrong comparison.

2015-08-14 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 14, 2015, 10:21 a.m., Jan Schlicht wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37475

Re: Review Request 37466: Update perf tests to including testing the supported perf output formats.

2015-08-14 Thread Mesos ReviewBot
, 37442, 37460, 37462, 37466] All tests passed. - Mesos ReviewBot On Aug. 14, 2015, 3:39 a.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37466

Re: Review Request 37414: Fix flaky ExamplesTest.JavaLog

2015-08-14 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 13, 2015, 4:49 p.m., Greg Mann wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37414/ --- (Updated

Re: Review Request 37415: Fix flaky ExamplesTest.JavaFramework

2015-08-14 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 14, 2015, 4:56 a.m., Greg Mann wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37415/ --- (Updated

Re: Review Request 37268: Style checker checking for { on newline

2015-08-09 Thread Mesos ReviewBot
://reviews.apache.org/r/37266 fatal: empty ident name (for guilherme@gmail.com) not allowed Failed to commit patch - Mesos ReviewBot On Aug. 9, 2015, 4:57 a.m., José Guilherme Vanz wrote: --- This is an automatically generated e-mail

Re: Review Request 37189: Added std::hash template specializations.

2015-08-10 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On Aug. 10, 2015, 12:26 p.m., Jan Schlicht wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37189

Re: Review Request 37311: Implemented a 'read-only' Appc image store.

2015-08-10 Thread Mesos ReviewBot
, 37311] All tests passed. - Mesos ReviewBot On Aug. 10, 2015, 7:19 p.m., Jiang Yan Xu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37311

Re: Review Request 37334: Added a persistent volume test for linux filesystem isolator.

2015-08-10 Thread Mesos ReviewBot
, 37054, 37055, 37091, 37105, 37142, 37159, 37194, 37236, 37237, 37322, 37330, 37334] All tests passed. - Mesos ReviewBot On Aug. 11, 2015, 1:58 a.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request 37247: Added Docker image reference store.

2015-08-10 Thread Mesos ReviewBot
: patch does not apply Failed to apply patch - Mesos ReviewBot On Aug. 10, 2015, 10:53 p.m., Lily Chen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37247

Re: Review Request 37328: Remove namespace ambiguity

2015-08-10 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On Aug. 11, 2015, 12:17 a.m., Anand Mazumdar wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37328

Re: Review Request 37488: Fix cgroup CHECK failure cases test harness to abort.

2015-08-14 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 14, 2015, 5:41 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37488/ --- (Updated

Re: Review Request 37427: Docker registry: adding TokenManager.

2015-08-14 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On Aug. 14, 2015, 5:28 p.m., Jojy Varghese wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37427

Re: Review Request 37479: Move QoS plug-ins to a specified folder like resource_estimator

2015-08-14 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.24.0; then find mesos-0.24.0 -type d ! -perm -200 -exec chmod u+w {} ';' rm -rf mesos-0.24.0 || { sleep 5 rm -rf mesos-0.24.0

Re: Review Request 37489: Make scheduler tests parameterized

2015-08-14 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 14, 2015, 8:23 p.m., Anand Mazumdar wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37489

Re: Review Request 37491: Added a trivial test on how the protobuf-json string don't match due to incorrect escaping of bytes

2015-08-14 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 14, 2015, 8:19 p.m., Anand Mazumdar wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37491

Re: Review Request 37490: Convert CHECK functions to ADD_FAILURE in mesos.cpp to fail the test without aborting the test harness.

2015-08-14 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 14, 2015, 8:05 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37490/ --- (Updated

Re: Review Request 37382: Introduced provisioner Backend interface.

2015-08-14 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 14, 2015, 5:51 p.m., Jiang Yan Xu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37382/ --- (Updated

Re: Review Request 37500: Update the FrameworkInfo.user on scheduler failover

2015-08-15 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 15, 2015, 8:03 a.m., Aditi Dixit wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37500/ --- (Updated

Re: Review Request 37501: Add support for removing capabilities.

2015-08-15 Thread Mesos ReviewBot
/apply-review.sh -n -r 37501 Error: 2015-08-15 11:08:50 URL:https://reviews.apache.org/r/37501/diff/raw/ [4846/4846] - 37501.patch [1] error: patch failed: src/master/master.hpp:1519 error: src/master/master.hpp: patch does not apply Failed to apply patch - Mesos ReviewBot On Aug. 15, 2015, 10:50

Re: Review Request 37493: Detect gflags when present and link when building Python module

2015-08-14 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 15, 2015, 12:28 a.m., Greg Mann wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37493/ --- (Updated

Re: Review Request 37479: Move QoS plug-ins to a specified folder like resource_estimator

2015-08-14 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 15, 2015, 1:44 a.m., Guangya Liu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37479/ --- (Updated

Re: Review Request 37494: Fixed JSON wrapper to properly encode bytes.

2015-08-14 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 14, 2015, 11:24 p.m., Vinod Kone wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37494/ --- (Updated

Re: Review Request 37502: Add CMAKE_CXX_FLAGS to GMOCK_CONFIG_CMD in CMake.

2015-08-15 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 15, 2015, 12:07 p.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37502

Re: Review Request 37247: Added Docker image reference store.

2015-08-11 Thread Mesos ReviewBot
- Mesos ReviewBot On Aug. 11, 2015, 11:34 p.m., Lily Chen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37247/ --- (Updated Aug

Re: Review Request 37382: Introduced provisioner Backend interface.

2015-08-12 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 12, 2015, 5:49 a.m., Jiang Yan Xu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37382/ --- (Updated

Re: Review Request 37336: Added `wait()` method to process::Subprocess

2015-08-11 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 11, 2015, 11:36 p.m., Marco Massenzio wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37336

Re: Review Request 37192: More basic call validation tests for http api

2015-08-11 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On Aug. 11, 2015, 10:58 p.m., Anand Mazumdar wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37192

Re: Review Request 37377: Disallow HTTP schedulers when authentication is required.

2015-08-11 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 11, 2015, 11:52 p.m., Ben Mahler wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37377/ --- (Updated

Re: Review Request 37378: Updated http api tests to use V1 Protobufs

2015-08-11 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On Aug. 12, 2015, 12:14 a.m., Anand Mazumdar wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37378

Re: Review Request 37399: Ignore no statistics condition for containers with no qdisc.

2015-08-12 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 12, 2015, 4:57 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37399/ --- (Updated

Re: Review Request 37275: [3/3]Generate make batch file to build project in windows.

2015-08-09 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On Aug. 9, 2015, 6:21 p.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37275

Re: Review Request 37277: (WIP) Added Heartbeater to master to send periodic heartbeats to HTTP schedulers.

2015-08-09 Thread Mesos ReviewBot
/apply-review.sh -n -r 37277 Error: 2015-08-10 02:30:31 URL:https://reviews.apache.org/r/37277/diff/raw/ [5524/5524] - 37277.patch [1] error: patch failed: src/master/master.hpp:1628 error: src/master/master.hpp: patch does not apply Failed to apply patch - Mesos ReviewBot On Aug. 9, 2015, 8:31

Re: Review Request 37260: mesos: Fixed break before braces style for 'enum'.

2015-08-09 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On Aug. 10, 2015, 1 a.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37260

Re: Review Request 37284: Maintenance Primitives: Added support for Accept / Decline of InverseOffers in master.

2015-08-10 Thread Mesos ReviewBot
/registrar.hpp:49 error: src/master/registrar.hpp: patch does not apply Failed to apply patch - Mesos ReviewBot On Aug. 10, 2015, 7:06 a.m., Joris Van Remoortere wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request 37241: Renamed master's /call endpoint to /api/v1/scheduler.

2015-08-07 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On Aug. 7, 2015, 11:44 p.m., Vinod Kone wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37241

Re: Review Request 37237: Added a few MesosContainerizer filesystem tests to test the linux filesystem isolator.

2015-08-07 Thread Mesos ReviewBot
, 37054, 37055, 37091, 37105, 37142, 37159, 37194, 37236, 37237] All tests passed. - Mesos ReviewBot On Aug. 8, 2015, 12:32 a.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r

Re: Review Request 37244: Fixed scheduler message metrics to work with scheduler Calls.

2015-08-07 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 8, 2015, 12:53 a.m., Vinod Kone wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37244/ --- (Updated

Re: Review Request 37023: Add an endpoint that exposes component flags.

2015-08-07 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 8, 2015, 3:50 a.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37023

Re: Review Request 37252: Fix typos in master endpoint help messages.

2015-08-07 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 8, 2015, 4:14 a.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37252

Re: Review Request 37244: Fixed scheduler message metrics to work with scheduler Calls.

2015-08-08 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On Aug. 8, 2015, 3:32 p.m., Vinod Kone wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37244

Re: Review Request 37018: [3/3] Removed whitespace padding in overloaded operators (stout).

2015-08-08 Thread Mesos ReviewBot
/version.hpp: patch does not apply Failed to apply patch - Mesos ReviewBot On Aug. 8, 2015, 9:57 p.m., Artem Harutyunyan wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37018

Re: Review Request 37263: Add support for non-subscribe HTTP calls in the master.

2015-08-08 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 9, 2015, 1:32 a.m., Vinod Kone wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37263/ --- (Updated

Re: Review Request 37160: Add GTEST_LANG_CXX11 to StoutTestsConfigure when compile.

2015-08-06 Thread Mesos ReviewBot
. - Mesos ReviewBot On Aug. 6, 2015, 5:13 a.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37160

Re: Review Request 37165: Introduced v1 API.

2015-08-06 Thread Mesos ReviewBot
/apply-review.sh -n -r 37165 Error: 2015-08-06 09:27:43 URL:https://reviews.apache.org/r/37165/diff/raw/ [253063/253063] - 37165.patch [1] error: patch failed: src/tests/scheduler_tests.cpp:275 error: src/tests/scheduler_tests.cpp: patch does not apply Failed to apply patch - Mesos ReviewBot

Re: Review Request 37142: Removed the unneeded ExecutorInfo from Container struct in MesosContainerizer.

2015-08-05 Thread Mesos ReviewBot
, 37054, 37055, 37091, 37105, 37142] All tests passed. - Mesos ReviewBot On Aug. 5, 2015, 9:22 p.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37142

Re: Review Request 37147: Timeout perf stat command if it does not complete.

2015-08-05 Thread Mesos ReviewBot
/apply-review.sh -n -r 37147 Error: 2015-08-06 00:09:00 URL:https://reviews.apache.org/r/37147/diff/raw/ [1484/1484] - 37147.patch [1] error: patch failed: src/linux/perf.cpp:296 error: src/linux/perf.cpp: patch does not apply Failed to apply patch - Mesos ReviewBot On Aug. 5, 2015, 11:25 p.m

Re: Review Request 36773: Implemented a LRU entry selection criteria for cache eviction.

2015-07-27 Thread Mesos ReviewBot
. - Mesos ReviewBot On July 27, 2015, 3:38 p.m., Jan Schlicht wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36773/ --- (Updated

Re: Review Request 36844: Libprocess: Used THREAD_LOCAL to replace ThreadLocal.

2015-07-27 Thread Mesos ReviewBot
, 36826, 36827, 36828, 36829, 36830, 36842, 36845, 36844] All tests passed. - Mesos ReviewBot On July 27, 2015, 9:15 p.m., Joris Van Remoortere wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request 36862: Added metric for number of preempted executors.

2015-07-27 Thread Mesos ReviewBot
. - Mesos ReviewBot On July 27, 2015, 10:30 p.m., Niklas Nielsen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36862

Re: Review Request 36870: Added - operator for Future.

2015-07-27 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On July 28, 2015, 12:56 a.m., Ben Mahler wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36870

Re: Review Request 36867: Add labels to FrameworkInfo.

2015-07-27 Thread Mesos ReviewBot
/apply-review.sh -n -r 36867 Error: 2015-07-28 00:55:40 URL:https://reviews.apache.org/r/36867/diff/raw/ [5396/5396] - 36867.patch [1] error: patch failed: src/master/master.hpp:1411 error: src/master/master.hpp: patch does not apply Failed to apply patch - Mesos ReviewBot On July 28, 2015, 12:04

Re: Review Request 36867: Add labels to FrameworkInfo.

2015-07-27 Thread Mesos ReviewBot
. - Mesos ReviewBot On July 28, 2015, 1:25 a.m., Neil Conway wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36867/ --- (Updated

Re: Review Request 36742: Fixed the sanity check on OSX for persistent volumes.

2015-07-23 Thread Mesos ReviewBot
. - Mesos ReviewBot On July 23, 2015, 7:37 p.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36742/ --- (Updated July

Re: Review Request 36718: Converted Limitation and ExecutorRunState structs into protobufs.

2015-07-23 Thread Mesos ReviewBot
. - Mesos ReviewBot On July 23, 2015, 11:24 p.m., Kapil Arya wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36718/ --- (Updated

Re: Review Request 36698: Updated scheduler library to send REQUEST call.

2015-07-22 Thread Mesos ReviewBot
. - Mesos ReviewBot On July 22, 2015, 6:16 p.m., Vinod Kone wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36698/ --- (Updated

Re: Review Request 36617: Improved task reconciliation documentation.

2015-07-23 Thread Mesos ReviewBot
. - Mesos ReviewBot On July 23, 2015, 12:01 p.m., Jan Schlicht wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36617

Re: Review Request 36763: Made the Isolator interface not dependent on IsolatorProcess.

2015-07-24 Thread Mesos ReviewBot
. - Mesos ReviewBot On July 24, 2015, 5:10 a.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36763/ --- (Updated July

Re: Review Request 36755: Passed env vars returned by Isolator::prepare() to executor.

2015-07-23 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On July 24, 2015, 12:20 a.m., Kapil Arya wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36755

Re: Review Request 36663: Added ip_address field to MasterInfo

2015-07-23 Thread Mesos ReviewBot
. - Mesos ReviewBot On July 24, 2015, 12:17 a.m., Marco Massenzio wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36663

Re: Review Request 36757: Added https support.

2015-07-23 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On July 24, 2015, 2:28 a.m., Jojy Varghese wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36757

Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-07-24 Thread Mesos ReviewBot
: 3rdparty/libprocess/3rdparty/CMakeLists.txt 3rdparty/libprocess/cmake/ProcessConfigure.cmake 3rdparty/libprocess/cmake/ProcessTestsConfigure.cmake 3rdparty/libprocess/include/process/http.hpp 3rdparty/libprocess/src/tests/CMakeLists.txt Failed to commit patch - Mesos ReviewBot On July 24

Re: Review Request 36501: MESOS-3023

2015-07-24 Thread Mesos ReviewBot
. - Mesos ReviewBot On July 24, 2015, 9:57 a.m., Klaus Ma wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36501/ --- (Updated

Re: Review Request 36715: Used std::thread instead of pthread for Libprocess process.

2015-07-23 Thread Mesos ReviewBot
, 36676, 36705, 36715] All tests passed. - Mesos ReviewBot On July 23, 2015, 7:49 p.m., Joris Van Remoortere wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36715

Re: Review Request 36799: Renamed Executor{Limitation, RunState} to Container{Limitation, State}.

2015-07-26 Thread Mesos ReviewBot
] All tests passed. - Mesos ReviewBot On July 26, 2015, 6:49 p.m., Kapil Arya wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36799

Re: Review Request 36821: Fix disable endpoints rule fails to recognize HTTP path delegates.

2015-07-26 Thread Mesos ReviewBot
. - Mesos ReviewBot On July 26, 2015, 6:14 p.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36821

Re: Review Request 36658: Updated SSL documentation. Added to home page.

2015-07-26 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On July 26, 2015, 6:01 p.m., Joris Van Remoortere wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36658

Re: Review Request 36757: Added https support.

2015-07-24 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On July 24, 2015, 3:23 p.m., Jojy Varghese wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36757

Re: Review Request 36837: Update gmock to 1.7.0 .

2015-07-26 Thread Mesos ReviewBot
-1.7.0.tar.gz' error: 3rdparty/libprocess/3rdparty/gmock-1.7.0.tar.gz: patch does not apply Failed to apply patch - Mesos ReviewBot On July 27, 2015, 3:36 a.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply

Re: Review Request 36837: Update gmock to 1.7.0 .

2015-07-26 Thread Mesos ReviewBot
-1.7.0.tar.gz' error: 3rdparty/libprocess/3rdparty/gmock-1.7.0.tar.gz: patch does not apply Failed to apply patch - Mesos ReviewBot On July 27, 2015, 4:03 a.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply

Re: Review Request 36831: Allowed Isolator::prepare to return a list of CommandInfos.

2015-07-26 Thread Mesos ReviewBot
, 36831] All tests passed. - Mesos ReviewBot On July 27, 2015, 12:04 a.m., Kapil Arya wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36831

Re: Review Request 36830: Libprocess: Introduce configurable `UNIMPLEMENTED` macro.

2015-07-26 Thread Mesos ReviewBot
, 36826, 36827, 36828, 36829, 36830] All tests passed. - Mesos ReviewBot On July 26, 2015, 9:01 p.m., Joris Van Remoortere wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36830

Re: Review Request 36819: Use setup.py in python cli package.

2015-07-26 Thread Mesos ReviewBot
. - Mesos ReviewBot On July 26, 2015, 2:47 p.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36819

Re: Review Request 36820: Fixed clang warning [-Wunevaluated-expression] in process tests.

2015-07-26 Thread Mesos ReviewBot
. - Mesos ReviewBot On July 26, 2015, 4:51 p.m., Joris Van Remoortere wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36820

Re: Review Request 36839: Fix typos in oversubscription doc.

2015-07-27 Thread Mesos ReviewBot
. - Mesos ReviewBot On July 27, 2015, 6:09 a.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36839

Re: Review Request 36841: Changed comparison functions for floating-point numbers

2015-07-27 Thread Mesos ReviewBot
/apply-review.sh -n -r 36840 Error: 2015-07-27 07:38:27 URL:https://reviews.apache.org/r/36840/diff/raw/ [3155/3155] - 36840.patch [1] error: src/tests/perf_tests.cpp: does not exist in index Failed to apply patch - Mesos ReviewBot On July 27, 2015, 6:48 a.m., Jihun Kang wrote

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