From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.4 required=3.0 tests=DKIMWL_WL_MED,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,USER_IN_DEF_DKIM_WL autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id E3222C4BA0A for ; Wed, 26 Feb 2020 08:28:43 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id B6970222C2 for ; Wed, 26 Feb 2020 08:28:43 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="CDw0SIaW" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727379AbgBZI2m (ORCPT ); Wed, 26 Feb 2020 03:28:42 -0500 Received: from mail-qt1-f169.google.com ([209.85.160.169]:42552 "EHLO mail-qt1-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727387AbgBZI2m (ORCPT ); Wed, 26 Feb 2020 03:28:42 -0500 Received: by mail-qt1-f169.google.com with SMTP id r5so1648644qtt.9 for ; Wed, 26 Feb 2020 00:28:41 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=mime-version:from:date:message-id:subject:to:cc :content-transfer-encoding; bh=TCKiJfk1tuX6JcQsxBCHZJ/LwaPkV6Zrs8OdjgYk6a0=; b=CDw0SIaWUZSx7ENi65bSwjHzQAWLf217RuK8tOjLweGrxEljFC5e2v65jurQ2QrUVk +5HfROicgsDND0ECwSUtjOPXCq42smEuX7j9ckuDcTb6gLPAs7m+Pm/rnBnw8xtdpz2/ U1IAm27CYn2i2+TH4wS1dbs+d73vyMf4x0H6qc0vYg2V9liVP/0U/LjbWqD00lKPcaH+ EaqkEnYkk2TP8LSMxJOzJ9QsnzYTEQKXWZ9yJXWzmA4cNHYuroGM5BKMZdh9aICwYceV pU4/8Rh+89HE2w6vX4ACzsGX/aYgrdm3Gj4rgLLttLxK5IT7M3oGhC7+VbGAH24nD01e /jXw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:from:date:message-id:subject:to:cc :content-transfer-encoding; bh=TCKiJfk1tuX6JcQsxBCHZJ/LwaPkV6Zrs8OdjgYk6a0=; b=eck4LbvP5p/EaetQqRFnLWzCbClqz5HWbK1ndR7ZT2lOhF0oxr2ACUoqPGQnwB3PX6 JFTj9vC7OKJ2+oy5GOfJltJNqCWoFwqXEknd9FooBhUnC7/4bIMrrJSx9yrid13wlA95 5LwoPtJB/PUn/se7lh2nePeve5Kxvz+Q6bPPIRVDrR4P4ydS6AC4T5m2WIUrgAWZ1Ees 9dtc0j8IQSo9n1eaB9ORdS4ny6TvNjk5LBMl9fdAeFj5V1lSVx8wrLjMddpnKv/DWBEu YUfbib6SkDsbvJJ40moIeW4H5XAVgq+GXOIzheIVS194T+sTP8fZPMky7Fovynlb233Q rm9A== X-Gm-Message-State: APjAAAWvNhJlbOD4G2REz2sOu8wA+v+tnGYDYIFZiwd4G5GvdSDg9fjG jBc6o8l/oyaTP3GNKnU+1PMTiK2Rc3vNEv0/IvoMcZZKgyhEZw== X-Google-Smtp-Source: APXvYqzIvgC0UaVeAAwn2e2tklNhtMrCKv0uUcP6KZReoNaHA+YKvaS0da2RRppS9PoApOf/RXdhr8C/6c5EUk+kF2A= X-Received: by 2002:ac8:7159:: with SMTP id h25mr3593895qtp.380.1582705720488; Wed, 26 Feb 2020 00:28:40 -0800 (PST) MIME-Version: 1.0 From: Dmitry Vyukov Date: Wed, 26 Feb 2020 09:28:29 +0100 Message-ID: Subject: public gerrit instance for kernel To: workflows@vger.kernel.org, LKML Cc: Han-Wen Nienhuys , Brendan Higgins , Ben Gardon , Jonathan Nieder , Alexei Starovoitov Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Sender: workflows-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: workflows@vger.kernel.org Hi, We've setup a public Gerrit instance for use with Linux kernel development: https://linux.googlesource.com/Documentation/#gerrit-code-reviews-for-the-l= inux-kernel After one-time setup changes can be pushed with a single command: $ git push gerrit-net HEAD:refs/for/master Gerrit has several (subjective) benefits over email-based reviews: - full context (you can expand more context as necessary) - diffs between version, e.g. full change is +547 lines: https://linux-review.googlesource.com/c/linux/kernel/git/torvalds/linux/+/2= 265/2 but diff between v1 and v2 is just 2 empty lines: https://linux-review.googlesource.com/c/linux/kernel/git/torvalds/linux/+/2= 265/1..2 (no need to write that up, trust subjective write ups) - colored side-by-side diffs, e.g. here you can easily see that even that line has changed it's only slash at the end that's added: https://linux-review.googlesource.com/c/linux/kernel/git/torvalds/linux/+/2= 103/1/kunit/Makefile#2 - marking files as "reviewed", always correct base tree/revision, etc But note there is no "official" story for gerrit in the linux process. You may use it as you find fit. Some uses that we found useful so far: - upload to do self-pre-review - review within a team of people who agree to use gerrit - include a link to gerrit into the upstream patch email as FYI (after =E2=80=9C---=E2=80=9D line) - upload somebody else patch just to review with side-by-side diffs and full context The branches are mirrored automatically from kernel.org; you can upload changes for review against those branches, but submission has to be routed through the traditional process. If you are brave enough, you may use a gerrit-managed tree as well, then with ability to merge/edit change on the web, non-losing comment threads attached to lines of code, change status tracking, etc. But that will need to be setup separately. There are some improvements planned like not requiring Change-ID and proxying comments to/from kernel mailing lists. But that's only in plans now. Thanks