Re: [Ietf-and-github] Alvaro Retana's Discuss on draft-ietf-git-using-github-05: (with DISCUSS and COMMENT)

Martin Thomson <mt@lowentropy.net> Wed, 11 March 2020 23:17 UTC

Return-Path: <mt@lowentropy.net>
X-Original-To: ietf-and-github@ietfa.amsl.com
Delivered-To: ietf-and-github@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 605C03A09EF; Wed, 11 Mar 2020 16:17:34 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.098
X-Spam-Level:
X-Spam-Status: No, score=-2.098 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_MSPIKE_H3=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=lowentropy.net header.b=S528HfPC; dkim=pass (2048-bit key) header.d=messagingengine.com header.b=pAgONOsm
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id b0bqrvW8AwN4; Wed, 11 Mar 2020 16:17:33 -0700 (PDT)
Received: from out2-smtp.messagingengine.com (out2-smtp.messagingengine.com [66.111.4.26]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 3B98C3A077C; Wed, 11 Mar 2020 16:17:24 -0700 (PDT)
Received: from compute2.internal (compute2.nyi.internal [10.202.2.42]) by mailout.nyi.internal (Postfix) with ESMTP id 60B9B21FAE; Wed, 11 Mar 2020 19:17:23 -0400 (EDT)
Received: from imap2 ([10.202.2.52]) by compute2.internal (MEProxy); Wed, 11 Mar 2020 19:17:23 -0400
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=lowentropy.net; h=mime-version:message-id:in-reply-to:references:date:from:to :cc:subject:content-type:content-transfer-encoding; s=fm1; bh=UI Et2mbnYp/HQz46rfrhgBqJGqq6m3fx+phqad4oW94=; b=S528HfPC+9xLXbD21e L1tQKi9EMoEvVMfmNKIAoGpUjj+IuMlpWDfihML7YpNbE+S0Vb/VLNYbAeck13f6 FEcICvSVIVxbYHVarClhtM067cqVKtj2gf6l7i4YGdzZFChvOJrj6eimUH8kFqx4 hstNGGV+7/bbL1tkVgXRXxMOmqRhdL6boCafqM1t7ZVLzrVy3CyT7e8F1KKRcLEP Bo9OmZRVxKR2hUcPA8ATst8pcjDyVmHgAg5EdCZizNayykrlgpqSqJ2li5W7358p rSsvX78o0W43vC7Kb22bTK4QDRqqv/xxnqOUxidd8G1ZKMTigug6IOAcgDz1Bq0Q vGVw==
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm2; bh=UIEt2mbnYp/HQz46rfrhgBqJGqq6m3fx+phqad4oW 94=; b=pAgONOsmC5sIiMv8yi9FVkn5iWs4gi+IMm5EuojR2ilYMaSBkLnrL5ecc LspC5Hqis0X5QEH23EHIFD+QHi1wBDT8X3WdfLNlL811CRK9p+F6MY/31LRz5mYk pOpKhsvwq/mQGZmyQGDWT0WRj/l7FJO63OFU6SYd1OrC7PQRa17cI0HeiI8EVN45 U5hJil77eAU0uJfrwcrmy+x1fllRS+TGREcuN0CCbElZvPwqRzwVRjXLIkiw86l7 L+gjtFoonf3AQgk0mfhPkeTadcOxCGaYt6je03FJiL8YgfjtMdzmhyAeGfKs5mc0 YHjnqaMtb++qhhJjwPGiI9Am1Cz0g==
X-ME-Sender: <xms:gnFpXmUF_u3GHV5aDdfPbswYblbF1M7akMzh8B-NLK-3tXExqB70Fw>
X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedugedruddvgedgudelucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepofgfggfkjghffffhvffutgfgsehtqhertderreejnecuhfhrohhmpedfofgr rhhtihhnucfvhhhomhhsohhnfdcuoehmtheslhhofigvnhhtrhhophihrdhnvghtqeenuc ffohhmrghinhepghhithhhuhgsrdgtohhmnecuvehluhhsthgvrhfuihiivgeptdenucfr rghrrghmpehmrghilhhfrhhomhepmhhtsehlohifvghnthhrohhphidrnhgvth
X-ME-Proxy: <xmx:gnFpXnfh7pmFVgfDCdB5wQEuH9ooAwfna1vOFQhee3HRfCc0rxkizA> <xmx:gnFpXi8yR2iXbdcevCPASgy0TtGYD5FaJCHTvjPxfirFRylmKPxeYQ> <xmx:gnFpXunmvT50c7VV1z1GbczDRExZPBJzJNo_ZmVfTuPJlH55nst2CQ> <xmx:g3FpXqNXho0fVDTb2iiCO18z2JD_fg8yfL7uQelQ8akL9KITibJESA>
Received: by mailuser.nyi.internal (Postfix, from userid 501) id D2DF6E00B2; Wed, 11 Mar 2020 19:17:22 -0400 (EDT)
X-Mailer: MessagingEngine.com Webmail Interface
User-Agent: Cyrus-JMAP/3.1.7-991-g5a577d3-fmstable-20200305v3
Mime-Version: 1.0
Message-Id: <a0492c6f-1610-40fc-be81-130893f457ff@www.fastmail.com>
In-Reply-To: <158395281137.1671.933778421064897517@ietfa.amsl.com>
References: <158395281137.1671.933778421064897517@ietfa.amsl.com>
Date: Thu, 12 Mar 2020 10:17:04 +1100
From: Martin Thomson <mt@lowentropy.net>
To: Alvaro Retana <aretana.ietf@gmail.com>, The IESG <iesg@ietf.org>
Cc: "draft-ietf-git-using-github@ietf.org" <draft-ietf-git-using-github@ietf.org>, git-chairs@ietf.org, ietf-and-github@ietf.org, Christopher Wood <caw@heapingbits.net>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: <https://mailarchive.ietf.org/arch/msg/ietf-and-github/TOuGvvGIY0rqEIdV2UDSBYVKvmI>
Subject: Re: [Ietf-and-github] Alvaro Retana's Discuss on draft-ietf-git-using-github-05: (with DISCUSS and COMMENT)
X-BeenThere: ietf-and-github@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "Discussion of using GitHub in IETF activities, particularly for Working Groups" <ietf-and-github.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf-and-github>, <mailto:ietf-and-github-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ietf-and-github/>
List-Post: <mailto:ietf-and-github@ietf.org>
List-Help: <mailto:ietf-and-github-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf-and-github>, <mailto:ietf-and-github-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 11 Mar 2020 23:17:35 -0000

Hi Alvaro,

Thanks for the feedback.  I think that we'll pursue the DISCUSS thing separately.  I want to (attempt to) discharge your comments.

I have prepared a pull request that you can review here: https://github.com/ietf-gitwg/using-github/pull/47
I appreciate that you didn't repeat comments from others, but in case you care, there is a collection of changes proposed as a result of other reviews: https://github.com/ietf-gitwg/using-github/pulls

On Thu, Mar 12, 2020, at 05:53, Alvaro Retana via Datatracker wrote:
> (2) It would be nice to have a short terminology section; I assume many people
> reading this document will not already be GitHub-savy, so push, pull, commits,
> may not be familiar to them.  Alternatively, an Informational pointer to a
> tutorial would also be ok.

I've added a reference to the GitHub glossary that covers both git and GitHub terminology in some detail.
 
> (3) Personally, I don't have an issue with the use of GitHub, but some of the
> statements in the Introduction sound like marketing blurbs, for example:
> 
> - "Use of this service has been found to reduce the time that Working Groups
>    need to produce documents and to improve the quality of the final result."
> 
> - "...encourage contributions from a larger set of contributors."
> 
> - "Using GitHub can also broaden the community of contributors for a
>    specification."

That's a fair observation, yes.

> (5) [major]  §5.3: "Working Group chairs SHOULD confirm that the Working Group
> has consensus to adopt any process."  When would the chairs not confirm
> consensus to adopt a process?  IOW, why is this not a MUST?
> 
> Note that §3 says this:
> 
>    Working Group Chairs are responsible for determining how to best
>    accomplish the charter objectives in an open and transparent fashion.
>    The Working Group Chairs are responsible for determining if there is
>    interest in using GitHub and making a consensus call to determine if
>    the proposed policy and use is acceptable.
> 
> Even though this text doesn't use rfc2119 keywords, my impression of the intent
> is that it is required for the chairs to make the consensus call.  IOW, I think
> that this text and the one above (from §5.3) are in conflict.

I'm going to remove the keywords here.  Because this is a specialization of language from RFC 2418 and while the specialization is useful, the use of 2119 keywords isn't consistent with the original:

  The Chair has the responsibility and the authority to make decisions,
   on behalf of the working group, regarding all matters of working
   group process and staffing, in conformance with the rules of the
   IETF.  

> (6) §5.3.2:
> 
>    Gaining Working Group consensus about the resolution of issues can be
>    done in the abstract, with editors being permitted to capture the
>    outcome of discussions as they see fit.
> 
> This sentence doesn't sound right to me: "consensus...can be done in the
> abstract, with editors being permitted to capture the outcome...as they see
> fit".  That last part doesn't sound right: Chairs call consensus.  Maybe I'm
> misinterpreting...

Others have had trouble with this.  Perhaps rephrasing is in order.  How does this sound?

> Chairs are able to declare Working Group consensus about the resolution of issues in the abstract, allowing editors discretion on how to capture the decisions in documents.

> (7) [major]. Why is draft-ietf-git-github-wg-configuration listed as a
> Normative Reference?  I don't think that dependency is needed.

I will adjust.  There is no normative dependency.  Thanks for noticing.