Open Stack

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.

The wiki [5] has also been updated to make this explicit.

[0]
https://review.openstack.org/#/q/Ief8186006386af8ae7e40cffeeaeef5a5c0f3c70,n,z
[1]
https://review.openstack.org/#/q/I4c94bb03501f4238ead2378cf504485b7d67b236,n,z
[2]
https://review.openstack.org/#/q/Ic15a3bfb6238e4281b06aae0a3f9fe4abf96590d,n,z
[3]
https://review.openstack.org/#/q/I7ab6dc2341821c3b82ef3d3ac63b64a5a9958fa9,n,z
[4]
https://review.openstack.org/#/q/Iff947f0053577f19441c04101e5a35a7820e40a0,n,z
[5]
https://wiki.openstack.org/wiki/Fuel/How_to_contribute#Backport_bugfixes_to_stable_release_series

Thanks,
Ryan

On Tue, Mar 10, 2015 at 4:20 AM, Tomasz Napierala <tnapierala at mirantis.com>
wrote:

>
> > On 09 Mar 2015, at 18:21, Ryan Moe <rmoe at mirantis.com> wrote:
> >
> > Hi All,
> >
> > 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.
>
> Hi Ryan,
>
> I was trying to find any examples, and the only one I see is:
> https://review.openstack.org/#/c/152674/
>
> 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.
>
> Do you have any other examples? I’d like to clarify them
>
> Regards,
> --
> Tomasz 'Zen' Napierala
> Sr. OpenStack Engineer
> tnapierala at mirantis.com
>
>
>
>
>
>
>
> __________________________________________________________________________
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20150310/dca6c3cc/attachment.html>