On Fri, Jul 24, 2015 at 02:45:33PM +0900, Minoru TAKAHASHI wrote:
> Hi,
> 
> On 2015年07月22日 17:25, Minoru TAKAHASHI wrote:
> > Hi,
> > 
> > Thank you for reply.
> > 
> > On 2015年07月22日 00:19, Ben Pfaff wrote:
> >> On Tue, Jul 21, 2015 at 03:10:08PM +0900, Minoru TAKAHASHI wrote:
> >>>> they do not make the corresponding change to the decoder
> >>>
> >>> Should I fix this problem?
> >>> If so, please tell me that which source file should I fix.
> >>> (Sorry, I am new in implementation of OpenvSwitch.)
> >>
> >> I cannot apply anything to the repository that makes tests fail, so yes
> >> you should fix the decoder.
> >>
> >> The decode function is right next to the encode function.
> 
> > ## ------------------------------ ##
> > ## openvswitch 2.4.90 test suite. ##
> > ## ------------------------------ ##
> > 
> > 694: ofproto - del group (OpenFlow 1.5)              FAILED (ofproto.at:350)
> > 696: ofproto - insert buckets                        FAILED (ofproto.at:424)
> 
> I think that the following code is causing the above error.
> 
> [lib/ofp-msgs.h]
> https://github.com/openvswitch/ovs/blob/master/lib/ofp-msgs.h#L336
> 
> [tests/ofp-print.at]
> https://github.com/openvswitch/ovs/blob/master/tests/ofp-print.at#L2003
> 
> After I fix these, the above error does not occur.

Did you run all the tests?  I see three failures:

## ------------------------ ##
## Summary of the failures. ##
## ------------------------ ##
Failed tests:
openvswitch 2.4.90 test suite test groups:

 NUM: FILE-NAME:LINE     TEST-GROUP-NAME
      KEYWORDS

 695: ofproto.at:343     ofproto - del group (OpenFlow 1.5)
 697: ofproto.at:418     ofproto - insert buckets
 698: ofproto.at:476     ofproto - remove buckets
_______________________________________________
discuss mailing list
discuss@openvswitch.org
http://openvswitch.org/mailman/listinfo/discuss

Reply via email to