Re: [mpls] MPLS-RT review of draft-bryant-mpls-sfl-control

Stewart Bryant <stewart.bryant@gmail.com> Thu, 02 January 2020 16:28 UTC

Return-Path: <stewart.bryant@gmail.com>
X-Original-To: mpls@ietfa.amsl.com
Delivered-To: mpls@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id AA8F2120142; Thu, 2 Jan 2020 08:28:13 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -0.998
X-Spam-Level:
X-Spam-Status: No, score=-0.998 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, FREEMAIL_REPLY=1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001] autolearn=no autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com
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 rNLypqPdAObs; Thu, 2 Jan 2020 08:28:12 -0800 (PST)
Received: from mail-wr1-x429.google.com (mail-wr1-x429.google.com [IPv6:2a00:1450:4864:20::429]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id C752512013C; Thu, 2 Jan 2020 08:28:11 -0800 (PST)
Received: by mail-wr1-x429.google.com with SMTP id w15so27174134wru.4; Thu, 02 Jan 2020 08:28:11 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:message-id:mime-version:subject:date:in-reply-to:cc:to :references; bh=KYz3O1Q+qfDg1U4s1yLV0L4H1kggFIPzN5ekJT0/iFo=; b=cFK0TlUOO8cvShP7M5ltraXCrNFb76Sciuy5RuJ5WFnUcHNlwdwYBULxsjf+2y5Pfd F200bUSNVvsoVB8xMBj2tDCFmA3HP0RUJT9cB44b7qPK7mhDbOagQO1a5WcSw6l4zZDK uTTAmzghEbstTZNXuU0sQBwitHwLgYH+i0Xdey1J+uZxUBn05kWuDJoc4XNDe4Oh5ij7 NUrqyVmVKia+nDprzrDFpD2siw0oZ5pvoKuU+bi5wWR8FMrfdAzy3l86l9wipVCXYWv4 83yMuYYJp5ohQAySu9yJZWGU+h0r0ielGRF3wkUKR9hfJdYFcjHjJ36CAHfaxkd4b38W 5FbA==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:message-id:mime-version:subject:date :in-reply-to:cc:to:references; bh=KYz3O1Q+qfDg1U4s1yLV0L4H1kggFIPzN5ekJT0/iFo=; b=Obk6+X/GgSN3m+OTRJ/pOaeKfP1QfT1w6JyFsV7xTQo71Qul1EDYDWm5n5CH7SB2oM ILJM8a85jp/4h/vxt0IddidtG/FQc2ieVf4g9LMvOXj6zW3SpBEhQt2esxZrvYXN/PMN AnOS/u0H5lZCa7DGwCZ+xSvvbVo3yOCd+vRKzmjjP7XUpR9HjWxczEqmr4qDJbBXrqAH DMw+DLW2CeVHWnG+GBo3Py51Gj9FyS/dyamEDyHGrr5tsLDBNQqvFKbm76SyNrGrkWs+ O3p6EhyTkkx48dH+p2GYb1YDImr+rzpMWh73SVsfC+x5Sz0EvxEhsOS/GK4q4F2PHqeQ jkTw==
X-Gm-Message-State: APjAAAWw/RP9onttd6ITtG4E42Q3bhkyk7bms+Hsm1vNd6h9ea+7oH4g CN9MrMEo0IOVFvCn2hJB6cc=
X-Google-Smtp-Source: APXvYqz8jqRBxF1LiSuNPrfTf8rJeddZlSZuxvtJkfJOpVqxf2TnZBnWInV/2ImgCSG2rFcjxXbnCQ==
X-Received: by 2002:a5d:558d:: with SMTP id i13mr83961672wrv.364.1577982490314; Thu, 02 Jan 2020 08:28:10 -0800 (PST)
Received: from broadband.bt.com ([2a00:23a8:4140:0:94cd:2284:e44c:6235]) by smtp.gmail.com with ESMTPSA id s16sm56632052wrn.78.2020.01.02.08.28.08 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 02 Jan 2020 08:28:09 -0800 (PST)
From: Stewart Bryant <stewart.bryant@gmail.com>
Message-Id: <09C6EB05-90B1-48AD-82E4-E2B833940AFB@gmail.com>
Content-Type: multipart/alternative; boundary="Apple-Mail=_616A259E-40DB-4E66-91B8-6F864A2BA3A7"
Mime-Version: 1.0 (Mac OS X Mail 13.0 \(3601.0.10\))
Date: Thu, 02 Jan 2020 16:28:08 +0000
In-Reply-To: <CAA=duU1yW+rpn2PXkUg7Kd23HiUpDb7CMnke9M62usHQG8_rkg@mail.gmail.com>
Cc: draft-bryant-mpls-sfl-control@ietf.org, mpls <mpls@ietf.org>, mpls-chairs <mpls-chairs@ietf.org>
To: "Andrew G. Malis" <agmalis@gmail.com>
References: <CAA=duU3_AaiH4J7o0vai=KCL_LY93e2xc0JQO3t6WsUSWM3dJg@mail.gmail.com> <F395ECEC-E39D-4AE1-8277-11A73DC3EF71@gmail.com> <CAA=duU18ZJoamK_tZcUP=3pJTmA7OYP+Pa8bu9-wtcPu-6DzLg@mail.gmail.com> <CAA=duU1yW+rpn2PXkUg7Kd23HiUpDb7CMnke9M62usHQG8_rkg@mail.gmail.com>
X-Mailer: Apple Mail (2.3601.0.10)
Archived-At: <https://mailarchive.ietf.org/arch/msg/mpls/mBQu71pgTzghU8J_fDCxAMvfr8Y>
Subject: Re: [mpls] MPLS-RT review of draft-bryant-mpls-sfl-control
X-BeenThere: mpls@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Multi-Protocol Label Switching WG <mpls.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mpls>, <mailto:mpls-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/mpls/>
List-Post: <mailto:mpls@ietf.org>
List-Help: <mailto:mpls-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mpls>, <mailto:mpls-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 02 Jan 2020 16:28:14 -0000

Fixed in the master I will use for the next release.

- Stewart

> On 2 Jan 2020, at 14:47, Andrew G. Malis <agmalis@gmail.com> wrote:
> 
> Stewart,
> 
> Looking at the diff from the previous version, I just noticed a typo that I missed earlier. In the first paragraph of the introduction, "withdrawn" should be "withdraw". I also see that you added an Oxford comma, which is fine by me! :-)
> 
> Cheers,
> Andy
> 
> 
> On Thu, Jan 2, 2020 at 9:40 AM Andrew G. Malis <agmalis@gmail.com <mailto:agmalis@gmail.com>> wrote:
> Stewart,
> 
> Glad to be of assistance.
> 
> Cheers,
> Andy
> 
> 
> On Thu, Jan 2, 2020 at 9:09 AM Stewart Bryant <stewart.bryant@gmail.com <mailto:stewart.bryant@gmail.com>> wrote:
> 
> Hi Andy
> 
> Thank you for the review.
> 
> > On 30 Dec 2019, at 19:31, Andrew G. Malis <agmalis@gmail.com <mailto:agmalis@gmail.com>> wrote:
> > 
> > All,
> > 
> > I’ve been selected as an MPLS-RT reviewer for draft-bryant-mpls-sfl-control,
> > which is currently a candidate for MPLS WG adoption.
> > 
> > In general I believe that the draft is ready for WG adoption. However, I have a few minor comments which may be addressed either before or after WG adoption.
> > 
> > 1. The Security Considerations section says "It is assumed that this protocol is run in a well managed MPLS network with strict access controls preventing unwanted parties from generating MPLS OAM packets." While this is true of most (all?) MPLS networks, this assumption should also be stated in the abstract or Introduction as well.
> 
> I have added a line to the Introduction.
> 
> I generally work on the assumption that the purpose of the Abstract is to help the potential reader decide if they want to read the document (or more likely to help a search engine match the document against a query) and I am unconvinced as to value of the qualification text in deciding whether to read the rest of the document.
> 
> 
> > 
> > 2. Section 8 contains the line "Force references to appear with mkd [RFC3032] [RFC5036]". However, both references appear elsewhere as well, so this line can be removed.
> 
> The earlier references are in text that I included as a figure to force the layout style I wanted, and the markdown compiler ignores references in figures. The compiling process excluded unused references so they have to be forced. I have changed the text to
> 
> RFC Editor please remove this note which is used to force the following references to appear {{RFC3032}} {{RFC5036}}
> 
> > 
> > 3. I would move "I-D.ietf-mpls-sfl-framework" to the Normative References, as understanding it is necessary to understand this draft.
> > 
> 
> The framework is informational and so the reference would become a down-ref. I will leave it to the chairs and ADs on how they want to proceed with this. It is not unusual for a framework to be required reading and yet informational.
> 
> 
> > 4. The text in Section 2 is out of date. The current wording is:
> > 
> > The key words "MUST", "MUST NOT", "REQUIRED", "SHALL", "SHALL NOT", "SHOULD", "SHOULD NOT", "RECOMMENDED", "NOT RECOMMENDED", "MAY", and "OPTIONAL" in this document are to be interpreted as described in BCP 14 [RFC2119] [RFC8174] when, and only when, they appear in all capitals, as shown here.
> > 
> > RFC 8174 should also be added as a normative reference.
> 
> Done
> 
> > 
> > 5. When this draft was last updated, Stewart included the following in an email message to the MPLS WG:
> > 
> > "Next to do is authors of draft-bryant-mpls-sfl-control to check that 
> > this is suitable to be called by another protocol to act on its behalf.
> > When we are satisfied that that is that case and any consequentially 
> > necessary amendments have been make to the other drafts we will
> > request adoption of draft-bryant-mpls-sfl-control and then WGLC on all 
> > three."
> > 
> > There's been no follow-up indication that this analysis has occurred, or further revision to the draft. If it has, the authors should indicate as such on the list. If it hasn't, then this will serve as a reminder to the authors.
> 
> 
> George and I discussed this and concluded that the design was satisfactory.
> 
> I certainly indicated this to the chairs but I cannot remember if I posted that to the list. If any reader of this note (which will go to the MPLS list) has any technical concerns WRT this point, please raise them and we will work with you to address the issue.
> 
> New version (05) has been uploaded.
> 
> Best regards
> 
> Stewart
> 
> 
> 
> > 
> > Thanks,
> > Andy
> > 
>