Wow, this code is such a disaster.

I sent out a fix:
        http://openvswitch.org/pipermail/dev/2015-October/061212.html

On Fri, Sep 11, 2015 at 01:58:43PM -0700, Ray Li wrote:
> Hi, here's another issue. If I create an empty group first without buckets,
> and then run insert-buckets with command_bucket_id=last, it will fail to
> add the bucket and somehow delete the empty group. it works if I ran
> insert-buckets with command_bucket_id=first though.
> 
> On Thu, Sep 10, 2015 at 5:10 PM, Ray Li <rayli1...@gmail.com> wrote:
> 
> > Hi, I've ran into another issue. I'm trying this on a new environment.
> > When I tried to add a group for the first time using OpenFlow15 with
> > bucket_id, I got an OFPBAC_BAD_LEN error. It worked though when I tried to
> > add a group using OpenFlow11 without bucket_id. Any idea what this means?
> >
> > ovs-ofctl add-group br1 group_id=0,type=all,bucket="actions=output:10" -O
> > OpenFlow11
> > (This works.)
> >
> > ovs-ofctl del-groups br1 -O OpenFlow11
> >
> > ovs-ofctl add-group br1
> > group_id=0,type=all,bucket="bucket_id=100,actions=output:10" -O OpenFlow15
> > OFPT_ERROR (OF1.5) (xid=0x2): OFPBAC_BAD_LEN
> > OFPT_GROUP_MOD (OF1.5) (xid=0x2):
> >  ADD group_id=0,type=all,bucket=bucket_id:100,actions=output:10
> >
> > When I looked at the log files it shows:
> > 2015-09-11T00:08:59.622Z|00437|ofp_actions|WARN|bad action at offset 0
> > (OFPBAC_BAD_LEN):
> > 00000000  00 00 00 10 00 00 00 0a-
> > 2015-09-11T00:08:59.622Z|00438|connmgr|INFO|br0_vxlan<->unix: sending
> > OFPBAC_BAD_LEN error reply to OFPT_GROUP_MOD message
> >
> >
> >
> > Thanks
> >
> >
> >
> > On Thu, Sep 10, 2015 at 2:56 PM, Ben Pfaff <b...@nicira.com> wrote:
> >
> >> With the current behavior, insert-buckets doesn't automatically allocate
> >> non-overlapping bucket ids.  This could be considered a bug in the
> >> documentation or in the code.  It would be more user-friendly to
> >> consider a bug in the code and fix it in ovs-ofctl, but that's a bigger
> >> job than just fixing the crash as these commits do.
> >>
> >> On Thu, Sep 10, 2015 at 02:51:03PM -0700, Ray Li wrote:
> >> > According to ovs-ofctl's man page, if no bucket_id is specified it's
> >> > support to allocate one. Is that not being done? Or is that the bug
> >> itself?
> >> >
> >> > On Thu, Sep 10, 2015 at 1:21 PM, Ben Pfaff <b...@nicira.com> wrote:
> >> >
> >> > > On Thu, Sep 10, 2015 at 11:46:16AM -0700, Ray Li wrote:
> >> > > > Does OVS 2.4 support ovs-ofctl insert-buckets?
> >> > > >
> >> > > > I just upgraded to OVS 2.4.0, and here's what I tried to do:
> >> > > >
> >> > > > ovs-vsctl set bridge br0
> >> > > >
> >> > >
> >> protocols=OpenFlow10,OpenFlow11,OpenFlow12,OpenFlow13,OpenFlow14,OpenFlow15
> >> > > >
> >> > > > ovs-ofctl add-group br0
> >> group_id=0,type=all,bucket="actions=output:10" -O
> >> > > > OpenFlow11
> >> > > >
> >> > > > ovs-ofctl dump-groups br0 -O OpenFlow11
> >> > > > (which shows the groups table correctly with a single bucket)
> >> > > >
> >> > > > ovs-ofctl insert-buckets br0
> >> > > > group_id=0,command_bucket_id=last,bucket="actions=output:1" -O
> >> OpenFlow15
> >> > > >
> >> > > > ovs-ofctl dump-groups br0 -O OpenFlow11
> >> > > > (This then hangs indefinitely)
> >> > >
> >> > > It's a bug, thanks for reporting it.
> >> > >
> >> > > The underlying problem is that in both cases no bucket_id was
> >> specified,
> >> > > so it defaults to the same value in each case, and duplicate bucket
> >> IDs
> >> > > were not correctly handled.  You can avoid the problem by specifying a
> >> > > different bucket_it in your "insert-buckets" command.
> >> > >
> >> > > I posted a fix.  You can test it by applying the following two
> >> patches,
> >> > > in this order:
> >> > >
> >> http://openvswitch.org/pipermail/dev/2015-September/059865.html
> >> > >
> >> http://openvswitch.org/pipermail/dev/2015-September/059864.html
> >> > >
> >>
> >
> >
_______________________________________________
discuss mailing list
discuss@openvswitch.org
http://openvswitch.org/mailman/listinfo/discuss

Reply via email to