From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mp12.migadu.com ([2001:41d0:2:bcc0::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by ms9.migadu.com with LMTPS id mPAQG9KCB2SKhgEASxT56A (envelope-from ) for ; Tue, 07 Mar 2023 19:30:42 +0100 Received: from aspmx1.migadu.com ([2001:41d0:2:bcc0::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by mp12.migadu.com with LMTPS id sBwJG9KCB2TUSgEAauVa8A (envelope-from ) for ; Tue, 07 Mar 2023 19:30:42 +0100 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by aspmx1.migadu.com (Postfix) with ESMTPS id 1C385A2E1 for ; Tue, 7 Mar 2023 19:30:42 +0100 (CET) Authentication-Results: aspmx1.migadu.com; dkim=fail ("headers rsa verify failed") header.d=gmail.com header.s=20210112 header.b=pYOwC1kU; spf=pass (aspmx1.migadu.com: domain of "guix-patches-bounces+larch=yhetil.org@gnu.org" designates 209.51.188.17 as permitted sender) smtp.mailfrom="guix-patches-bounces+larch=yhetil.org@gnu.org"; dmarc=fail reason="SPF not aligned (relaxed)" header.from=gmail.com (policy=none) ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=yhetil.org; s=key1; t=1678213842; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:resent-cc:resent-from:resent-sender: resent-message-id:in-reply-to:in-reply-to:references:references: list-id:list-help:list-unsubscribe:list-subscribe:list-post: dkim-signature; bh=S6UnTeIscbV8xdLPFgS7yHcXo2HDBpsdNG9uaNFvKUY=; b=RU4Qh/qLIVHQdIpb4AagNOfZeQHQ0ePNKfX2u4W/HJ+KgkEBEnfx10MwvOoEIoSRqyWWF1 LIaUyKdHb2AlfF4Ephz3Su5tAAWOheXBgtwXdRBtnZmKNQEoMS4Ys479s8zUe+qbvAAGh0 Ll8T7nHPXvj6+XWJe0ixYXYYBP0UIf75zbXTXnIll9JwAg5NUnELQHu1f+PuY2I0VMtx9I WWlhpC+W9vMR4URLb14njX05SfloHk6l8W12mH/nQZg0ARjhDiIRJsH9DMo+86oHyUBWRO Cs0V+05LqZGBWSbmv7r8N3GUp1zQ59itbjr9qNOcg2I7mwfp51XZPFA0VHJnXA== ARC-Seal: i=1; s=key1; d=yhetil.org; t=1678213842; a=rsa-sha256; cv=none; b=P3ggLs6t981kdJq2lMaDeQteXN/2hk8KezWnHDDwk2Iq4EqRP3Ug6sZytO4gEeoxUQWwGr oB+I3qKVCdeKxkaeUqP3IfScV2uIM1bmiF5QJYN8i5dPFcLuc8745pV9Rfk3lCo+TF1MOk /UuW3lE3ZS9dgw2axzovlO+UDY/jzZacNjZzhkM0Yza4SiCVd2UjdlkSNWyzIi2pG4kIgr Wsr10EgNsKC7/3RWC4AQmFWCZJ9r2t6GmHsAKDD1Db5CSHbOd04kp8O3aEzBvv4D+e+dWU jORFqRT2Ne7y9FAznbShjO7RCfPNKkiQ8By/vKlzjdDMPmc64HKZUboJOz4B3Q== ARC-Authentication-Results: i=1; aspmx1.migadu.com; dkim=fail ("headers rsa verify failed") header.d=gmail.com header.s=20210112 header.b=pYOwC1kU; spf=pass (aspmx1.migadu.com: domain of "guix-patches-bounces+larch=yhetil.org@gnu.org" designates 209.51.188.17 as permitted sender) smtp.mailfrom="guix-patches-bounces+larch=yhetil.org@gnu.org"; dmarc=fail reason="SPF not aligned (relaxed)" header.from=gmail.com (policy=none) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1pZc4d-00017n-IL; Tue, 07 Mar 2023 13:30:07 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1pZc4Z-0000vB-5n for guix-patches@gnu.org; Tue, 07 Mar 2023 13:30:03 -0500 Received: from debbugs.gnu.org ([209.51.188.43]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1pZc4Y-0006Lh-O3 for guix-patches@gnu.org; Tue, 07 Mar 2023 13:30:02 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1pZc4Y-0005sP-DW for guix-patches@gnu.org; Tue, 07 Mar 2023 13:30:02 -0500 X-Loop: help-debbugs@gnu.org Subject: [bug#61894] [PATCH RFC] Team approval for patches Resent-From: Maxim Cournoyer Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Tue, 07 Mar 2023 18:30:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 61894 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: Simon Tournier Cc: guix-maintainers@gnu.org, Ludovic =?UTF-8?Q?Court=C3=A8s?= , Christopher Baines , 61894@debbugs.gnu.org, =?UTF-8?Q?=E5=AE=8B=E6=96=87=E6=AD=A6?= , Andreas Enge , guix-devel@gnu.org Received: via spool by 61894-submit@debbugs.gnu.org id=B61894.167821380022557 (code B ref 61894); Tue, 07 Mar 2023 18:30:02 +0000 Received: (at 61894) by debbugs.gnu.org; 7 Mar 2023 18:30:00 +0000 Received: from localhost ([127.0.0.1]:47174 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1pZc4W-0005rl-0u for submit@debbugs.gnu.org; Tue, 07 Mar 2023 13:30:00 -0500 Received: from mail-qt1-f172.google.com ([209.85.160.172]:39519) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1pZc4U-0005rS-Mh for 61894@debbugs.gnu.org; Tue, 07 Mar 2023 13:29:59 -0500 Received: by mail-qt1-f172.google.com with SMTP id w23so15410627qtn.6 for <61894@debbugs.gnu.org>; Tue, 07 Mar 2023 10:29:58 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; t=1678213793; h=mime-version:user-agent:message-id:in-reply-to:date:references :subject:cc:to:from:from:to:cc:subject:date:message-id:reply-to; bh=S6UnTeIscbV8xdLPFgS7yHcXo2HDBpsdNG9uaNFvKUY=; b=pYOwC1kUDwYDXiwuO+2m4nuUdKRUnJILSnmUVTmyO/sWz3R2hVHLxMXdK5Vluw4D4o 6OMkUI0/DG6vj5Fzjy1ce7IyixBG6RkGrrevfaXq13HNtyVp6eGlaCDbL0ak1Vm/GUEh UVLOmX1Wtof0xCNoel8QXOiri5M8lwPSewyaERK8zOdWPXIB9t6/76mcgzLLUPbBTmMc vfE7ouCl4W3oa+gkXyxtu0BCqbpDiupsHCcQoudJ3mPFwI/Sq+w2t9O9+am8ncp3ZRSF gFI4wHhjqNIiMp6lWaQ0d3UFQ2xq4zbA6MDjFKAH4X69eCiu91GGmqAy1CsNJZoV8oMB xDyg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1678213793; h=mime-version:user-agent:message-id:in-reply-to:date:references :subject:cc:to:from:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=S6UnTeIscbV8xdLPFgS7yHcXo2HDBpsdNG9uaNFvKUY=; b=lO83joe9C/8fBSD1qBhnHIDIb9KQupoGXSolhK+blgdTX1+Wq0vfnIiOgKbeBCn8sf DEDaaQozyoUBO7NJWwGe3HpNWE2PcJffSz7Nms4cHeZcvFC9dIBJzenYcYyWAJ0E3YAj 0FYzba0C+MKb6Bwv6IH8AIU5BDcwNFm/VIz2stYW9VgUOXGwAgEZ8xUNQnZFtyl2LxFC urq088+g1U2kNU17YmawQ9A7k8A3nbBCpnc0nDF74O5g6uT0B9wl4pusVa4PgLwPwZpS 9zgtJlyNwbREXcdKY+gbcHd99/F8KpPSvZYwMr/hqkzeXw+Lc/nNaFPyEI8z8g2JiqfD Khqw== X-Gm-Message-State: AO0yUKUBxz5Gm7DiqrW/HCwiHz1RCGWYXioY4FN/85xYykDoBEo9Lsk6 80+wBa/hf836frBUhrvRNU8= X-Google-Smtp-Source: AK7set/f7msbP7FDOwxfnSJf8hZw4eOShgmD2XmzqYOpCfkrdhL+7rlYuERgl0bf6rD0AxTY895+/w== X-Received: by 2002:a05:622a:54c:b0:3bf:d9d2:484f with SMTP id m12-20020a05622a054c00b003bfd9d2484fmr25440347qtx.11.1678213793108; Tue, 07 Mar 2023 10:29:53 -0800 (PST) Received: from hurd ([2607:fad8:4:3::1000]) by smtp.gmail.com with ESMTPSA id q1-20020ac87341000000b003bfa2c512e6sm9936747qtp.20.2023.03.07.10.29.52 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 07 Mar 2023 10:29:52 -0800 (PST) From: Maxim Cournoyer References: <878rgga1qv.fsf@inria.fr> <871qm8wf8e.fsf@cbaines.net> <87r0u86qgo.fsf_-_@gnu.org> <87y1o9mina.fsf_-_@envs.net> <861qm0da4y.fsf@gmail.com> Date: Tue, 07 Mar 2023 13:29:51 -0500 In-Reply-To: <861qm0da4y.fsf@gmail.com> (Simon Tournier's message of "Tue, 07 Mar 2023 13:22:05 +0100") Message-ID: <87sfegwh28.fsf@gmail.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: X-Spam: Yes X-Migadu-Scanner: scn0.migadu.com X-Migadu-Spam: Yes X-Migadu-Queue-Id: 1C385A2E1 X-Spam-Score: 8.30 X-Migadu-Spam-Score: 8.30 List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+larch=yhetil.org@gnu.org Sender: guix-patches-bounces+larch=yhetil.org@gnu.org X-Migadu-Flow: FLOW_IN X-Migadu-Country: US X-TUID: Dv+jIm7KYZbH Hi Simon, Simon Tournier writes: > Hi, > > On Tue, 07 Mar 2023 at 11:36, Andreas Enge wrote: > >> 1) Every current and potential new package is covered by a team. >> 2) Every team has at least 3 members, better yet 4 or 5. >> 3 members would make it possible that even if one of them is on vacation >> or otherwise busy a patch could be pushed without this additional one >> week if the other 2 agree. > > It would help if being committer implies appearing at least in one team, > no? > > Currently in etc/teams.scm.in, I count 26 members and 20 are committers > over the 48 ones. No blame. :-) If most committers end up being team members, aren't we back to where we currently stand? It seems the original motivation here is to add some extra control/guards against undesirable commits landing in the core of Guix. If a committer that previously landed such commits joined the core team (e.g., myself), it seems to me the situation would be little changed: 1. Our pool of reviewers would likely continue to be spread too thin. 2. The 2 weeks time window would quickly slip, even with a team looking at a more focused backlog, or the reviews would only be of the kind "I think that's not what we want" without more time or energy to offer the kind of concrete insights that can be turned into action for the submitter. 3. The team member might be tempted to take their chance and merge their change with little to no feedback, or feedback they perceived insufficient or not actionable enough to justify keeping their submission in limbo for longer. I think the main problem we have is social, not organizational. There's little incentive to jump into the laborious review process compared to hack on something we like in our free time. We need to promote and value review work more, without making it feel like a compulsory chore. That's a great challenge to solve for a project that's driven by volunteers. I'll venture a suggestion to explore: adding enticements to review (some playful guidelines such as "while waiting for your 2 weeks review period, please try to review twice as many other submissions that have been patiently waiting on the patches tracker :-)", or some stats crunched and advertised periodically to guix-devel or even our to our blog about our top reviewers, etc.). -- Maxim