On Thu, Sep 13, 2018 at 6:53 PM Amritha Nambiar <amritha.namb...@intel.com> wrote: > > This patch introduces a range classifier to support filtering based > on ranges. Only port-range filters are supported currently. This can > be combined with flower classifier to support filters that are a > combination of port-ranges and other parameters based on existing > fields supported by cls_flower.
Why should we have a special-purpose filter just for ports here? We have achieved almost the same goal with u32 filter: https://github.com/apache/mesos/blob/master/src/slave/containerizer/mesos/isolators/network/port_mapping.cpp There is a large overlap with other general purpose filters. I don't see you provide any justification for the purpose of it. If it is just for convenience, can't we just make it on top of other general purpose header-matching filters?
