[ovs-dev] clone action is broken in the master

2017-03-09 Thread Numan Siddique
Hi Andy and All,

The clone action is broken after the commit [1] - xlate: Translate openflow
clone into odp sample action.

Because of which, the openstack CI job for OVN is broken.
It works once I revert this commit.

Thanks
Numan


[1] -
https://github.com/openvswitch/ovs/commit/456024cb1c5e58ccd9475b95bfdf382391691aa2
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] clone action is broken in the master

2017-03-09 Thread Andy Zhou
On Thu, Mar 9, 2017 at 2:34 AM, Numan Siddique  wrote:
> Hi Andy and All,
>
> The clone action is broken after the commit [1] - xlate: Translate openflow
> clone into odp sample action.
>
> Because of which, the openstack CI job for OVN is broken.
> It works once I revert this commit.

Thanks for reporting. This change suppose to be used with the
corresponding kernel
datapath changes that is currently under review on the netdev mailing list.

Given that user space changes and kernel datapath change may not always line up,
this situation is not ideal.  To solve the issue reported, I have
posted a patch series,
that should solve the problem reported.

https://mail.openvswitch.org/pipermail/ovs-dev/2017-March/329625.html

Please let me know if this is not the case after they are pushed. Thanks.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] clone action is broken in the master

2017-03-09 Thread Numan Siddique
On Fri, Mar 10, 2017 at 4:35 AM, Andy Zhou  wrote:

> On Thu, Mar 9, 2017 at 2:34 AM, Numan Siddique 
> wrote:
> > Hi Andy and All,
> >
> > The clone action is broken after the commit [1] - xlate: Translate
> openflow
> > clone into odp sample action.
> >
> > Because of which, the openstack CI job for OVN is broken.
> > It works once I revert this commit.
>
> Thanks for reporting. This change suppose to be used with the
> corresponding kernel
> datapath changes that is currently under review on the netdev mailing list.
>
> Given that user space changes and kernel datapath change may not always
> line up,
> this situation is not ideal.  To solve the issue reported, I have
> posted a patch series,
> that should solve the problem reported.
>
> https://mail.openvswitch.org/pipermail/ovs-dev/2017-March/329625.html
>
> Please let me know if this is not the case after they are pushed. Thanks.
>

​Thanks. I tested the patches and it works.

Numan
​
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] clone action is broken in the master

2017-03-10 Thread Andy Zhou
On Thu, Mar 9, 2017 at 7:31 PM, Numan Siddique  wrote:
>
>
> On Fri, Mar 10, 2017 at 4:35 AM, Andy Zhou  wrote:
>>
>> On Thu, Mar 9, 2017 at 2:34 AM, Numan Siddique 
>> wrote:
>> > Hi Andy and All,
>> >
>> > The clone action is broken after the commit [1] - xlate: Translate
>> > openflow
>> > clone into odp sample action.
>> >
>> > Because of which, the openstack CI job for OVN is broken.
>> > It works once I revert this commit.
>>
>> Thanks for reporting. This change suppose to be used with the
>> corresponding kernel
>> datapath changes that is currently under review on the netdev mailing
>> list.
>>
>> Given that user space changes and kernel datapath change may not always
>> line up,
>> this situation is not ideal.  To solve the issue reported, I have
>> posted a patch series,
>> that should solve the problem reported.
>>
>> https://mail.openvswitch.org/pipermail/ovs-dev/2017-March/329625.html
>>
>> Please let me know if this is not the case after they are pushed. Thanks.
>
>
> Thanks. I tested the patches and it works.
>
> Numan
>
Thanks for testing it.  Just pushed them.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev