Re: Review Request 35354: Smaller fixes in libprocess firewall initialization

2015-06-24 Thread Till Toenshoff
> On June 22, 2015, 10:21 p.m., Ben Mahler wrote: > > Till, will you be able to help Alex get these committed? If not, I'm happy > > to help out. Sry for the delay, we had some holiday / sick-day slowdown here :). Comitted them now. - Till --

Re: Review Request 35354: Smaller fixes in libprocess firewall initialization

2015-06-24 Thread Alexander Rojas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35354/ --- (Updated June 24, 2015, 5:06 p.m.) Review request for mesos, Ben Mahler and Til

Re: Review Request 35354: Smaller fixes in libprocess firewall initialization

2015-06-24 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35354/#review89164 --- Ship it! src/slave/main.cpp (line 68)

Re: Review Request 35354: Smaller fixes in libprocess firewall initialization

2015-06-22 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35354/#review88865 --- Till, will you be able to help Alex get these committed? If not, I'm

Re: Review Request 35354: Smaller fixes in libprocess firewall initialization

2015-06-15 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35354/#review87912 --- Patch looks great! Reviews applied: [35353, 35354] All tests passe

Re: Review Request 35354: Smaller fixes in libprocess firewall initialization

2015-06-15 Thread Alexander Rojas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35354/ --- (Updated June 15, 2015, 11:24 a.m.) Review request for mesos, Ben Mahler and Ti