Re: [PATCH v4 1/3] docs: add two documents about regression handling

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 02.02.22 11:05, Thorsten Leemhuis wrote:
> On 02.02.22 00:13, Jonathan Corbet wrote:
>> OK, I'll try not to take so long to have a look at it this time.
>>
>> Thorsten Leemhuis <linux@xxxxxxxxxxxxx> writes:
>>
>>> Create two documents explaining various aspects around regression
>>> handling and tracking; one is aimed at users, the other targets
>>> developers.
>>>
>>> The texts among others describe the first rule of Linux kernel
>>> development and what it means in practice. They also explain what a
>>> regression actually is and how to report one properly.
>>>
>>> Both texts additionally provide a brief introduction to the bot the
>>> kernel's regression tracker uses to facilitate the work, but mention the
>>> use is optional.
>>>
>>> To sum things up, provide a few quotes from Linus in the document for
>>> developers to show how serious he takes regressions.
>>>
>>> Signed-off-by: Thorsten Leemhuis <linux@xxxxxxxxxxxxx>
>>> ---
>>>  Documentation/admin-guide/index.rst           |   1 +
>>>  .../admin-guide/regressions-users.rst         | 436 ++++++++++++
>>>  Documentation/process/index.rst               |   1 +
>>>  Documentation/process/regressions-devs.rst    | 672 ++++++++++++++++++
>>
>> I'll start with some *serious* bikesheddery...it's best if the names of
>> the files tell readers what's inside.  This isn't something I feel
>> really strongly about, but we could consider
> 
> I wasn't totally happy with the file names myself, so it's good that you
> bring it up.
> 
>> 	admin-guide/reporting-regressions.txt (or just regressions.txt)
>>         process/regression-policy.txt
> 
> I like "reporting-regressions.txt", but I wonder if using the word
> "policy" is a good idea. I tried to avoid it (and similar words, like
> guidelines), as they might do more harm then good. So how about:
> 
>  	admin-guide/reporting-regressions.rst
>         process/regressions.rst

BTW, I settled on these:

 admin-guide/reporting-regressions.rst
 process/handling-regressions.rst

>> [...] 
>>> +Send a mail to the regressions mailing list (regressions@xxxxxxxxxxxxxxx) while
>>> +CCing the Linux kernel's regression tracker (regressions@xxxxxxxxxxxxx); if the
>>> +issue might better be dealt with in private, feel free to omit the list.
>>
>> Perhaps a separate concern, but might you want to set up an @kernel.org
>> alias for the regression tracker?  Trust me, you're not gonna want to
>> run it forever, and the ability to quickly redirect the mail may prove
>> to be a nice thing to have.  An email address with your domain sitting
>> in the docs will circulate for years after it gets changed.
> 
> Yeah, it's on my mental to do list for a few weeks already, but never
> set down to actually get this rolling. You are right, I'll ask for an alias.

I liked the idea and requested this, but Konstantin raised a good point:
"we don't have generic aliases for all other subsystems, but the
situation where someone may want to discuss something off-list arises
for all of them". I have to agree with him. I first wanted to change the
text to something like "look up the address in MAINTAINERS", but that
just shifts the problem. Additionally: We have lot's of other private
addresses from maintainers in the docs as well and don't consider that
to be a problem. And if somebody else takes over I can also redirect
that email or create a autoreply on my local server. So I'd say we just
leave things as they are, it's not perfect, but good enough I'd say.

Ciao, Thorsten



[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux