<div dir="ltr"><div><div>I'll keep it in mind not to create unnecessary backports, although I really find it more convenient to do them once I submit changes to master for review. I apologize for [4], it indeed was wrong and it won't happen again.<br><br></div>Regards,<br></div>Bartłomiej<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 11, 2015 at 12:36 AM, Ryan Moe <span dir="ltr"><<a href="mailto:rmoe@mirantis.com" target="_blank">rmoe@mirantis.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>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" target="_blank">https://review.openstack.org/#/q/Ief8186006386af8ae7e40cffeeaeef5a5c0f3c70,n,z</a></div><div>[1] <a href="https://review.openstack.org/#/q/I4c94bb03501f4238ead2378cf504485b7d67b236,n,z" target="_blank">https://review.openstack.org/#/q/I4c94bb03501f4238ead2378cf504485b7d67b236,n,z</a></div><div>[2] <a href="https://review.openstack.org/#/q/Ic15a3bfb6238e4281b06aae0a3f9fe4abf96590d,n,z" target="_blank">https://review.openstack.org/#/q/Ic15a3bfb6238e4281b06aae0a3f9fe4abf96590d,n,z</a></div><div>[3] <a href="https://review.openstack.org/#/q/I7ab6dc2341821c3b82ef3d3ac63b64a5a9958fa9,n,z" target="_blank">https://review.openstack.org/#/q/I7ab6dc2341821c3b82ef3d3ac63b64a5a9958fa9,n,z</a></div><div>[4] <a href="https://review.openstack.org/#/q/Iff947f0053577f19441c04101e5a35a7820e40a0,n,z" target="_blank">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" target="_blank">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="HOEnZb"><div class="h5"><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><br>
> On 09 Mar 2015, at 18:21, Ryan Moe <<a href="mailto:rmoe@mirantis.com" target="_blank">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><font color="#888888">--<br>
Tomasz 'Zen' Napierala<br>
Sr. OpenStack Engineer<br>
<a href="mailto:tnapierala@mirantis.com" target="_blank">tnapierala@mirantis.com</a><br>
</font></span><div><div><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>
</div></div><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>
<br></blockquote></div><br></div>