On Wed, Feb 1, 2017 at 9:45 AM, Ben Pfaff <b...@ovn.org> wrote:
> On Tue, Jan 31, 2017 at 08:24:02PM -0800, Andy Zhou wrote:
>> When adding userspace datapath clone action, the corresponding odp
>> actions parser and unit tests were missing. This patch adds them.
>>
>> Reported-by: Ben Pfaff <b...@ovn.org>
>> Signed-off-by: Andy Zhou <az...@ovn.org>
>>
>> ---
>> Note: back port to branch-2.7
>
> Thanks!
>
> Probably, it would be good to be more tolerant of white space. When I
> add a space before the closing ) in a clone action, I get an error:
>
> blp@sigabrt:~/nicira/ovs/_build(130)$ echo 'clone(1,2 )' | tests/ovstest
> test-odp parse-actions
> odp_actions_from_string: error
> blp@sigabrt:~/nicira/ovs/_build(0)$
>
> Thanks,
>
> Ben.
Yes, It is worth while improvement to have. I just posted v2. Thanks Ben.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
