Re: [PATCH] docs: maintainer: discourage taking conversations off-list

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

 



Em Fri, 12 Jul 2024 17:05:58 -0700
Jakub Kicinski <kuba@xxxxxxxxxx> escreveu:

> On Fri, 12 Jul 2024 11:43:14 -0700 Dan Williams wrote:
> > This reads as a vague ambiguous quasi-threat with no actionable way to
> > enforce it. In contrast, successful maintainers already have a sense of
> > the benefits of pushing discussions to the list as much as possible.
> > 
> > For new and growing maintainers, which I assume are the audience for
> > this guidance, that are unaware of the pitfalls of taking conversations
> > off-list, they likely need help understanding the *benefits* of open
> > development.  
> 
> I don't think so. If your boss comes to you and says "Dan, from now on
> try not to reply to customers on the list, open a ticket first and only
> reply once tickets is resolved". Is it more useful to you to be able to
> extol the benefits of open source process; or to tell them "this is not
> allowed, here's a link to a quasi-threat"?

No matter what you write, between your text and their boss's directive,
I bet the latter will be a lot more effective.

> > So if this goes in as is, so be it, but it feels like a missed
> > opportunity to extoll the virtues of open development. The benefits are
> > in the same vector as the "release early, release often" guidance where
> > the urge to polish a solution in private is a common trait of newcomers.
> > Lets document some tribal knowledge of how one moves past that first
> > instinct.  
> 
> Hm, the disconnect may be that you think this happens with maintainers
> without upstream experience. So we can train them up to be better.

No, that's the case where you can still "fix" maintainer's behaviors.

> In most cases it happens to folks with experience who are good
> maintainers. They just get 2 orders of magnitudes more patches from
> inside the company that outside. Then a contribution comes from outside,
> the maintainer is overworked, and tries to shoehorn the patch into the
> existing, internal-only process.

It seems unavoidable that internal patches have higher priorities even
if the maintainer is not overloaded.

Even on a perfect world, the degree of confidence on internal patches is 
usually orders of magnitude higher, as internal devs have access to internal
product documentation, future line of products that will re-use the same
driver and, on larger projects, will also be already tested by internal
CI-based regression tests.

The external patches won't have that, so they need to be reviewed by
an internal developer, checked against internal docs and then submitted to 
the company's internal CI regression test infra to achieve the same degree
of confidence.

That not to mention that company will almost always prioritize new
product support over maintaining existing products.

No do/don't kind of document will change that.

A change like that should come top/down, so it has to be addressed 
via other strategies, like documents underlining the benefits of 
upstream first, and tutorials/speeches aimed at companies management
staff.

Thanks,
Mauro




[Index of Archives]     [Linux Samsung SoC]     [Linux Rockchip SoC]     [Linux Actions SoC]     [Linux for Synopsys ARC Processors]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]


  Powered by Linux