On Fri, Jun 13, 2014 at 12:27:12PM -0700, Ben Pfaff wrote:
> On Tue, Jun 10, 2014 at 07:04:28PM +0530, Avinash wrote:
> > This commit includes:
> > 1. Support for new config parameter in xbundle.
> > 2. Changes in determining the vlan id and number of VLANs to be present
> >    in the output packet.
> > 
> > For testing purpose, planned to make changes in normalize-actions to
> > support stacked VLANs.
> > 
> > Signed-off-by: Avinash <[email protected]>
> 
> I would probably squash this with the previous patch.
> 
> I don't see any update to the NEWS file describing the new feature.
> You should add one.

I also don't see any tests.  We certainly need some.  Please take a
look, to start with, at tests/ofproto-dpif.at, in particular the
"ofproto-dpif - VLAN handling" test.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to