Re: [openstack-dev] [neutron][qos] request to merge feature/qos back into master

2015-08-17 Thread Kyle Mestery
I've pushed the patch into the merge queue now. Any nits people find at this point we'll address post merge. Awesome work QoS team! On Mon, Aug 17, 2015 at 9:56 AM, Ihar Hrachyshka ihrac...@redhat.com wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA256 So the patch in question sit there

Re: [openstack-dev] [neutron][qos] request to merge feature/qos back into master

2015-08-17 Thread Ihar Hrachyshka
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 So the patch in question sit there for some time, and honestly, I haven't seen much interest from reviewers to take a look at it, apart from Assaf who played with the code and reported a bunch of minor issues . I think Kyle's plan was to wait until

Re: [openstack-dev] [neutron][qos] request to merge feature/qos back into master

2015-08-14 Thread Miguel Angel Ajo
and here's the video: http://www.ajo.es/post/126667247769/neutron-qos-service-plugin Cheers, Miguel Ángel. Miguel Angel Ajo wrote: I owe you all a video of the feature to show how does it work. I was supposed to deliver today, but I've been partly sick during today, The script is ready, I

Re: [openstack-dev] [neutron][qos] request to merge feature/qos back into master

2015-08-13 Thread Ihar Hrachyshka
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 08/13/2015 06:54 AM, Andreas Jaeger wrote: On 08/12/2015 09:55 PM, Ihar Hrachyshka wrote: Hi all, with great pleasure, I want to request a coordinated review for merging feature/qos branch back to master:

Re: [openstack-dev] [neutron][qos] request to merge feature/qos back into master

2015-08-13 Thread Miguel Angel Ajo
I owe you all a video of the feature to show how does it work. I was supposed to deliver today, but I've been partly sick during today, The script is ready, I just have to record and share, hopefully happening tomorrow (Friday). Best, Miguel Ángel Ihar Hrachyshka wrote: -BEGIN PGP SIGNED

Re: [openstack-dev] [neutron][qos] request to merge feature/qos back into master

2015-08-12 Thread Andreas Jaeger
On 08/12/2015 09:55 PM, Ihar Hrachyshka wrote: Hi all, with great pleasure, I want to request a coordinated review for merging feature/qos branch back to master: https://review.openstack.org/#/c/212170/ Great! Please send also a patch for project-config to remove the special handling of

Re: [openstack-dev] [neutron][qos] request to merge feature/qos back into master

2015-08-12 Thread Kevin Benton
If you want a quick visual diff of this, you can click on Files changed here: https://github.com/openstack/neutron/compare/feature/qos On Wed, Aug 12, 2015 at 12:55 PM, Ihar Hrachyshka ihrac...@redhat.com wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Hi all, with great pleasure, I