[openstack-dev] [nova] Non-priority feature freeze and FFEs
mriedem at linux.vnet.ibm.com
Mon Aug 29 14:42:57 UTC 2016
On 8/29/2016 6:27 AM, tiendc at vn.fujitsu.com wrote:
> Hi Matt, Dan, Andrew,
> @Matt: Hope you had a nice vacation.
> For the feature Nova serial console support for Ironic , there are some good update from Ironic side. All our Ironic-side works  have been done, currently there is only the nova patch that needs to review.
> Last week, I contacted Andrew and Dan for reviewing it. But both Andrew and Dan said they didn't notice you've removed -2 from the patch. So, Matt, can you notify the Nova core team about that so Andrew and Dan can review it again?
>  https://blueprints.launchpad.net/nova/+spec/ironic-serial-console-support
>  https://review.openstack.org/#/c/328157/ (Nova patch, in review)
>  https://review.openstack.org/#/c/319505/ (Ironic spec, merged)
>  https://review.openstack.org/#/c/328168/ (Ironic patch, merged)
>  https://review.openstack.org/#/c/293873/ (Ironic patch, merged)
> Thanks and Regards
> -----Original Message-----
> From: Matt Riedemann [mailto:mriedem at linux.vnet.ibm.com]
> Sent: Thursday, July 07, 2016 3:16 AM
> To: openstack-dev at lists.openstack.org
> Subject: Re: [openstack-dev] [nova] Non-priority feature freeze and FFEs
> On 7/5/2016 2:14 AM, tiendc at vn.fujitsu.com wrote:
>> Hi folks,
>> I want to give more information about our nova patch for bp ironic-serial-console-support. The whole feature needs work to be done in Nova and Ironic. The nova bp  has been approved, and the Ironic spec  has been merged.
>> This nova patch  is simple, we got some reviews by some Nova and Ironic core reviewers. The depended patches in Ironic are  which  will get merged soon and  is in review progress.
>> Hope Nova core team considers adding this case to the exception list.
>> ort (Nova bp, approved by dansmith) 
>> https://review.openstack.org/#/c/319505/ (Ironic spec, merged)
>>  https://review.openstack.org/#/c/328157/ (Nova patch, in review)
>>  https://review.openstack.org/#/c/328168/ (Ironic patch 1st, got
>> two +2, will get merged soon) 
>> https://review.openstack.org/#/c/293873/ (Ironic patch 2nd, in
>> Thanks and Regards
>> Dao Cong Tien
> When I looked last week the nova change was dependent on multiple ironic patches which weren't merged yet, so it wasn't ready to go for the non-priority feature freeze. The ironic changes are all merged yet either when we were going over FFE candidates. So this is going to have to wait for Ocata.
Sorry, the -2 removal was a mistake, that happened when I removed myself
as a reviewer from that change since I wasn't actively reviewing it but
it kept changing, so I wanted to turn off the email noise I was getting
I've re-applied the -2 on 328157 since we're long past non-priority
feature freeze and the normal feature freeze is this week (9/1) and need
to focus on closing out priority work.
More information about the OpenStack-dev