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

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

 



On Sat, Jul 13, 2024 at 10:13:28AM +0200, Mauro Carvalho Chehab wrote:
> Em Fri, 12 Jul 2024 17:05:58 -0700 Jakub Kicinski 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.

I don't agree with this. I have found clear written rules valuable
personally when discussing with management. Being able to point to
upstream policies and tell "this is not allowed" helped change internal
processes. It will obviously not have a 100% success rate, but it's not
useless.

> > > 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.

We don't seem to live in the same (perfect or imperfect) world. In my
experience, contributions from external kernel developers are often
better than patches originating from within a company. External
contributors are more used to follow proper upstream review processes.

> 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.

-- 
Regards,

Laurent Pinchart




[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