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
wrote:
> -BEGIN PGP SIGNED MESSAGE-
> Hash: SHA256
>
> So the patch in question sit there for some time, a
-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
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 ju
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 M
-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:
>>
>> https://review.open
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 tha
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
wrote:
> -BEGIN PGP SIGNED MESSAGE-
> Hash: SHA256
>
> Hi all,
>
> with great pleasure, I want to req
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA256
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/
Since it's a merge patch, gerrit fails to show the whole diff that it
introduces into maste