Github user jieyu commented on a diff in the pull request: https://github.com/apache/mesos/pull/232#discussion_r137335576 --- Diff: docs/cni.md --- @@ -487,6 +488,18 @@ guide on adding Calico-CNI to Mesos](https://github.com/projectcalico/calico-containers/blob/master/docs/mesos/ManualInstallCalicoCNI.md). +#### <a name="a-cilium-network">A Cilium network</a> --- End diff -- Please remove tailing white spaces. It failed our style checker: ```Jies-MacBook-Pro:mesos jie$ support/apply-review.sh -g 232 Cookie coming from patch-diff.githubusercontent.com attempted to set domain to github.com 2017-09-06 10:26:30 URL:https://patch-diff.githubusercontent.com/raw/apache/mesos/pull/232.patch [1509] -> "232.patch" [1] 232.patch:31: trailing whitespace. Cilium brings HTTP-aware network security filtering to Linux container 232.patch:32: trailing whitespace. frameworks. Using a new Linux kernel technology called BPF, Cilium 232.patch:33: trailing whitespace. provides a simple and efficient way to define and enforce both 232.patch:36: trailing whitespace. For more information on using Cilium with Mesos, check out the warning: 4 lines add whitespace errors. docs/cni.md:495: trailing whitespace. +Cilium brings HTTP-aware network security filtering to Linux container docs/cni.md:496: trailing whitespace. +frameworks. Using a new Linux kernel technology called BPF, Cilium docs/cni.md:497: trailing whitespace. +provides a simple and efficient way to define and enforce both docs/cni.md:500: trailing whitespace. +For more information on using Cilium with Mesos, check out the ```
---