On 09/19/2017 01:26 PM, Andy Zhou wrote:
On Tue, Sep 19, 2017 at 9:55 AM, Greg Rose <[email protected]> wrote:
On 09/12/2017 12:49 PM, Andy Zhou wrote:

When translating actions within open flow clone, actions generated
by finish_freezeing() should also be enclosed within the datapath
clone netlink encoding.

Signed-off-by: Andy Zhou <[email protected]>


Andy,

I am reviewing and testing your patches.  I have applied them to my private
github repository
on a branch named test-813027-35.

https://github.com/gvrose8192/ovs-experimental/tree/test-813027-35

However, the Travis 'TESTSUITE=1 KERNEL=3.16.46 build fails:

https://travis-ci.org/gvrose8192/ovs-experimental/jobs/277364409

Have you noticed this as well?

No. It passed my local test, and passed travis test from my private
branch (just rebased this morning)

https://github.com/azhou-nicira/ovs-review/tree/patch_port

https://travis-ci.org/azhou-nicira/ovs-review/builds/277412765

(The --disable-ssl build is slow for some reason, same as master).

May be this is caused by travis running slow for some reason?

Did your local test pass?


Yes, I just tried on a VM running Centos 7.3 with the 4.9 kernel and it passed 
there.

/shrug?

OK, I'll continue with review then.

Thanks!

- Greg
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to