Re: [RFC PATCH] doc: describe the project's decision-making process

From: Patrick Steinhardt <ps@pks.im>
Date: May 6, 2024, 07:12 UTC
Message-ID: <ZjiCz4_2KABLshLx@tanuki>
In-Reply-To: <ZjU7CWdwb+xKubul@nand.local>

On Fri, May 03, 2024 at 03:29:13PM -0400, Taylor Blau wrote:

Show 10 quoted lines

> On Fri, May 03, 2024 at 11:08:15AM -0700, Junio C Hamano wrote:
> > > Yes, sorry for silence on this thread. I am working on a V2 but
> > > probably won't have it ready today.
> >
> > Don't be sorry; the message was not addressed to you, but for wider
> > community participants---especially the ones with more "clout" (or
> > "long timers" or whatever word we would use to describe those whose
> > opinions are trusted by others and count more) need to buy in if we
> > were to first agree on that it is good to have a set of written
> > rules, and to then agree on what rules to adopt.

Fair enough. Given that I have been contributing quite a bit more recently I'll feel myself addressed here.

Show 18 quoted lines

> I have been meaning to respond to this thread since I was mentioned in
> it by Emily, but have been unsure of what to say.
> 
> On one hand, I think the document basically outlines the status-quo of
> decision making for issues that are larger than the scope of a single
> patch series (think "should we use Rust?", "what is our platform
> support policy?", or "how should we approach libification?" not "is this
> particular patch (series) correct?").
> 
> So in that sense, I think that the document is a good starting point,
> and I think that it reasonably captures the status quo.
> 
> But I wish that we didn't have to have such a document in the first
> place. In my opinion, I would much rather see decisions like "what is
> our platform policy?" made according to discussions on a patch that
> defines what that policy is. That way such decisions can be treated in
> the same way as ordinary review is today, and we can avoid the need for
> a separate process.

With "such a document", do you refer to the one documenting the process to do such changes or the RFC-style document?

If you mean the former I disagree and think that it would be great to document reasonable approaches for how to get to an agreement with the Git community. It's especially helpful for newcomers to the commuinity, and I do get questions around "How to reach consensus in Git" all the time at GitLab.

Now the important part to me is that we should retain flexibility and allow us to adapt. It should rather be a helpful resource to newcomers than a rigid set of requirements that everyone has to follow, in my opinion.

If it's the latter (no need for an RFC-style document) I somewhat agree. I'm of the opinion that patches are the best way to trigger a more informed discussion because participants will know how something will end up looking like. I also tend to send patch series for controversial topics upstream without prior discussion, but my intent here is never to force the result, but rather to force the discussion itself. This of course comes with the big downside that you may end up wasting quite a lot of your own time in case the community disagrees with your approach, but that's acceptable to me most of the time.

It's only part of the story though. There are bigger changes that simply don't make a ton of sense to propose in the above form, mostly because their sheer scope is much larger. As you point out further down, things like the SHA256 transition are in that realm.

Here I very much think that having a central place where such large projects are tracked. Given that we have no issue tracker, I think the next-best place to do this would be the Git repository itself. We already do this for some larger topics, but I feel like the bar is quite high right now. It doesn't make a ton of sense for example to have a huge RFC for the removal of `the_repository`.

So something that I'd really love to see is if we adopted the micro projects document [1] into Git itself and generalized it such that we can add smallish sections for new large-scale projects. Ideally, such projects could also have trailers that indicate who is interested in those projects such that folks knew whom to reach out to if they want to start contributing to the project. This would help us to document consensus around new projects, and would help newcomers of the community to pick up new projects.

Patrick

[1]: https://git.github.io/SoC-2024-Microprojects/

Show 17 quoted lines

> (For what it's worth, I thought that the SHA-256 transition was a good
> example of this. The RFC was posted, and the discussion was had on the
> patch series itself).
> 
> Another way of thinking about this is that I would be extremely
> reluctant to see a similar document proposed for reviewing at the patch
> series level. In my opinion, the system of reviewers and participants
> discussing the series and the maintainer solely determining whether or
> not consensus has been reached is a good one, and I would be extremely
> hesitant to recommend changing it.
> 
> And I would advocate for a similar approach to decisions that have
> implications beyond a single patch series.
> 
> Thanks,
> Taylor
>