Re: [ovs-dev] [PATCH] ovn: expose c validator to python

2016-07-13 Thread Aaron Rosen
Yea I totally agree. In the latest patch I sent out it skips these tests if the user doesn't have the ovs python lib installed. I'll try and see what I can do to fix this. On Wed, Jul 13, 2016 at 10:56 AM, Ben Pfaff wrote: > On Mon, Jul 04, 2016 at 12:17:54PM -0700, Aaron Rosen wrote: > > On S

Re: [ovs-dev] [PATCH] ovn: expose c validator to python

2016-07-13 Thread Ben Pfaff
On Mon, Jul 04, 2016 at 12:17:54PM -0700, Aaron Rosen wrote: > On Sun, Jul 3, 2016 at 12:04 PM, Ben Pfaff wrote: > > > On Thu, Jun 30, 2016 at 02:27:19PM -0700, Aaron Rosen wrote: > > > This patch exposes the c function expr_parse_string() to be called via > > > python. The motivation for this is

Re: [ovs-dev] [PATCH] ovn: expose c validator to python

2016-07-04 Thread Aaron Rosen
Thanks for the review. Response inline On Sun, Jul 3, 2016 at 12:04 PM, Ben Pfaff wrote: > On Thu, Jun 30, 2016 at 02:27:19PM -0700, Aaron Rosen wrote: > > This patch exposes the c function expr_parse_string() to be called via > > python. The motivation for this is so that clients interfacing wi

Re: [ovs-dev] [PATCH] ovn: expose c validator to python

2016-07-03 Thread Ben Pfaff
On Thu, Jun 30, 2016 at 02:27:19PM -0700, Aaron Rosen wrote: > This patch exposes the c function expr_parse_string() to be called via > python. The motivation for this is so that clients interfacing with > ovn can call this method in order to validate the data they are writting > to ovn. > > Previ

Re: [ovs-dev] [PATCH] ovn: expose c validator to python

2016-07-01 Thread Aaron Rosen
; Date: 06/30/2016 04:29 PM > > Subject: [ovs-dev] [PATCH] ovn: expose c validator to python > > Sent by: "dev" > > > > This patch exposes the c function expr_parse_string() to be called via > > python. The motivation for this is so that clients interfacing wit

Re: [ovs-dev] [PATCH] ovn: expose c validator to python

2016-07-01 Thread Ryan Moats
"dev" wrote on 06/30/2016 04:27:19 PM: > From: Aaron Rosen > To: dev@openvswitch.org > Cc: Aaron Rosen > Date: 06/30/2016 04:29 PM > Subject: [ovs-dev] [PATCH] ovn: expose c validator to python > Sent by: "dev" > > This patch exposes the c f

Re: [ovs-dev] [PATCH] ovn: expose c validator to python

2016-06-30 Thread Aaron Rosen
Here's the v2 version of this (Sorry this is a new email message I was hoping that this would be grouped together with the previous thread). This patch adds python3 support and hooks in the testcases with autotest. I was planning on reusing the existing test inputs that test-ovn parse-expr uses b

[ovs-dev] [PATCH] ovn: expose c validator to python

2016-06-30 Thread Aaron Rosen
This patch exposes the c function expr_parse_string() to be called via python. The motivation for this is so that clients interfacing with ovn can call this method in order to validate the data they are writting to ovn. Previously, there were several bugs in the neutron/ovn integration that went u

Re: [ovs-dev] [PATCH] ovn: expose c validator to python

2016-06-20 Thread Aaron Rosen
Hi, I figured I should post a WIP patch for this work just to gather some feedback if anyone has any. I still need to figure out how to test/add python3 support. I'm not sure how to install the python ovs module so that I can import it via python3. If anyone knows how to do that via the source tr

[ovs-dev] [PATCH] ovn: expose c validator to python

2016-06-20 Thread Aaron Rosen
This patch exposes the c function expr_parse_string() to be called via python. The motivation for this is so that clients interfacing with ovn can call this method in order to validate the data they are writting to ovn. Previously, there were several bugs in the neutron/ovn integration that went u