[openstack-dev] Gerrit tooling improvements(was Re: auto-abandon changesets considered harmful)

Duncan Thomas duncan.thomas at gmail.com
Tue Mar 3 15:10:42 UTC 2015


I feel the need to abandon changes that seem abandoned. I believe this has
been covered to death now, so I'm going to shelve that conversation for a
while, and talk about missing tooling in gerrit.

One of the examples of something that was auto-abandoned wrongly was a
patch on hold until some future development cycle (L1 in the case of nova
patches, and cinder batches up certain types of code clean-up commits). So,
one thing that is definitely missing from the tooling is some way of
flagging such patches such that they *don't* get marked as abandoned, at
least until some sensible amount of time after they were supposed to get
picked back up.

So, the semantics of abandonment *certainly don't fit* patches that are
just on hold, but we don't have any way of tagging such patches. Is this
something we can fix?

On 2 March 2015 at 20:44, James E. Blair <corvus at inaugust.com> wrote:

> Duncan Thomas <duncan.thomas at gmail.com> writes:
>
> > Why do you say auto-abandon is the wrong tool? I've no problem with the 1
> > week warning if somebody wants to implement it - I can see the value. A
> > change-set that has been ignored for X weeks is pretty much the
> dictionary
> > definition of abandoned, and restoring it is one mouse click. Maybe put
> > something more verbose in the auto-abandon message than we have been,
> > encouraging those who feel it shouldn't have been marked abandoned to
> > restore it (and respond quicker in future) but other than that we seem to
> > be using the right tool to my eyes
>
> Why do you feel the need to abandon changes submitted by other people?
>
> Is it because you have a list of changes to review, and they persist on
> that list?  If so, let's work on making a better list for you.  We have
> the tools.  What query/page/list/etc are you looking at where you see
> changes that you don't want to see?
>
> -Jim
>
> __________________________________________________________________________
> 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
>



-- 
Duncan Thomas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20150303/f6eda27d/attachment.html>


More information about the OpenStack-dev mailing list