[openstack-dev] [nova] Review priorities as we approach juno-3
Michael Still
mikal at stillhq.com
Thu Aug 14 20:53:41 UTC 2014
On Fri, Aug 15, 2014 at 6:37 AM, Dan Smith <dms at danplanet.com> wrote:
>> == Move Virt Drivers to use Objects (Juno Work) ==
>>
>> I couldn't actually find any code out for review for this one apart
>> from https://review.openstack.org/#/c/94477/, is there more out there?
>
> This was an umbrella one to cover a bunch of virt driver objects work
> done early in the cycle. Much of that is done, I haven't gone looking
> for anything to see if there are any obvious things to include under
> this anymore, but I'll try to do that.
Thanks, I'd appreciate that. If its all done, we should mark it implemented.
>> == Add a virt driver for Ironic ==
>>
>> This one is in progress, but we need to keep going at it or we wont
>> get it merged in time.
>>
>> * https://review.openstack.org/#/c/111223/ was approved, but a rebased
>> ate it. Should be quick to re-approve.
>> * https://review.openstack.org/#/c/111423/
>> * https://review.openstack.org/#/c/111425/
>> * ...there are more reviews in this series, but I'd be super happy to
>> see even a few reviewed
>
> I've been reviewing this pretty heavy and I think that it's just taking
> a while to make changes given the roundabout way they're getting done
> first in Ironic. I'm pretty confident that this one will be okay.
Yep, I appreciate your focus on this one -- as I am sure the ironic
people do too. If another core was available to pair up with you on
these we might be able to get them to land faster. I was doing that
for a while, but I haven't had time in the last week or so.
>> == VMware: spawn refactor ==
>>
>> * https://review.openstack.org/#/c/104145/
>> * https://review.openstack.org/#/c/104147/ (Dan Smith's -2 on this one
>> seems procedural to me)
>
> Yep, we're just trying to get MineSweeper votes on them before letting
> them in. We already had one refactor go in without a minesweeper run
> that ... broke minesweeper :)
Is there a way for the minesweeper people to manually kick off runs
against specific reviews? Doing that might unblock this faster than
rechecking and hoping.
Michael
--
Rackspace Australia
More information about the OpenStack-dev
mailing list