<p dir="ltr"><br>
Le 14 août 2014 22:02, "Michael Still" <<a href="mailto:mikal@stillhq.com">mikal@stillhq.com</a>> a écrit :<br>
><br>
> Hi.<br>
><br>
> We're rapidly approaching j-3, so I want to remind people of the<br>
> current reviews that are high priority. The definition of high<br>
> priority I am using here is blueprints that are marked high priority<br>
> in launchpad that have outstanding code for review -- I am sure there<br>
> are other reviews that are important as well, but I want us to try to<br>
> land more blueprints than we have so far. These are listed in the<br>
> order they appear in launchpad.<br>
><br>
> == Compute Manager uses Objects (Juno Work) ==<br>
><br>
> <a href="https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/compute-manager-objects-juno,n,z">https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/compute-manager-objects-juno,n,z</a><br>

><br>
> This is ongoing work, but if you're after some quick code review<br>
> points they're very easy to review and help push the project forward<br>
> in an important manner.<br>
><br>
> == Move Virt Drivers to use Objects (Juno Work) ==<br>
><br>
> I couldn't actually find any code out for review for this one apart<br>
> from <a href="https://review.openstack.org/#/c/94477/">https://review.openstack.org/#/c/94477/</a>, is there more out there?<br>
><br>
> == Add a virt driver for Ironic ==<br>
><br>
> This one is in progress, but we need to keep going at it or we wont<br>
> get it merged in time.<br>
><br>
> * <a href="https://review.openstack.org/#/c/111223/">https://review.openstack.org/#/c/111223/</a> was approved, but a rebased<br>
> ate it. Should be quick to re-approve.<br>
> * <a href="https://review.openstack.org/#/c/111423/">https://review.openstack.org/#/c/111423/</a><br>
> * <a href="https://review.openstack.org/#/c/111425/">https://review.openstack.org/#/c/111425/</a><br>
> * ...there are more reviews in this series, but I'd be super happy to<br>
> see even a few reviewed<br>
><br>
> == Create Scheduler Python Library ==<br>
><br>
> * <a href="https://review.openstack.org/#/c/82778/">https://review.openstack.org/#/c/82778/</a><br>
> * <a href="https://review.openstack.org/#/c/104556/">https://review.openstack.org/#/c/104556/</a><br>
><br>
> (There are a few abandoned patches in this series, I think those two<br>
> are the active ones but please correct me if I am wrong).<br>
></p>
<p dir="ltr">Nope that's OK, those are the only one implementing the spec. We had a +2 on 82778 but it needed a rebase, FYI.<br></p>
<p dir="ltr">> == VMware: spawn refactor ==<br>
><br>
> * <a href="https://review.openstack.org/#/c/104145/">https://review.openstack.org/#/c/104145/</a><br>
> * <a href="https://review.openstack.org/#/c/104147/">https://review.openstack.org/#/c/104147/</a> (Dan Smith's -2 on this one<br>
> seems procedural to me)<br>
> * <a href="https://review.openstack.org/#/c/105738/">https://review.openstack.org/#/c/105738/</a><br>
> * ...another chain with many more patches to review<br>
><br>
> Thanks,<br>
> Michael<br>
><br>
> --<br>
> Rackspace Australia<br>
><br>
> _______________________________________________<br>
> OpenStack-dev mailing list<br>
> <a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</p>