> On May 18, 2017, 2:36 p.m., Jie Yu wrote: > > configure.ac > > Lines 347 (patched) > > <https://reviews.apache.org/r/59372/diff/3/?file=1724123#file1724123line347> > > > > Should we set enable_port_mapping_isolator=no here explicitly?
Doing that would override the setting from `--enable-port-mapping-isolator` as it would just set the flag to `no` always. - Kapil ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59372/#review175419 ----------------------------------------------------------- On May 18, 2017, 2:28 p.m., Kapil Arya wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59372/ > ----------------------------------------------------------- > > (Updated May 18, 2017, 2:28 p.m.) > > > Review request for mesos, Jie Yu and Tim Hansen. > > > Repository: mesos > > > Description > ------- > > The corresponding flag (--enable-port-mapping-isolator) was accidentally > enabled by-default in 20dee4190838b5bc7eb9cb524af413e3fe3fe082 > (https://reviews.apache.org/r/59193/). > > > Diffs > ----- > > configure.ac d5236700d2ab06232960f0bed43e8394e9a29aa4 > > > Diff: https://reviews.apache.org/r/59372/diff/3/ > > > Testing > ------- > > Tested the following configurations: > Configure flag Isolator status > `./configure` Disabled > `./configure --with-port-mapping-isolator` Enabled > `./configure --with-network-isolator` Enabled > > > Thanks, > > Kapil Arya > >
