Re: Review Request 44087: Moved logic to assign process to freezer hierarchy into parentHook.

2016-03-03 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44087/ --- (Updated March 3, 2016, 3:39 p.m.) Review request for mesos and Joris Van

Re: Review Request 43884: Added allocator metrics for used quotas.

2016-03-03 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43884/ --- (Updated March 3, 2016, 5:17 p.m.) Review request for mesos, Alexander

Re: Review Request 43883: Added allocator metrics for the number of offer filters per framework.

2016-03-03 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43883/ --- (Updated March 3, 2016, 5:17 p.m.) Review request for mesos, Alexander

Re: Review Request 44261: Introduced a RAII helper for process::metrics::Timer.

2016-03-03 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44261/ --- (Updated March 3, 2016, 5:17 p.m.) Review request for mesos, Alexander

Re: Review Request 44260: Moved metrics of the hierarchical allocator to its own file.

2016-03-03 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44260/ --- (Updated March 3, 2016, 5:17 p.m.) Review request for mesos, Alexander

Re: Review Request 43881: Added allocator metric for the number of allocations to a framework.

2016-03-03 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43881/ --- (Updated March 3, 2016, 5:17 p.m.) Review request for mesos, Alexander

Re: Review Request 43879: Added allocator metrics for number of allocations made.

2016-03-03 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43879/ --- (Updated March 3, 2016, 5:17 p.m.) Review request for mesos, Alexander

Re: Review Request 43882: Added allocation metrics for allocation time.

2016-03-03 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43882/ --- (Updated March 3, 2016, 5:17 p.m.) Review request for mesos, Alexander

Re: Review Request 43880: Added allocator metrics for total and allocated scalar resources.

2016-03-03 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43880/ --- (Updated March 3, 2016, 5:17 p.m.) Review request for mesos, Alexander

Re: Review Request 44260: Moved metrics of the hierarchical allocator to its own file.

2016-03-03 Thread Benjamin Bannier
> On March 2, 2016, 2:58 p.m., Alexander Rukletsov wrote: > > src/master/allocator/mesos/metrics.hpp, line 37 > > > > > > You don't really need to call `.self()` here, there exists an `defer` > > override taking

Re: Review Request 44200: Add agent flags for specifying CNI plugin and config directories.

2016-03-03 Thread Avinash sridharan
> On March 1, 2016, 5:03 p.m., Gilbert Song wrote: > > src/slave/flags.cpp, lines 698-705 > > > > > > While users can config the CNI network form a JSON file, is it possible > > to provide an option to define the

Review Request 44343: Used single space in license check error message.

2016-03-03 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44343/ --- Review request for mesos and Till Toenshoff. Repository: mesos Description

Re: Review Request 44269: Added the framework of 'network/cni' isolator.

2016-03-03 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44269/#review121885 --- src/slave/containerizer/mesos/isolators/network/cni.cpp (lines

Re: Review Request 44289: Added support for contender and detector modules.

2016-03-03 Thread Anurag Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44289/ --- (Updated March 3, 2016, 5:29 p.m.) Review request for mesos and Benjamin

Re: Review Request 44288: Changed MasterDetector/Contender namespace.

2016-03-03 Thread Anurag Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44288/ --- (Updated March 3, 2016, 5:30 p.m.) Review request for mesos and Benjamin

Re: Review Request 44287: Added MasterContender and MasterDetector abstract classes.

2016-03-03 Thread Anurag Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44287/ --- (Updated March 3, 2016, 5:29 p.m.) Review request for mesos and Benjamin

Re: Review Request 44342: XFS disk resource isolator.

2016-03-03 Thread James Peach
> On March 3, 2016, 4:58 p.m., Jie Yu wrote: > > Can you rebase? Looks like the diff is not correct. Bleh. Sorry I didn't notice that :-/ - James --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 44342: XFS disk resource isolator.

2016-03-03 Thread James Peach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44342/ --- (Updated March 3, 2016, 5 p.m.) Review request for mesos, Jie Yu and Jiang Yan

Re: Review Request 44278: Added HTTP scheduler stream IDs.

2016-03-03 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44278/ --- (Updated March 3, 2016, 5:13 p.m.) Review request for mesos, Anand Mazumdar

Re: Review Request 44342: XFS disk resource isolator.

2016-03-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44342/#review121889 --- src/slave/containerizer/mesos/containerizer.cpp (line 216)

Re: Review Request 44269: Added the framework of 'network/cni' isolator.

2016-03-03 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44269/#review121884 --- src/slave/containerizer/mesos/isolators/network/cni.cpp (line

Re: Review Request 44290: Added tests involving HTTP scheduler stream IDs.

2016-03-03 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44290/ --- (Updated March 3, 2016, 5:15 p.m.) Review request for mesos, Anand Mazumdar

Review Request 44345: Removing references to ShutdownFramework acl.

2016-03-03 Thread Srinivas Brahmaroutu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44345/ --- Review request for mesos. Repository: mesos Description --- Removing

Re: Review Request 44339: Disabled Appc simple fetch test.

2016-03-03 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44339/#review121871 --- Isn't there already a CurlFilter for tests? - Joerg Schad On

Re: Review Request 39939: Make docker_socket option support different protocols.

2016-03-03 Thread haosdent huang
> On Feb. 22, 2016, 4:41 a.m., Klaus Ma wrote: > > src/slave/flags.cpp, line 437 > > > > > > I think we need to handle feature interaction with > > `docker_mesos_image`. Currently, we mount unix socket into

Re: Review Request 39939: Make docker_socket option support different protocols.

2016-03-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39939/ --- (Updated March 3, 2016, 4:40 p.m.) Review request for mesos, Klaus Ma and

Review Request 44342: XFS disk resource isolator.

2016-03-03 Thread James Peach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44342/ --- Review request for mesos, Jie Yu and Jiang Yan Xu. Bugs: MESOS-4828

Re: Review Request 44279: Added stream IDs to the HTTP API docs.

2016-03-03 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44279/ --- (Updated March 3, 2016, 5:16 p.m.) Review request for mesos, Anand Mazumdar

Re: Review Request 39939: Make docker_socket option support different protocols.

2016-03-03 Thread haosdent huang
> On Feb. 22, 2016, 4:41 a.m., Klaus Ma wrote: > > src/slave/flags.cpp, line 437 > > > > > > I think we need to handle feature interaction with > > `docker_mesos_image`. Currently, we mount unix socket into

Re: Review Request 44200: Add agent flags for specifying CNI plugin and config directories.

2016-03-03 Thread Avinash sridharan
> On March 2, 2016, 4:07 p.m., Avinash sridharan wrote: > > docs/endpoints/slave/state.json.md, line 91 > > > > > > Do we need to change some install scripts to create these paths? I > > think we should leave the

Re: Review Request 44186: Added authentication to master endpoints.

2016-03-03 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44186/ --- (Updated March 3, 2016, 4:53 p.m.) Review request for mesos, Adam B and

Re: Review Request 44342: XFS disk resource isolator.

2016-03-03 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44342/#review121878 --- Can you rebase? Looks like the diff is not correct. - Jie Yu

Re: Review Request 44339: Disabled Appc simple fetch test.

2016-03-03 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44339/#review121883 --- I commited a fix using the CURL test filter. - Jie Yu On March

Re: Review Request 44342: XFS disk resource isolator.

2016-03-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44342/#review121888 --- src/Makefile.am (line 869)

Re: Review Request 43806: Add comments for rebalance.

2016-03-03 Thread Yongqiao Wang
> On March 2, 2016, 2:07 p.m., Alexander Rukletsov wrote: > > src/master/allocator/mesos/hierarchical.cpp, lines 1066-1069 > > > > > > For posterity, we backtick types and variables names. > > Yongqiao Wang wrote:

Re: Review Request 43824: Addressed comments of 41672.

2016-03-03 Thread Alexander Rukletsov
> On March 3, 2016, 9:03 a.m., Alexander Rukletsov wrote: > > src/tests/hierarchical_allocator_tests.cpp, lines 2635-2640 > > > > > > Does it fit one line? I know we are a bit inconsistent about it, but > > let's

Re: Review Request 44186: Added authentication to master endpoints.

2016-03-03 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44186/#review121833 --- Bad patch! Reviews applied: [44186, 44286] Failed command:

Review Request 44335: Moved variable declarations closer to where they are used.

2016-03-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44335/ --- Review request for mesos, Bernd Mathiske and Ben Mahler. Repository: mesos

Review Request 44333: Corrected a typo in the allocator tests.

2016-03-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44333/ --- Review request for mesos and Ben Mahler. Repository: mesos Description

Review Request 44334: Cleaned up empty hashmaps from allocator tests.

2016-03-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44334/ --- Review request for mesos, Bernd Mathiske and Ben Mahler. Repository: mesos

Re: Review Request 44260: Moved metrics of the hierarchical allocator to its own file.

2016-03-03 Thread Alexander Rukletsov
> On March 2, 2016, 1:58 p.m., Alexander Rukletsov wrote: > > src/master/allocator/mesos/metrics.hpp, line 37 > > > > > > You don't really need to call `.self()` here, there exists an `defer` > > override taking

Re: Review Request 43824: Addressed comments of 41672.

2016-03-03 Thread Yongqiao Wang
> On March 3, 2016, 9:03 a.m., Alexander Rukletsov wrote: > > src/tests/hierarchical_allocator_tests.cpp, lines 2607-2609 > > > > > > I'm an ESL, but having both "per weight" and "by weight" sounds a bit > >

Re: Review Request 43882: Added allocation metrics for allocation time.

2016-03-03 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43882/ --- (Updated March 3, 2016, 2:32 p.m.) Review request for mesos, Alexander

Re: Review Request 43806: Add comments for rebalance.

2016-03-03 Thread Yongqiao Wang
> On March 2, 2016, 2:07 p.m., Alexander Rukletsov wrote: > > src/master/allocator/mesos/hierarchical.cpp, lines 1066-1069 > > > > > > For posterity, we backtick types and variables names. Do you mean we should

Re: Review Request 44260: Moved metrics of the hierarchical allocator to its own file.

2016-03-03 Thread Benjamin Bannier
> On March 2, 2016, 2:58 p.m., Alexander Rukletsov wrote: > > src/master/allocator/mesos/metrics.hpp, line 37 > > > > > > You don't really need to call `.self()` here, there exists an `defer` > > override taking

Re: Review Request 44260: Moved metrics of the hierarchical allocator to its own file.

2016-03-03 Thread Benjamin Bannier
> On March 2, 2016, 12:37 p.m., Alexander Rukletsov wrote: > > src/master/allocator/mesos/metrics.hpp, line 23 > > > > > > s/ > s/>/" > > > > We use <> only for "library" or public includes: libprocess,

Re: Review Request 43613: Refactor cluster test helpers into self-contained objects.

2016-03-03 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43613/#review121835 --- src/tests/cluster.hpp (line 168)

Re: Review Request 43614: Refactor MesosTest and remove cleanup logic.

2016-03-03 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43614/#review121855 --- LGTM, pending dependencies. - Bernd Mathiske On March 2, 2016,

Review Request 44332: Fixed a typo in the HierarchicalAllocatorTest.CoarseGrained test.

2016-03-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44332/ --- Review request for mesos, Ben Mahler and Joris Van Remoortere. Repository:

Review Request 44331: Added a test to ensure the allocator does not double account resources.

2016-03-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44331/ --- Review request for mesos and Ben Mahler. Repository: mesos Description

Re: Review Request 43138: Updated a comment for oversubscribed resources for clarity.

2016-03-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43138/ --- (Updated March 3, 2016, 1 p.m.) Review request for mesos and Ben Mahler.

Re: Review Request 43105: Ensured the allocator does not double account resources.

2016-03-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43105/ --- (Updated March 3, 2016, 1 p.m.) Review request for mesos and Ben Mahler.

Re: Review Request 43880: Added allocator metrics for total and allocated scalar resources.

2016-03-03 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43880/ --- (Updated March 3, 2016, 2:12 p.m.) Review request for mesos, Alexander

Re: Review Request 44260: Moved metrics of the hierarchical allocator to its own file.

2016-03-03 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44260/ --- (Updated March 3, 2016, 2:11 p.m.) Review request for mesos, Alexander

Re: Review Request 44261: Introduced a RAII helper for process::metrics::Timer.

2016-03-03 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44261/ --- (Updated March 3, 2016, 2:11 p.m.) Review request for mesos, Alexander

Re: Review Request 44260: Moved metrics of the hierarchical allocator to its own file.

2016-03-03 Thread Alexander Rukletsov
> On March 2, 2016, 11:37 a.m., Alexander Rukletsov wrote: > > src/master/allocator/mesos/metrics.hpp, line 23 > > > > > > s/ > s/>/" > > > > We use <> only for "library" or public includes: libprocess,

Re: Review Request 43883: Added allocator metrics for the number of offer filters per framework.

2016-03-03 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43883/ --- (Updated March 3, 2016, 2:12 p.m.) Review request for mesos, Alexander

Re: Review Request 43881: Added allocator metric for the number of allocations to a framework.

2016-03-03 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43881/ --- (Updated March 3, 2016, 2:11 p.m.) Review request for mesos, Alexander

Re: Review Request 43879: Added allocator metrics for number of allocations made.

2016-03-03 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43879/ --- (Updated March 3, 2016, 2:12 p.m.) Review request for mesos, Alexander

Re: Review Request 43806: Add comments for rebalance.

2016-03-03 Thread Alexander Rukletsov
> On March 2, 2016, 2:07 p.m., Alexander Rukletsov wrote: > > src/master/allocator/mesos/hierarchical.cpp, lines 1066-1069 > > > > > > For posterity, we backtick types and variables names. > > Yongqiao Wang wrote:

Review Request 44336: Removed numeric suffixes where appropriate in allocator tests.

2016-03-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44336/ --- Review request for mesos, Bernd Mathiske and Ben Mahler. Repository: mesos

Re: Review Request 44260: Moved metrics of the hierarchical allocator to its own file.

2016-03-03 Thread Alexander Rukletsov
> On March 2, 2016, 11:37 a.m., Alexander Rukletsov wrote: > > src/master/allocator/mesos/metrics.hpp, line 43 > > > > > > Is it what clnag-format suggest? I doubt it : ) > > Benjamin Bannier wrote: > Yes,

Re: Review Request 43884: Added allocator metrics for used quotas.

2016-03-03 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43884/ --- (Updated March 3, 2016, 2:11 p.m.) Review request for mesos, Alexander

Re: Review Request 43824: Addressed comments of 41672.

2016-03-03 Thread Yongqiao Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43824/ --- (Updated March 3, 2016, 1:28 p.m.) Review request for mesos, Adam B and

Re: Review Request 43629: Especially updated tests to use the updated MesosTest helpers.

2016-03-03 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43629/#review121854 --- src/tests/slave_recovery_tests.cpp (line 3342)

Re: Review Request 44260: Moved metrics of the hierarchical allocator to its own file.

2016-03-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44260/#review121840 --- src/Makefile.am (line 632)

Re: Review Request 43824: Addressed comments of 41672.

2016-03-03 Thread Yongqiao Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43824/ --- (Updated March 3, 2016, 2:05 p.m.) Review request for mesos, Adam B and

Re: Review Request 43879: Added allocator metrics for number of allocations made.

2016-03-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43879/#review121856 --- docs/monitoring.md (line 872)

Re: Review Request 43880: Added allocator metrics for total and allocated scalar resources.

2016-03-03 Thread Alexander Rukletsov
> On March 1, 2016, 11:48 a.m., Alexander Rukletsov wrote: > > src/tests/hierarchical_allocator_tests.cpp, line 2406 > > > > > > I believe the actual value in JSON is double, right? > > Benjamin Bannier wrote: >

Re: Review Request 44269: Added the framework of 'network/cni' isolator.

2016-03-03 Thread Qian Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44269/ --- (Updated March 3, 2016, 9:57 p.m.) Review request for mesos, Avinash

Re: Review Request 43880: Added allocator metrics for total and allocated scalar resources.

2016-03-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43880/#review121860 --- docs/monitoring.md (line 879)

Review Request 44338: Added URI test for HDFS.

2016-03-03 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44338/ --- Review request for mesos and Jie Yu. Repository: mesos Description ---

Review Request 44337: Updated utils for URI constructor streaming.

2016-03-03 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44337/ --- Review request for mesos and Jie Yu. Bugs: MESOS-4851

Re: Review Request 44326: Added missing includes in metrics.

2016-03-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44326/ --- (Updated March 3, 2016, 2:42 p.m.) Review request for mesos, Benjamin Bannier

Re: Review Request 44260: Moved metrics of the hierarchical allocator to its own file.

2016-03-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44260/#review121861 --- src/master/allocator/mesos/metrics.hpp (line 29)

Re: Review Request 43629: Especially updated tests to use the updated MesosTest helpers.

2016-03-03 Thread Joseph Wu
> On March 3, 2016, 5:55 a.m., Bernd Mathiske wrote: > > src/tests/slave_recovery_tests.cpp, line 3461 > > > > > > Why was this moved up here? Couldn't this be in line 3389/3402? In this case, it's because all

Review Request 44349: Added change of ACL objects to CHANGELOG.

2016-03-03 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44349/ --- Review request for mesos and Vinod Kone. Bugs: MESOS-4591

Review Request 44350: Added net_cls isolator to CHANGELOG.

2016-03-03 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44350/ --- Review request for mesos and Jie Yu. Bugs: MESOS-4854

Re: Review Request 44255: Add metrics for {RESERVE, UNRESERVE, CREATE, DESTROY} offer operation.

2016-03-03 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44255/#review121913 --- Ship it! Ship It! - Greg Mann On March 3, 2016, 5:40 a.m.,

Re: Review Request 44325: Removed superfluous periods in failure messages.

2016-03-03 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44325/#review121921 --- Fix it, then Ship it!

Review Request 44348: Updated CHANGELOG for floating point resource changes.

2016-03-03 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44348/ --- Review request for mesos and Vinod Kone. Repository: mesos Description

Re: Review Request 44326: Added missing includes in metrics.

2016-03-03 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44326/#review121916 --- 3rdparty/libprocess/src/metrics/metrics.cpp (line 25)

Re: Review Request 44350: Added net_cls isolator to CHANGELOG.

2016-03-03 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44350/#review121918 --- Ship it! Ship It! - Jie Yu On March 3, 2016, 7:24 p.m.,

Re: Review Request 44325: Removed superfluous periods in failure messages.

2016-03-03 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44325/#review121920 --- Ship it! - Benjamin Bannier On March 3, 2016, 10:20 a.m.,

Re: Review Request 43615: Update test suite to use the reworked MesosTest helpers.

2016-03-03 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43615/ --- (Updated March 3, 2016, 11:59 a.m.) Review request for mesos, Bernd Mathiske

Review Request 44353: Updated CHANGELOG about experimental support for Unified containerizer.

2016-03-03 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44353/ --- Review request for mesos, Gilbert Song, Jojy Varghese, Timothy Chen, and Vinod

Re: Review Request 44342: XFS disk resource isolator.

2016-03-03 Thread James Peach
> On March 3, 2016, 5:30 p.m., haosdent huang wrote: > > src/Makefile.am, line 869 > > > > > > Do you forgot to update CMake? Nope, I just didn't want to block on learning how to do this in Cmake. Can we split

Re: Review Request 44332: Fixed a typo in the HierarchicalAllocatorTest.CoarseGrained test.

2016-03-03 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44332/#review121907 --- Fix it, then Ship it!

Re: Review Request 43613: Refactor cluster test helpers into self-contained objects.

2016-03-03 Thread Joseph Wu
> On March 3, 2016, 5:22 a.m., Bernd Mathiske wrote: > > src/tests/cluster.hpp, line 213 > > > > > > I don't think we need this constructor. We are using an instance of > > this merely as an aggregate temporary

Re: Review Request 44337: Updated utils for URI constructor streaming.

2016-03-03 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44337/#review121909 --- src/uri/utils.cpp (lines 57 - 61)

Re: Review Request 44342: XFS disk resource isolator.

2016-03-03 Thread James Peach
> On March 3, 2016, 5:51 p.m., haosdent huang wrote: > > Hi, @jamespeach Your patch looks great. I have a question, `ENOSPC` only > > happen when call `write` and we could not register any singal handler in > > this isolator to receive a exceed quote singal, right? The write fails with

Re: Review Request 44342: XFS disk resource isolator.

2016-03-03 Thread James Peach
> On March 3, 2016, 5:51 p.m., haosdent huang wrote: > > src/slave/containerizer/mesos/isolators/disk/xfs.cpp, line 597 > > > > > > Why we don't move assign logic to `isolate()`? You want to do the project ID

Re: Review Request 44342: XFS disk resource isolator.

2016-03-03 Thread haosdent huang
> On March 3, 2016, 5:51 p.m., haosdent huang wrote: > > Hi, @jamespeach Your patch looks great. I have a question, `ENOSPC` only > > happen when call `write` and we could not register any singal handler in > > this isolator to receive a exceed quote singal, right? > > James Peach wrote: >

Re: Review Request 44278: Added HTTP scheduler stream IDs.

2016-03-03 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44278/#review121902 --- Ship it! Ship It! - Vinod Kone On March 3, 2016, 5:13 p.m.,

Re: Review Request 44342: XFS disk resource isolator.

2016-03-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44342/#review121890 --- Hi, @jamespeach Your patch looks great. I have a question,

Re: Review Request 44342: XFS disk resource isolator.

2016-03-03 Thread James Peach
> On March 3, 2016, 5:38 p.m., haosdent huang wrote: > > src/slave/containerizer/mesos/containerizer.cpp, line 216 > > > > > > Does your isolator only work in linux? Yes. - James

Re: Review Request 44342: XFS disk resource isolator.

2016-03-03 Thread haosdent huang
> On March 3, 2016, 5:30 p.m., haosdent huang wrote: > > src/Makefile.am, line 869 > > > > > > Do you forgot to update CMake? > > James Peach wrote: > Nope, I just didn't want to block on learning how to do

Re: Review Request 44342: XFS disk resource isolator.

2016-03-03 Thread James Peach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44342/ --- (Updated March 3, 2016, 6:06 p.m.) Review request for mesos, Jie Yu and Jiang

Re: Review Request 44353: Updated CHANGELOG about experimental support for Unified containerizer.

2016-03-03 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44353/#review121926 --- Ship it! Ship It! - Vinod Kone On March 3, 2016, 8 p.m.,

Re: Review Request 44348: Updated CHANGELOG for floating point resource changes.

2016-03-03 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44348/ --- (Updated March 3, 2016, 8:14 p.m.) Review request for mesos and Vinod Kone.

<    1   2   3   >