[openstack-dev] [ironic] Remember to follow RFE process
wanghaomeng at gmail.com
Thu Mar 3 07:33:36 UTC 2016
Thanks Ruby to point this out.
On Thu, Mar 3, 2016 at 3:25 PM, Haomeng, Wang <wanghaomeng at gmail.com> wrote:
> Hi Ruby,
> Yes, just noticed that RFE is in 'Wishlist' status now, sorry for missing
> the bug status yesterday, so we need to follow the process, and I will help
> to revert the patch and get it back to review again once the REF is
> -- Haomeng
> On Thu, Mar 3, 2016 at 3:07 AM, Ruby Loo <rlooyahoo at gmail.com> wrote:
>> Ironic'ers, please remember to follow the RFE process; especially the
>> I noticed that a patch  got merged yesterday. The patch was associated
>> with an RFE  that hadn't been approved yet :-( What caught my eye was
>> that the commit message didn't describe the actual API change so I took a
>> quick look at the (RFE) bug and it wasn't documented there either.
>> As a reminder, the RFE process is documented .
>> Spec cores need to try to be more timely wrt specs (I admit, I am
>> guilty). And folks, especially cores, ought to take more care when
>> reviewing. Although I do feel like there are too many things that a
>> reviewer needs to keep in mind.
>> Should we revert the patch  for now? (Disclaimer. I haven't looked at
>> the patch itself. But I don't think I should have to, to know what the API
>> change is.)
>>  https://review.openstack.org/#/c/264005/
>>  https://bugs.launchpad.net/ironic/+bug/1530626
>> OpenStack Development Mailing List (not for usage questions)
>> OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the OpenStack-dev