On Thu, Nov 1, 2012 at 3:50 PM, Ben Pfaff <b...@nicira.com> wrote: > On Thu, Nov 01, 2012 at 03:48:33PM -0700, Jesse Gross wrote: >> On Thu, Nov 1, 2012 at 1:26 PM, Pravin Shelar <pshe...@nicira.com> wrote: >> > >> > On Thu, Nov 1, 2012 at 1:09 PM, Ben Pfaff <b...@nicira.com> wrote: >> >> >> >> On Thu, Nov 01, 2012 at 12:47:47PM -0700, Pravin B Shelar wrote: >> >> > Following patch fixes `ovs-appctl ofproto/trace` command with >> >> > ipv4_tunnel key in flow. >> >> > >> >> > Bug #13785 >> >> > Signed-off-by: Pravin B Shelar <pshe...@nicira.com> >> >> >> >> Thank you! >> >> >> >> Will you please add a test to tests/odp.at? >> >> >> >> Please use "%"SCNi32 for flags instead of "%i", because tun_flags is a >> >> __u32 not an int. >> > >> > >> > Jesse mentioned that he has a patch for same and will send it. >> >> I realized that the tests that I have are dependent on the rest of the >> infrastructure to actually read the tunnel flow information into >> userspace - otherwise you won't get the same thing out as is put in. >> I don't want put that code in yet and I'm not sure that it's worth >> adding a special test for something that's going to change almost >> immediately. >> >> I'll still post my patch since it has a few small improvements but >> it's basically the same as Pravin's at this point. > > All I want is a one-liner, like this: > > diff --git a/tests/odp.at b/tests/odp.at > index 9617af2..577d487 100644 > --- a/tests/odp.at > +++ b/tests/odp.at > @@ -91,6 +91,7 @@ push_vlan(tpid=0x9100,vid=13,pcp=5) > push_vlan(tpid=0x9100,vid=13,pcp=5,cfi=0) > pop_vlan > sample(sample=9.7%,actions(1,2,3,push_vlan(vid=1,pcp=2))) > +ipv4_tunnel(...something...) > ]) > AT_CHECK_UNQUOTED([test-odp parse-actions < actions.txt], [0], > [`cat actions.txt` > > with the ...something... filled in reasonably.
OK, I guess it's fine to treat it as an action. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
