> On Dec. 27, 2018, 3:44 p.m., Qian Zhang wrote: > > src/slave/flags.hpp > > Lines 192 (patched) > > <https://reviews.apache.org/r/68020/diff/9/?file=2111366#file2111366line192> > > > > Why not make this flag as optional too?
If we make this flag as optional, then we do not need to use empty value to indicate disabling default profile, instead we will disable default profile when this flag is not specified which seems more intuitive to me. - Qian ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68020/#review211537 ----------------------------------------------------------- On Nov. 8, 2018, 11:24 p.m., Andrei Budnik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68020/ > ----------------------------------------------------------- > > (Updated Nov. 8, 2018, 11:24 p.m.) > > > Review request for mesos, Gilbert Song, Jie Yu, James Peach, and Qian Zhang. > > > Repository: mesos > > > Description > ------- > > `--seccomp_config_dir` and `--seccomp_profile_name` flags have been > added to the agent. These flags are used by the `linux/seccomp` > isolator to specify the path of the directory containing Seccomp > profiles and the path of the default Seccomp profile. > > > Diffs > ----- > > docs/configuration/agent.md 7a8df6852dc2af174a6c5a552dca88fa1b1c29f3 > src/slave/flags.hpp 29d8b7985ffde57da02b5fe0d3a524e98acc27c8 > src/slave/flags.cpp ccaf65029ec2d0e78041fc3992a0bf5ca0798686 > > > Diff: https://reviews.apache.org/r/68020/diff/9/ > > > Testing > ------- > > > Thanks, > > Andrei Budnik > >
