----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32219/#review77446 -----------------------------------------------------------
Patch looks great! Reviews applied: [32219] All tests passed. - Mesos ReviewBot On March 23, 2015, 5:29 p.m., Cong Wang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32219/ > ----------------------------------------------------------- > > (Updated March 23, 2015, 5:29 p.m.) > > > Review request for mesos, Chi Zhang, Ian Downes, and Jie Yu. > > > Bugs: MESOS-2514 > https://issues.apache.org/jira/browse/MESOS-2514 > > > Repository: mesos > > > Description > ------- > > When we enable bandwidth cap, htb is used on egress side inside containers, > however, the default leaf qdisc for a htb class is still pfifo_fast, which is > known to have buffer bloat. Change the default leaf qdisc to fq_codel too. > > > Diffs > ----- > > src/slave/containerizer/isolators/network/port_mapping.cpp > 7b4aac88d40e4e5dc3331cb19b647a1d872ef7e2 > > Diff: https://reviews.apache.org/r/32219/diff/ > > > Testing > ------- > > Manually started a container and fq_codel qdisc was successfully created. > > > Thanks, > > Cong Wang > >
