<div dir="ltr"><span></span>Hijacked from ' [openstack-dev] [ironic] Remember to follow RFE process' thread:<br><div><br><div><span></span><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class=""><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span>
> 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></span><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><span class=""><div> </div></span></div></div></div></blockquote><div><br></div><div>This brings up another issue, which I recall discussing before. Did we decide that we'd never revert something that touches the API/microversion? It might be good to have guidelines on this if we don't already. IF the API is incorrect? If the API could be improved? If the API was only in master for eg 48 hours?<br><br></div><div>--ruby <br></div></div></div></div></div></div>