On Mon, Feb 27, 2023 at 07:52:39PM -0500, Sasha Levin wrote:
Nothing has changed, but that doesn't mean that your process is actually working. 7 days might be appropriate for something that looks like a security fix, but not for a random commit with no indications it is fixing anything.
How do we know if this is working or not though? How do you quantify the amount of useful commits?
Sasha, 7 days is too short. People have to be allowed to take holiday.
That's true, and I don't have strong objections to making it longer. How often did it happen though? We don't end up getting too many replies past the 7 day window.
I'll bump it to 14 days for a few months and see if it changes anything.
It's not just for the review time, but also for the longer soak time in mainline.
Of course, for that to work properly you have to actually take advantage of it, for example by re-checking for fixes when actually applying the patch, not just when sending the initial AUTOSEL email. Which I hope you're doing already, but who knows.
- Eric