Re: Review Request 64084: Made explicit that a 'switch' covers all branches.

2017-11-27 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64084/#review191883 --- Ship it! Ship It! - Alexander Rukletsov On Nov. 27, 2017,

Re: Review Request 64084: Made explicit that a 'switch' covers all branches.

2017-11-27 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64084/ --- (Updated Nov. 27, 2017, 2:07 p.m.) Review request for mesos and Alexander

Review Request 64084: Made explicit that a 'switch' covers all branches.

2017-11-27 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64084/ --- Review request for mesos and Alexander Rukletsov. Repository: mesos

Re: Review Request 64054: Refactored offer operation handling for speculative operations.

2017-11-27 Thread Benjamin Bannier
> On Nov. 26, 2017, 4:40 a.m., Jie Yu wrote: > > src/master/master.hpp > > Lines 2795-2841 (original) > > > > > > This should still be part of `addOfferOperation`. In fact, anything to > > do with `used` resourecs

Review Request 64081: Fix a code snippet in libprocess documentation.

2017-11-27 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64081/ --- Review request for mesos and Benjamin Hindman. Repository: mesos Description

Re: Review Request 64001: Made sure all true allocator agent updates trigger allocations.

2017-11-27 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64001/ --- (Updated Nov. 27, 2017, 11:51 a.m.) Review request for mesos and Jie Yu.

Re: Review Request 64001: Made sure all true allocator agent updates return correct status.

2017-11-27 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64001/#review191876 --- Ship it! Please correct the commit message to omit mentioning

Re: Review Request 64001: Made sure all true allocator agent updates return correct status.

2017-11-27 Thread Jan Schlicht
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64001/#review191877 --- Ship it! Ship It! - Jan Schlicht On Nov. 24, 2017, 3:07

Re: Review Request 64068: Increased executor log verbosity in tests.

2017-11-27 Thread Armand Grillet
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64068/#review191875 --- Ship it! Ship It! - Armand Grillet On Nov. 24, 2017, 7:12

Re: Review Request 63910: Added 3 tests for TCP/HTTP(S) health check support for Docker container.

2017-11-27 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63910/#review191871 --- src/tests/containerizer/docker_containerizer_tests.cpp Lines

Re: Review Request 63022: Imported resources from CSI plugins in storage local resource provider.

2017-11-27 Thread Chun-Hung Hsiao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63022/ --- (Updated Nov. 27, 2017, 9:32 a.m.) Review request for mesos, James DeFelice,

Re: Review Request 63795: Made `mesos-tcp-connect` support IPv6.

2017-11-27 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63795/#review191869 --- Ship it! src/checks/tcp_connect.cpp Lines 33-34 (patched)

Re: Review Request 63796: Made `CheckerProcess` support IPv6 for HTTP/TCP check.

2017-11-27 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63796/#review191874 --- Ship it! Ship It! - Alexander Rukletsov On Nov. 16, 2017,

Re: Review Request 63794: Added a new member field `ipv6` to the `CheckerProcess` class.

2017-11-27 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63794/#review191873 --- Ship it! Ship It! - Alexander Rukletsov On Nov. 17, 2017,

<    1   2