Re: Next steps from GTI TAC meeting on 2023-03-08 - Evaluate cost of glibc migration.

Siddhesh Poyarekar <[email protected]> Wed, 24 May 2023 11:18:49 -0400
Newsgroups org.linuxfoundation.lists.cti-tac
Message-ID <[email protected]>
On 2023-05-24 10:12, Carlos O'Donell wrote:
> Is 100% coverage an achievable and timely goal? :-)

It's an achievable, timely *and* trivial goal if we post missing commits 
(i.e. commits in git without a matching patch on libc-alpha) to 
libc-alpha as [pushed].

> I think that Patchwork *could* meet this need but isn't quite there yet.
> 
>> If noting where you stopped/started is cumbersome then you could add
>> another Audited terminal state in patchwork to track that.
> 
> That would need to be a per-reviewer state that doesn't exist today in patchwork.
> 
> The Audited terminal state would be global, and Joseph and I notice different things
> in our reviews and apply different criteria for calling out a patch that needs follow
> up work or review.
> 
> In summary:
> 
> - Post-commit review facilitation for projects continues to be a value-add step
>    in the maintainer process and is currently mbox-based.
> 
> - The bar for post-commit review today is an mbox review-style process and a
>    replacement needs to handle multiple reviewers walking the queue at different
>    rates and applying different criteria to review (tag-based scheme?)

Wouldn't a [pushed] email to libc-alpha provide more visibility then, 
given that multiple people (I do too, albeit infrequently) tend to 
review these commits?  It shouldn't need a separate mailing list, given 
our email traffic volume.  Basically:

- Commits to the main branch go to libc-alpha
- Commits to release/* branches go to libc-stable
- All other commits (i.e. to private branches) get ignored.

Thanks,
Sid