----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39417/#review107322 -----------------------------------------------------------
Bad patch! Reviews applied: [39415, 39416, 40497, 40506, 39417] Failed command: ./support/apply-review.sh -n -r 39417 Error: 2015-11-20 07:23:28 URL:https://reviews.apache.org/r/39417/diff/raw/ [9792/9792] -> "39417.patch" [1] error: patch failed: src/slave/containerizer/mesos/isolators/network/port_mapping.cpp:1385 error: src/slave/containerizer/mesos/isolators/network/port_mapping.cpp: patch does not apply Failed to apply patch - Mesos ReviewBot On Nov. 19, 2015, 10:59 p.m., Cong Wang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39417/ > ----------------------------------------------------------- > > (Updated Nov. 19, 2015, 10:59 p.m.) > > > Review request for mesos, Ian Downes and Jie Yu. > > > Repository: mesos > > > Description > ------- > > When --egress_unique_flow_per_container is enabled, we need to install a flow > classifier (fq_codel) qdisc on egress side. This flag specifies where to > install it in the hierarchy. By default, we install it at root. But, for > example, if you have already installed HTB qdisc at root, you may want this > to be installed in other place than root, specify an HTB class ID as its > parent here. > > > Diffs > ----- > > docs/configuration.md 72847e5efe7008fdec8287cce100857f9e7c0fe0 > src/slave/containerizer/mesos/isolators/network/port_mapping.cpp > cc0152ce44e9fa34210efda8a1c4a6374167eab1 > src/slave/flags.hpp 6ae7c94d2e05d81c8b970e7dcaa82d8aa4de7936 > src/slave/flags.cpp 26f554e1a73f1f7f3d7baec7bfc1a7f456c5677c > > Diff: https://reviews.apache.org/r/39417/diff/ > > > Testing > ------- > > Manual tests, with and without a pre-installed HTB qdisc and classes. > > > Thanks, > > Cong Wang > >
