Re: Review Request 68022: Enabled Seccomp filter in the containerizer launcher.

2019-01-27 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68022/#review212371 --- Ship it! Ship It! - Gilbert Song On Aug. 6, 2018, 6:39 a.m.,

Re: Review Request 68022: Enabled Seccomp filter in the containerizer launcher.

2019-01-22 Thread Gilbert Song
> On Jan. 22, 2019, 12:31 a.m., Qian Zhang wrote: > > src/slave/containerizer/mesos/launch.cpp > > Line 945 (original), 1028 (patched) > > > > > > So we call `calculateCapabilities` twice in this file, can we merge

Re: Review Request 68022: Enabled Seccomp filter in the containerizer launcher.

2019-01-22 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68022/#review212210 --- Fix it, then Ship it! src/slave/containerizer/mesos/launch.cpp

Re: Review Request 68022: Enabled Seccomp filter in the containerizer launcher.

2019-01-22 Thread Qian Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68022/#review212191 --- src/slave/containerizer/mesos/launch.cpp Line 945 (original), 102

Re: Review Request 68022: Enabled Seccomp filter in the containerizer launcher.

2019-01-14 Thread Qian Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68022/#review211988 --- Ship it! Ship It! - Qian Zhang On Aug. 6, 2018, 9:39 p.m., A

Re: Review Request 68022: Enabled Seccomp filter in the containerizer launcher.

2019-01-04 Thread Andrei Budnik
> On Dec. 27, 2018, 9:16 a.m., Qian Zhang wrote: > > src/slave/containerizer/mesos/launch.cpp > > Lines 1196-1197 (patched) > > > > > > Hmm, this seems unfortunate, will it cause container cannot be launched? Since

Re: Review Request 68022: Enabled Seccomp filter in the containerizer launcher.

2018-12-27 Thread Qian Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68022/#review211543 --- src/slave/containerizer/mesos/launch.cpp Lines 1196-1197 (patched

Re: Review Request 68022: Enabled Seccomp filter in the containerizer launcher.

2018-11-14 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68022/#review210549 --- FAIL: Failed to apply the dependent review: 67844. Failed command

Re: Review Request 68022: Enabled Seccomp filter in the containerizer launcher.

2018-11-08 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68022/#review210419 --- Bad patch! Reviews applied: [68022, 68021, 68020, 68019, 68018, 6

Re: Review Request 68022: Enabled Seccomp filter in the containerizer launcher.

2018-11-08 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68022/#review210417 --- FAIL: Failed to apply the dependent review: 67844. Failed command

Re: Review Request 68022: Enabled Seccomp filter in the containerizer launcher.

2018-10-22 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68022/#review209904 --- Bad patch! Reviews applied: [68022, 68021, 68020, 68019, 68018, 6

Re: Review Request 68022: Enabled Seccomp filter in the containerizer launcher.

2018-10-22 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68022/#review209888 --- FAIL: Failed to apply the dependent review: 67844. Failed command

Re: Review Request 68022: Enabled Seccomp filter in the containerizer launcher.

2018-08-07 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68022/#review206965 --- Bad patch! Reviews applied: [68022, 68021, 68020, 68019, 68018, 6

Re: Review Request 68022: Enabled Seccomp filter in the containerizer launcher.

2018-08-06 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68022/#review206922 --- FAIL: Failed to apply the dependent review: 67844. Failed command

Re: Review Request 68022: Enabled Seccomp filter in the containerizer launcher. (WIP)

2018-08-06 Thread Andrei Budnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68022/ --- (Updated Aug. 6, 2018, 1:39 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 68022: Enabled Seccomp filter in the containerizer launcher. (WIP)

2018-07-23 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68022/#review206372 --- Bad patch! Reviews applied: [68022, 68021, 68020, 68019, 68018, 6

Re: Review Request 68022: Enabled Seccomp filter in the containerizer launcher. (WIP)

2018-07-23 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68022/#review206337 --- FAIL: Failed to apply the dependent review: 67844. Failed command

Review Request 68022: Enabled Seccomp filter in the containerizer launcher. (WIP)

2018-07-23 Thread Andrei Budnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68022/ --- Review request for mesos, Gilbert Song, Jie Yu, James Peach, and Qian Zhang. Bu