<div dir="ltr"><div>Here are some examples of proposing changes prior to being merged in master [0][1][2][3][4]. [0] is a perfect example of why this isn't a good process. A change was proposed to stable/6.0 before master was merged, and now the change to master needs to be reworked based on review feedback. Premature backporting just creates unnecessary additional work. I'd also like to give a friendly reminder to make sure we maintain the Change-Id and author of any change we backport.<br></div><div><br></div><div>The wiki [5] has also been updated to make this explicit.<br><div><div><br></div><div>[0] <a href="https://review.openstack.org/#/q/Ief8186006386af8ae7e40cffeeaeef5a5c0f3c70,n,z">https://review.openstack.org/#/q/Ief8186006386af8ae7e40cffeeaeef5a5c0f3c70,n,z</a></div><div>[1] <a href="https://review.openstack.org/#/q/I4c94bb03501f4238ead2378cf504485b7d67b236,n,z">https://review.openstack.org/#/q/I4c94bb03501f4238ead2378cf504485b7d67b236,n,z</a></div><div>[2] <a href="https://review.openstack.org/#/q/Ic15a3bfb6238e4281b06aae0a3f9fe4abf96590d,n,z">https://review.openstack.org/#/q/Ic15a3bfb6238e4281b06aae0a3f9fe4abf96590d,n,z</a></div><div>[3] <a href="https://review.openstack.org/#/q/I7ab6dc2341821c3b82ef3d3ac63b64a5a9958fa9,n,z">https://review.openstack.org/#/q/I7ab6dc2341821c3b82ef3d3ac63b64a5a9958fa9,n,z</a></div><div>[4] <a href="https://review.openstack.org/#/q/Iff947f0053577f19441c04101e5a35a7820e40a0,n,z">https://review.openstack.org/#/q/Iff947f0053577f19441c04101e5a35a7820e40a0,n,z</a></div></div></div><div>[5] <a href="https://wiki.openstack.org/wiki/Fuel/How_to_contribute#Backport_bugfixes_to_stable_release_series">https://wiki.openstack.org/wiki/Fuel/How_to_contribute#Backport_bugfixes_to_stable_release_series</a></div><div><br></div><div>Thanks,</div><div>Ryan</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 10, 2015 at 4:20 AM, Tomasz Napierala <span dir="ltr"><<a href="mailto:tnapierala@mirantis.com" target="_blank">tnapierala@mirantis.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
> On 09 Mar 2015, at 18:21, Ryan Moe <<a href="mailto:rmoe@mirantis.com">rmoe@mirantis.com</a>> wrote:<br>
><br>
> Hi All,<br>
><br>
> I've noticed a few times recently where reviews have been abandoned by people who were not the original authors. These reviews were only days old and there was no prior notice or discussion. This is both rude and discouraging to contributors. Reasons for abandoning should be discussed on the review and/or in email before any action is taken.<br>
<br>
</span>Hi Ryan,<br>
<br>
I was trying to find any examples, and the only one I see is:<br>
<a href="https://review.openstack.org/#/c/152674/" target="_blank">https://review.openstack.org/#/c/152674/</a><br>
<br>
I spoke to Bogdan and he agreed it was not proper way to do it, but they were in a rush - I know, it does not explain anything really.<br>
<br>
Do you have any other examples? I’d like to clarify them<br>
<br>
Regards,<br>
<span class="HOEnZb"><font color="#888888">--<br>
Tomasz 'Zen' Napierala<br>
Sr. OpenStack Engineer<br>
<a href="mailto:tnapierala@mirantis.com">tnapierala@mirantis.com</a><br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
<br>
<br>
<br>
<br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div>