Re: Review Request 49955: Disabled the `--registry_strict` master flag.

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

Re: Review Request 49955: Disabled the `--registry_strict` master flag.

2016-07-21 Thread Vinod Kone
> On July 22, 2016, 12:26 a.m., Vinod Kone wrote: > > Ship It! can you rebase? - Vinod --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49955/#review143161

Re: Review Request 49955: Disabled the `--registry_strict` master flag.

2016-07-21 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49955/#review143161 --- Ship it! Ship It! - Vinod Kone On July 20, 2016, 1:01 p.m.,

Re: Review Request 49955: Disabled the `--registry_strict` master flag.

2016-07-20 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49955/ --- (Updated July 20, 2016, 1:01 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 49955: Disabled the `--registry_strict` master flag.

2016-07-20 Thread Neil Conway
> On July 19, 2016, 10:56 p.m., Vinod Kone wrote: > > src/master/flags.cpp, line 86 > > > > > > // TODO(neilc): This flag is deprecated in 1.0 and will be removed 6 > > months later. I used the text "This flag is

Re: Review Request 49955: Disabled the `--registry_strict` master flag.

2016-07-20 Thread Neil Conway
> On July 15, 2016, 7:26 p.m., Benjamin Mahler wrote: > > Could we clarify the description a bit? I'm having a hard time convincing > > myself that this makes sense in the world we'd like to move to. > > > > The strict flag was not intended as a mechanims for ensuring that > > partitioned

Re: Review Request 49955: Disabled the `--registry_strict` master flag.

2016-07-19 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49955/#review142863 --- LGTM, module benm's comments. src/master/flags.cpp (line 86)

Re: Review Request 49955: Disabled the `--registry_strict` master flag.

2016-07-15 Thread Benjamin Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49955/#review142433 --- Could we clarify the description a bit? I'm having a hard time

Re: Review Request 49955: Disabled the `--registry_strict` master flag.

2016-07-12 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49955/#review141935 --- Patch looks great! Reviews applied: [49955] Passed command:

Re: Review Request 49955: Disabled the `--registry_strict` master flag.

2016-07-12 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49955/ --- (Updated July 12, 2016, 5:40 p.m.) Review request for mesos, Benjamin Hindman,

Review Request 49955: Disabled the `--registry_strict` master flag.

2016-07-12 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49955/ --- Review request for mesos, Benjamin Hindman, Benjamin Mahler, Joris Van