Re: [ovs-dev] [PATCH v6 1/1] ovn: Apply ACL changes to existing connections.

2016-07-20 Thread Justin Pettit
> On Jul 20, 2016, at 2:26 PM, Russell Bryant wrote: > > After some help rebasing and re-testing from Babu Shanmugam, I went through > this again, addressed comments, and applied this to master. Great! Glad to see it finally merged. Thanks for being patient with me.

Re: [ovs-dev] [PATCH v6 1/1] ovn: Apply ACL changes to existing connections.

2016-07-20 Thread Russell Bryant
On Thu, Jul 7, 2016 at 1:13 PM, Justin Pettit wrote: > > > On Jun 30, 2016, at 10:14 PM, Russell Bryant wrote: > > > > diff --git a/ovn/northd/ovn-northd.8.xml b/ovn/northd/ovn-northd.8.xml > > index 260cc14..d2bddcb 100644 > > --- a/ovn/northd/ovn-northd.8.xml

Re: [ovs-dev] [PATCH v6 1/1] ovn: Apply ACL changes to existing connections.

2016-07-07 Thread Russell Bryant
On Thu, Jul 7, 2016 at 12:13 PM, Justin Pettit wrote: > > > On Jun 30, 2016, at 10:14 PM, Russell Bryant wrote: > > > > diff --git a/ovn/northd/ovn-northd.8.xml b/ovn/northd/ovn-northd.8.xml > > index 260cc14..d2bddcb 100644 > > ---

Re: [ovs-dev] [PATCH v6 1/1] ovn: Apply ACL changes to existing connections.

2016-07-07 Thread Justin Pettit
> On Jun 30, 2016, at 10:14 PM, Russell Bryant wrote: > > diff --git a/ovn/northd/ovn-northd.8.xml b/ovn/northd/ovn-northd.8.xml > index 260cc14..d2bddcb 100644 > --- a/ovn/northd/ovn-northd.8.xml > +++ b/ovn/northd/ovn-northd.8.xml > > > @@ -282,20 +299,37 @@ > > >

[ovs-dev] [PATCH v6 1/1] ovn: Apply ACL changes to existing connections.

2016-06-30 Thread Russell Bryant
Prior to this commit, once a connection had been committed to the connection tracker, the connection would continue to be allowed, even if the policy defined in the ACL table changed. This patch changes the implementation so that existing connections are affected by policy changes. The