> On Dec. 12, 2018, 3:51 p.m., Gilbert Song wrote: > > configure.ac > > Lines 352 (patched) > > <https://reviews.apache.org/r/68016/diff/9/?file=2110248#file2110248line352> > > > > Do we have a plan to deprecate this configuraton flag in the future? > > E.g., always only compile the seccomp library on linux. > > Andrei Budnik wrote: > Currently, we support old kernel version that does not support `seccomp`. > E.g., `centos 6` is bases on pre-seccomp kernel. > I think we will deprecate this flag in the future.
ok, could we add a TODO here? so that people would capture our deprecation plan:) - Gilbert ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68016/#review211258 ----------------------------------------------------------- On Nov. 8, 2018, 7:23 a.m., Andrei Budnik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68016/ > ----------------------------------------------------------- > > (Updated Nov. 8, 2018, 7:23 a.m.) > > > Review request for mesos, Andrew Schwartzmeyer, Gilbert Song, Jie Yu, James > Peach, and Qian Zhang. > > > Bugs: MESOS-9032 > https://issues.apache.org/jira/browse/MESOS-9032 > > > Repository: mesos > > > Description > ------- > > This library is needed to implement Seccomp syscall filtering in the > Mesos containerizer. This patch introduces `seccomp-isolator` build > flag, which is used to include or exclude sources related to Seccomp > from the build. Since Seccomp is a Linux-specific feature, the flag > is disabled by default. Enabling `seccomp-isolator` means either: > > 1. Compiling and linking against the bundled version of libseccomp from > sources (default). > > 2. Linking against the libseccomp installed in the OS, > if `--with-libseccomp` build flag is provided. > > > Diffs > ----- > > 3rdparty/CMakeLists.txt fb80b39d1717948e87256272bc5989f73d4f5995 > 3rdparty/Makefile.am e625e7be1743348d02c6dbb8e0a92d1a395b0ef4 > 3rdparty/cmake/Versions.cmake 69fc594ec5ba2887b20b88ec0767a5d801411411 > 3rdparty/versions.am 99ef92087f6958d83ba415e84db5cbbb0c597573 > cmake/CompilationConfigure.cmake 2130c9b70eae62948d469b37692ff4eea3a4baf2 > configure.ac c193adf93531170fb31a31c296e3d4ae1d0300b2 > src/CMakeLists.txt bde070445b644e15d46c390d1c983caabaa1fec8 > src/Makefile.am 8da1a05b618f17542fec9b5057484a9bae57658a > src/python/native_common/ext_modules.py.in > 1f2e6c131d18e3e2fbc2e865c4698c83e73b87ba > > > Diff: https://reviews.apache.org/r/68016/diff/9/ > > > Testing > ------- > > > Thanks, > > Andrei Budnik > >
