workflows.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jakub Kicinski <kuba@kernel.org>
To: Dan Williams <dan.j.williams@intel.com>
Cc: <corbet@lwn.net>, <workflows@vger.kernel.org>,
	<linux-doc@vger.kernel.org>
Subject: Re: [PATCH] docs: maintainer: discourage taking conversations off-list
Date: Fri, 12 Jul 2024 17:05:58 -0700	[thread overview]
Message-ID: <20240712170558.50c89238@kernel.org> (raw)
In-Reply-To: <669179424b589_1a77429479@dwillia2-xfh.jf.intel.com.notmuch>

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"?

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

  reply	other threads:[~2024-07-13  0:05 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-07-12 14:49 Jakub Kicinski
2024-07-12 15:06 ` Greg KH
2024-07-12 15:25 ` Mark Brown
2024-07-12 15:42   ` Shuah Khan
2024-07-12 18:11     ` Mauro Carvalho Chehab
2024-07-12 18:19       ` Randy Dunlap
2024-07-12 23:45       ` Jakub Kicinski
2024-07-13  0:00         ` Dan Williams
2024-07-13  0:12           ` Jakub Kicinski
2024-07-13  7:43         ` Mauro Carvalho Chehab
2024-07-12 18:43 ` Dan Williams
2024-07-13  0:05   ` Jakub Kicinski [this message]
2024-07-13  1:18     ` Dan Williams
2024-07-13  8:13     ` Mauro Carvalho Chehab
2024-07-13 14:19       ` Laurent Pinchart
2024-07-13 16:07         ` Mauro Carvalho Chehab
2024-07-13 23:29           ` Jakub Kicinski
2024-07-15 13:29       ` Mark Brown
2024-07-13 14:26 ` Laurent Pinchart
2024-07-13 23:23   ` Jakub Kicinski
2024-07-13 14:28 ` Carlos Bilbao
2024-07-13 16:25   ` Mauro Carvalho Chehab

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=20240712170558.50c89238@kernel.org \
    --to=kuba@kernel.org \
    --cc=corbet@lwn.net \
    --cc=dan.j.williams@intel.com \
    --cc=linux-doc@vger.kernel.org \
    --cc=workflows@vger.kernel.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