Re: [ovs-dev] [PATCH] [OVN] Add QoS to NB schema

2015-07-17 Thread Ben Pfaff
[Adding back ovs-dev, hope that's OK] Hmm, shutting down your QoS effort is not my goal. What if we consider this patch as the start of an RFC series for the NB part of QoS and then you work on the implications of pushing it down through ovn-northd into SB and ovn-controller/OpenFlow? That's whe

Re: [ovs-dev] [PATCH] [OVN] Add QoS to NB schema

2015-07-16 Thread Ben Pfaff
On Thu, Jul 16, 2015 at 09:47:19PM +0300, Gal Sagie wrote: > If i understand your concern right, you worry that the NB schema is getting > too big. > I can't think of a better way to do this both use full and smaller, if you > feel that we don't want to support > QoS at this point, thats fine. but

Re: [ovs-dev] [PATCH] [OVN] Add QoS to NB schema

2015-07-16 Thread Gal Sagie
These features are not yet designed in OpenStack API, for Liberty (the next release) its going to be only bandwidth limit, but they certainly will be added to the next release as there is a high demand for them. I can start trying to propose them, but i think at this point its a waste of time if th

Re: [ovs-dev] [PATCH] [OVN] Add QoS to NB schema

2015-07-16 Thread Ben Pfaff
Could you extend the proposal to include those features then? At the moment it is a skeleton. On Thu, Jul 16, 2015 at 02:11:12AM +0300, Gal Sagie wrote: > Yes, there are currently plans for marking (DSCP) and flow classification > profiles. > And other ideas like connections limiting. > > On Wed

Re: [ovs-dev] [PATCH] [OVN] Add QoS to NB schema

2015-07-15 Thread Gal Sagie
Yes, there are currently plans for marking (DSCP) and flow classification profiles. And other ideas like connections limiting. On Wed, Jul 15, 2015 at 8:38 PM, Ben Pfaff wrote: > On Sun, Jul 12, 2015 at 09:21:41AM +0300, Gal Sagie wrote: > > Add QoS profile and QoS rule tables and explanations >

Re: [ovs-dev] [PATCH] [OVN] Add QoS to NB schema

2015-07-15 Thread Ben Pfaff
On Sun, Jul 12, 2015 at 09:21:41AM +0300, Gal Sagie wrote: > Add QoS profile and QoS rule tables and explanations > to the NB schema. > Add specific bandwidth limit rule configurations > in the document > > Signed-off-by: Gal Sagie This is a lot of schema to support just a little bit of function

[ovs-dev] [PATCH] [OVN] Add QoS to NB schema

2015-07-11 Thread Gal Sagie
Add QoS profile and QoS rule tables and explanations to the NB schema. Add specific bandwidth limit rule configurations in the document Signed-off-by: Gal Sagie --- ovn/ovn-nb.ovsschema | 25 ++ ovn/ovn-nb.xml | 72 2 fil