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

Kyle Mestery mestery at mestery.com
Mon Aug 17 16:57:03 UTC 2015


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 <ihrachys at redhat.com>
wrote:

> -----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 Fri and then merge.
>
> We had a git conflict on Fri though, so today I respin the patch
> again, hoping that it will either get more reviews or it's merged
> before we hit another conflict that can be inflicted by any new db
> migration.
>
> Ihar
>
> 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/
> >
> > Since it's a merge patch, gerrit fails to show the whole diff that
> > it introduces into master. To get over it, fetch the patch:
> >
> > $ git review -d 212170
> >
> > and then check the difference:
> >
> > $ git fetch origin && git diff origin/master...
> >
> > I think we should stick to review process originally suggested at
> > [1]. Specifically, since it's not reasonable to expect the whole
> > feature branch to be reviewed by a single person, I hope multiple
> > people will assign themselves to the job and split the pieces to
> > review based on devref document that describes the feature [2]
> > (Note that a new RPC push/pull mechanism is described in a separate
> > devref section [3]).
> >
> > Note that we don't expect to tackle all review comments, however
> > tiny, in feature/qos. We are happy to handle major flaws there, but
> > for minor stuff, it's good to proceed in master. Nevertheless we
> > are happy to get minors too and collect them for post-merge.
> >
> > Things we have in the tree:
> >
> > - server: QoS API extension; QoS core resource extension; QoS ML2
> > extension driver; QoS versioned objects + base for new objects;
> > QoS supported rule types mechanism for ML2; QoS notification
> > drivers mechanism to update SDN controllers;
> >
> > - RPC: new push/pull mechanisms for versioned objects to propagate
> > QoS objects into the agents;
> >
> > - agent side: new L2 agent extensions mechanism, integrated into
> > OVS and SR-IOV agents; QoS l2 agent extension; OVS and SR-IOV QoS
> > drivers; ovs_lib and pci_lib changes.
> >
> > I suggest to split review into following logical pieces:
> >
> > - API controller + service plugin + API tests; - Versioned objects:
> > neutron.objects.* - ML2: supported_qos_rule_types mechanism,
> > extension driver, update for get_device_details payload; - RPC
> > mechanism (push/pull), resource manager, registries + notification
> > drivers integration; - l2 extensions (manager, base interface) +
> > qos extension; - OVS integration with extension manager + OVS QoS
> > driver + ovs_lib changes; - SR-IOV agent integration with extension
> > manager + SR-IOV QoS driver + pci_lib changes; - functional tests.
> >
> > We will also need to update the spec:
> > https://review.openstack.org/#/c/199112/
> >
> > Included test coverage:
> >
> > - unit tests; - API tests; - functional tests (more scenarios to
> > come in master); - fullstack tests [4] (not in the tree since we
> > need to merge client and base fullstack patches first).
> >
> > We have client patches up for review [5][6] and expect them to go
> > in after merge of server component.
> >
> > We hope that we'll make fullstack in before closing the blueprint
> > in this cycle.
> >
> > [1]:
> > http://lists.openstack.org/pipermail/openstack-dev/2015-July/069188.ht
> ml
> >
> >
> [2]:
> > http://git.openstack.org/cgit/openstack/neutron/tree/doc/source/devref
> /q
> >
> >
> uality_of_service.rst?h=feature/qos
> > [3]:
> > http://git.openstack.org/cgit/openstack/neutron/tree/doc/source/devref
> /r
> >
> >
> pc_callbacks.rst?h=feature/qos
> > [4]: https://review.openstack.org/202492 [5]:
> > https://review.openstack.org/189655 [6]:
> > https://review.openstack.org/198277 [7]:
> > https://review.openstack.org/202061
> >
> > ______________________________________________________________________
> ____
> >
> >
> OpenStack Development Mailing List (not for usage questions)
> > Unsubscribe:
> > OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> >
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v2
>
> iQEcBAEBCAAGBQJV0fYnAAoJEC5aWaUY1u57rtsH/iaQ5HCRuFDbhFsFAkGeW/hB
> gn/pR9lmx/hXUIkEWfGPIsgtEnuA8nIQ3knwLfvkrPxR60YHkCK5YeRDaTVd0IQb
> oV5njw3eMJablTtquPybSzUljfx+oCQ2pxwhXgWAcj5KucksXLcvC+lkfk5uQ1OT
> iFum1jCmZ+7Te8uPdjkgGxxxpLjnJJs0Na6i+GhRppRc/HK77jM31MggfA3dJw9y
> cdB0JN3w2tT4wbjtmtCsVgKVWeDuuKXlnZjmI0Do1Qm1YDC0NNPLNGcBTV70vyPB
> B8lGyk9kvtbzSQ701T3LEp8hRIL6Oto8cHRrt3jkfygrlXPQL8x1pwtjSD59bXU=
> =s4FB
> -----END PGP SIGNATURE-----
>
> __________________________________________________________________________
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20150817/e2595cc5/attachment.html>


More information about the OpenStack-dev mailing list