[heat] Addressing the large patch backlog.

David Peacock dpeacock at redhat.com
Mon Dec 23 19:11:57 UTC 2019


Thanks so much for working this Zane, it's much appreciated! :-)

On Tue, Dec 3, 2019 at 2:51 PM Zane Bitter <zbitter at redhat.com> wrote:

> I added a column for "Core Reviewer comments" as well.
>

I hijacked this column for my amendments too.

> > *_Rebase + Merge_*
>
> Would you have time to start going through these and rebase or recheck
> them (as required), then provide a list somewhere of priority reviews
> (i.e. those that rebased easily and passed tests) for cores to go
> through? It should be pretty easy to merge a good number of these quite
> quickly.
>

I went through a big blitz of all those rebase + merge patches today; as
expected many of them were trivial rebases but a lot were very broken due
to their age.  I fixed up each and every single one, and even iterated on
core review suggestions on a some.  They're all in good shape for review /
pushing through the merge now.

When cores have time it would be great to get those reviewed and hopefully
workflowed.

>
> > *_Research_*
> >
> > 76 patches are sufficiently complex that they'll need a much closer
> > look.  Some of these patches are in a seemingly "finished" state, some
> > are a way off.  Some have unanswered concerns from core review and have
> > been left dangling. *If you're the original developer or otherwise
> > interested in working these patches through to completion, please do get
> > involved.*
>
> As a first step it'd be great if we could extract from this a list of
> stuff that looks promising but hasn't been reviewed by a core. Those
> would be the next highest priority to review I'd expect.
>

Absolutely - that's a great idea!  I'll handle this.
This is a very rewarding journey, and as I work the last few days before I
go on my winter solstice break, it's fantastic for reflection too.

Looking forward to continuing this endeavour into the new year.

For those who celebrate, enjoy your holidays.  For everyone else, keep
being excellent! :-)

Cheers,
David
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-discuss/attachments/20191223/4e8edf0d/attachment.html>


More information about the openstack-discuss mailing list