From: Siddhesh Poyarekar <siddhesh@gotplt.org>
To: GNU C Library <libc-alpha@sourceware.org>
Subject: [STATS] Auto-update patchwork for committed patches
Date: Wed, 17 May 2023 11:30:37 -0400 [thread overview]
Message-ID: <ca01ca16-189b-51e3-68fa-e159491f73a1@gotplt.org> (raw)
Hello,
We have been using patchwork for status tracking for a couple of years
now (it's been a great run since the previous one didn't last too long)
and we're at a point where we use it enough to want to maintain its
state more closely.
To make sure that the backlog remains sane enough to be useful, I've
been running a cron job that reviews new commits to the master branch
daily and closes matching patchwork patches as COMMITTED. This works
only when the diff of a commit is exactly the same as that in patchwork;
the script ignores commits that don't have a matching diff in patchwork.
I get an email every day from the job, telling me which commits had
matching patches on patchwork and which didn't. Going forward, I'm
going to publish approximately weekly (until I can get another cron job
that parses these emails and does this automatically) statistics of how
many (and which) commits did not have a matching patch on patchwork.
The intent is to try and drive behaviour towards pushing patches exactly
as they were posted upstream. It's understandable that there may be
some trivial nits that get fixed before pushing, but the ideal behaviour
there should be to post a [committed] patch to the list so that there's
always a matching patch on list (and on patchwork) for every commit.
This has a number of advantages:
1. Verifiable claim that every commit in the repository has a record on
the mailing list (including conversations following the patch) and
patchwork, thus improving auditability and transparency.
2. A pre-commit CI run, through patchwork for every such commit
3. It enables further automation, such as marking previous versions of
patches with the same subject line as superseded (which makes subject
lines important or maybe the use of something like b4, but that's a
different topic for later)
If you find a commit that that's incorrectly marked as MISSING, please
let me know and we can dig out the root cause for it.
Thanks,
Sid
The stats for the last week-ish
===============================
In the last 10 days, there have been 34 commits to glibc and 22 had
matching patches on patchwork. This makes it a success rate of 64.7%.
Here are the commits that were missing from patchwork:
4571fb8fe64644c79d91a8f76c148a05b7088ea8 Update hurd/intr-msg.c to be
more portable: MISSING
3f433cb895dee51dee57cb487bc33b1425fa7ef6 Update
sysdeps/mach/hurd/ioctl.c to make it more portable: MISSING
a26238d3ca21fda6d7d41b4d56541fcf4546fbe7 Enable new device_open_new RPC
in libmachuser.: MISSING
bf88b47ecb54888a789c02fa81aa4ab81ec2f3a5 Revert "riscv: Resolve symbols
directly for symbols with STO_RISCV_VARIANT_CC.": MISSING
ab5aa2ee3d3f978e474803cbbc5fe805ad30e293 dlopen: skip debugger
notification for DSO loaded from sprof (bug 30258): MISSING
d6c72f976c61d3c1465699f2bcad77e62bafe61d hurd: rule out some mach
headers when generating errno.h: MISSING
3ca9f43d1007956251130ee5a59abb63bff8a6b6 Stop checking if MiG supports
retcode.: MISSING
088136aa02de6fa13061ef6f754071a5652fdabd i386: Use pthread_barrier for
synchronization on tst-bz21269: MISSING
114f1b7881e63e2b4e5d0e9a9e4fb142b9cd886c hurd: Fix computing user stack
pointer: MISSING
e333759f7752593a69a8f9920a247ed3878fafef hurd: Fix sc_i386_thread_state
layout: MISSING
ce96593c882b393461084048533120e9c1e9d328 hurd: Align signal stack
pointer after allocating stackframe: MISSING
ff0f87632a74a369a2b992f4436ae406065a4012 hurd: Fix aligning signal stack
pointer: MISSING
next reply other threads:[~2023-05-17 15:30 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-05-17 15:30 Siddhesh Poyarekar [this message]
2023-05-17 17:14 ` Samuel Thibault
2023-05-17 18:02 ` Siddhesh Poyarekar
2023-06-06 14:05 Siddhesh Poyarekar
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ca01ca16-189b-51e3-68fa-e159491f73a1@gotplt.org \
--to=siddhesh@gotplt.org \
--cc=libc-alpha@sourceware.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).