<div dir="ltr">Thanks Ruby to point this out.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 3, 2016 at 3:25 PM, Haomeng, Wang <span dir="ltr"><<a href="mailto:wanghaomeng@gmail.com" target="_blank">wanghaomeng@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>Hi Ruby,<br><br></div>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 reviewed.<br><br></div>-- Haomeng<br><br><br></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Thu, Mar 3, 2016 at 3:07 AM, Ruby Loo <span dir="ltr"><<a href="mailto:rlooyahoo@gmail.com" target="_blank">rlooyahoo@gmail.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="ltr"><div><div>Hi,<br><br></div><div>Ironic'ers, please remember to follow the RFE process; especially the cores.<br><br></div>I noticed that a patch [1] got merged yesterday. The patch was associated with an RFE [2] 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.<br><br></div>As a reminder, the RFE process is documented [3].<br><div><br></div><div>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.<br><br></div><div>Should we revert the patch [1] 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.)<br></div><div><br></div><div>--ruby<br><br></div><div><br>[1] <a href="https://review.openstack.org/#/c/264005/" target="_blank">https://review.openstack.org/#/c/264005/</a><br>[2] <a href="https://bugs.launchpad.net/ironic/+bug/1530626" target="_blank">https://bugs.launchpad.net/ironic/+bug/1530626</a><br>[3] <a href="http://docs.openstack.org/developer/ironic/dev/code-contribution-guide.html#adding-new-features" target="_blank">http://docs.openstack.org/developer/ironic/dev/code-contribution-guide.html#adding-new-features</a><br></div></div>
<br></div></div>__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br></div>
</blockquote></div><br></div>