<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">2016-03-03 11:01 GMT+01:00 Lucas Alvares Gomes <span dir="ltr"><<a href="mailto:lucasagomes@gmail.com" target="_blank">lucasagomes@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Hi,<br>
<br>
> Ironic'ers, please remember to follow the RFE process; especially the cores.<br>
><br>
> I noticed that a patch [1] got merged yesterday. The patch was associated<br>
> with an RFE [2] that hadn't been approved yet :-( What caught my eye was<br>
> that the commit message didn't describe the actual API change so I took a<br>
> quick look at the (RFE) bug and it wasn't documented there either.<br>
><br>
> As a reminder, the RFE process is documented [3].<br>
><br>
> Spec cores need to try to be more timely wrt specs (I admit, I am guilty).<br>
> And folks, especially cores, ought to take more care when reviewing.<br>
> Although I do feel like there are too many things that a reviewer needs to<br>
> keep in mind.<br>
><br>
> Should we revert the patch [1] for now? (Disclaimer. I haven't looked at the<br>
> patch itself. But I don't think I should have to, to know what the API<br>
> change is.)<br>
><br>
<br>
</span>Thanks for calling it out Ruby, that's unfortunate that the patch was<br>
merged without the RFE being approved. About reverting the patch I<br>
think we shouldn't do that now because the patch is touching the API<br>
and introducing a new microversion to it.<br></blockquote><div><br></div><div>Exactly. I've -2'ed the revert, as removing API version is even worse than landing a change without an RFE approved. Let us make sure to approve RFE asap, and then adjust the code according to it.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
And yes, as reviewers let's try to improve our process. We probably<br>
should talk about how we can do it in the next upstream meeting.<br>
<br>
Cheers,<br>
Lucas<br>
<div class="HOEnZb"><div class="h5"><br>
__________________________________________________________________________<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>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature"><div dir="ltr"><div>--<br></div>-- Dmitry Tantsur<br><div>--<br></div></div></div>
</div></div>