From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp1.linuxfoundation.org (smtp1.linux-foundation.org [172.17.192.35]) by mail.linuxfoundation.org (Postfix) with ESMTPS id A95BA49F for ; Sat, 24 Aug 2019 16:34:53 +0000 (UTC) Received: from mail-io1-f65.google.com (mail-io1-f65.google.com [209.85.166.65]) by smtp1.linuxfoundation.org (Postfix) with ESMTPS id 904F3A7 for ; Sat, 24 Aug 2019 16:34:52 +0000 (UTC) Received: by mail-io1-f65.google.com with SMTP id t6so27406407ios.7 for ; Sat, 24 Aug 2019 09:34:52 -0700 (PDT) Received: from mail-io1-f45.google.com (mail-io1-f45.google.com. [209.85.166.45]) by smtp.gmail.com with ESMTPSA id r7sm4731018ioa.71.2019.08.24.09.34.50 for (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 24 Aug 2019 09:34:50 -0700 (PDT) Received: by mail-io1-f45.google.com with SMTP id b10so18332412ioj.2 for ; Sat, 24 Aug 2019 09:34:50 -0700 (PDT) MIME-Version: 1.0 References: <20190823013619.GA8130@mit.edu> <20190823151843.GH8130@mit.edu> <20190823161947.GA112509@dtor-ws> <20190823164602.GB112509@dtor-ws> In-Reply-To: From: Doug Anderson Date: Sat, 24 Aug 2019 09:34:34 -0700 Message-ID: To: Christian Brauner Content-Type: text/plain; charset="UTF-8" Cc: Joel Fernandes , Barret Rhoden , ksummit , Greg Kroah-Hartman , Jonathan Nieder , Tomasz Figa , Han-Wen Nienhuys , Theodore Tso , David Rientjes , Dmitry Torokhov , Dmitry Vyukov Subject: Re: [Ksummit-discuss] Allowing something Change-Id (or something like it) in kernel commits List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Hi, On Fri, Aug 23, 2019 at 1:02 PM Christian Brauner wrote: > > > > On Fri, Aug 23, 2019, 21:17 Thomas Gleixner wrote: >> >> On Fri, 23 Aug 2019, Dmitry Torokhov wrote: >> > On Fri, Aug 23, 2019 at 12:35:03PM -0400, Joel Fernandes wrote: >> > > On Fri, Aug 23, 2019 at 12:19 PM Dmitry Torokhov >> > > wrote: >> > > > On Fri, Aug 23, 2019 at 05:48:55PM +0200, Thomas Gleixner wrote: >> > > > > >> > > > > Yes, it's work for the submitter, but it's always work if the submitter >> > > > > wants to have a proper trace. >> > > > >> > > > Here is where I disagree with you. As a patch submitter, I frankly could >> > > > not care less about proper trace, history, etc. I might be putting cover >> > > >> > > But that is exactly what the problem statement is. Doug does care >> > > about tracing/history and wants that to be more robust etc. >> > >> > Doug here is not a submitter ;) >> >> Well, if he wants the changeids there then submitters need to insert them, >> right? So it's work no matter what unless it can be automated with tooling. >> >> Guess what, how I inject the Link to the coverletter of the previous >> version of a patch series? Definitely not manualy. >> >> So yes, if you want proper traceability then all involved people have to do >> something. If it can be done with tooling fully automated, fine. If not, >> it's work whatever method you chose. >> >> We cannot enforce the changeid thing in the community, but Google can >> inforce it internally. So we give them a way to be traceable w/o plastering >> the kernel logs with potentially useless information. >> >> That said, I'm fine with a Link as well, as long as it is public > > > A link is definitely more helpful then the change-id. > Quite a few maintainers are already making use of links to various sites anyway so I don't see a good reason not allow Links to Gerrit or whatever. Just wanted to mention again that for the use case I'm trying to solve: there is no link. Here is the relevant part of the email chain explaining: I have no gerrit server involved when I submit patches to the list. I do: 1. Write patch on my local machine. 2. Post v1 to mailing list. 3. Make changes. 4. Post v2 to mailing list. 5. Make changes. 5. Post v3 to mailing list. I have never uploaded to a gerrit in this process. THERE IS NO GERRIT LINK! >> accessible. If not, then the coverletter/discard area is the place to use. > > > Right, change-id should go after --- which is also what Dmitry Vyukov suggested. It is not my favorite because it will have less adoption (people will need to manually move the Change-Id or adopt new tools that do this for them). The advantage of keeping Change-Id above the cut is that those who have adopted the git hooks that gerrit requires (and there appears to be a lot of people who use gerrit) will magically start doing the right thing instead of getting yelled at. Overall the success of this depends on submitters adopting it and submitters are much harder to influence than maintainers (and that's saying a lot). That all being said: I guess I will settle for not getting yelled at for Change-Id existing after-the-cut. Is anyone on this thread upset if people put Change-Id after-the-cut? > One thing I wonder though. What's the ultimate goal here? > Enabling people to review on Gerrit and lkml simultaneously? > I mean, apart from tracking versions of patch series/patches this can't be all, right? Ultimate goal: to allow tools (maybe gerrit, maybe patchwork, maybe something entirely new) to identify when a patch is a newer version of an old one. Yes, that might someday allow a nice integration of gerrit with kernel mailing lists. Short term goal: allow people to manually find patch v1 from patch v2 using text search. ...or to find all mailing list discussion from a landed patch. Example of how short term goal works (from earlier in the thread): https://lore.kernel.org/lkml/?q=Change-Id%3A+I23e218cd964f16c0b2b26127d4a5ca6529867673 > We can already do that right now and I'm already doing that when applying stuff to my tree: inserting the link to the version of the patch set I applied and linking to the previous version in each new version of the patchset. > That could also be automated. > So is allowing reviews both on Gerrit or whatever the goal here and if so how do we ensure that lkml sees all reviews? The flow I'm talking about doesn't depend on ANY gerrit. There is no server and there are no private reviews. There is only a Change-Id linking versions of patches. IF (and only if) someone wanted to make a gerrit (or several gerrits) that scraped LKML and created a gerrit change for each new patch, this Change Id could be useful to them. Presumably nobody would care if someone made a private comment on one of those gerrits. ...but if gerrit every fully integrated email lists then (you could imagine) that publishing your review on gerrit would send it out to the lists. Any other gerrits that also happened to be scraping LKML would notice this review and (presumably) mirror the comments on their own gerrit. So in my mind allowing Change-Id preserves the distributed mailing list centric workflow of the kernel but would help people implement a GUI. -Doug