Re: [spring] Ben Campbell's No Objection on draft-ietf-spring-oam-usecase-09: (with COMMENT)

Ben Campbell <ben@nostrum.com> Thu, 14 December 2017 23:34 UTC

Return-Path: <ben@nostrum.com>
X-Original-To: spring@ietfa.amsl.com
Delivered-To: spring@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 4BF5B127843; Thu, 14 Dec 2017 15:34:04 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.879
X-Spam-Level:
X-Spam-Status: No, score=-1.879 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, T_SPF_HELO_PERMERROR=0.01, T_SPF_PERMERROR=0.01, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
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 5kBBYwQKYQ33; Thu, 14 Dec 2017 15:34:02 -0800 (PST)
Received: from nostrum.com (raven-v6.nostrum.com [IPv6:2001:470:d:1130::1]) (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 76D55126DFF; Thu, 14 Dec 2017 15:34:02 -0800 (PST)
Received: from [10.0.1.99] (cpe-66-25-7-22.tx.res.rr.com [66.25.7.22]) (authenticated bits=0) by nostrum.com (8.15.2/8.15.2) with ESMTPSA id vBENY0ET050028 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NO); Thu, 14 Dec 2017 17:34:01 -0600 (CST) (envelope-from ben@nostrum.com)
X-Authentication-Warning: raven.nostrum.com: Host cpe-66-25-7-22.tx.res.rr.com [66.25.7.22] claimed to be [10.0.1.99]
From: Ben Campbell <ben@nostrum.com>
Message-Id: <CCE7B795-D86A-4D76-A4A1-186EC1A8A86A@nostrum.com>
Content-Type: multipart/signed; boundary="Apple-Mail=_8FBF5CC4-9701-4AD4-8FED-9830250370EF"; protocol="application/pgp-signature"; micalg="pgp-sha512"
Mime-Version: 1.0 (Mac OS X Mail 11.2 \(3445.5.20\))
Date: Thu, 14 Dec 2017 17:34:00 -0600
In-Reply-To: <LEXPR01MB0094B84B0C104FFE8219C9839C0A0@LEXPR01MB0094.DEUPRD01.PROD.OUTLOOK.DE>
Cc: spring@ietf.org, spring-chairs@ietf.org, aretana.ietf@gmail.com, bruno.decraene@orange.com, iesg@ietf.org, draft-ietf-spring-oam-usecase@ietf.org
To: "<Ruediger.Geib@telekom.de>" <Ruediger.Geib@telekom.de>
References: <151322313115.6120.8756591218425505436.idtracker@ietfa.amsl.com> <LEXPR01MB0094B84B0C104FFE8219C9839C0A0@LEXPR01MB0094.DEUPRD01.PROD.OUTLOOK.DE>
X-Mailer: Apple Mail (2.3445.5.20)
Archived-At: <https://mailarchive.ietf.org/arch/msg/spring/bUHzj0uv0gscrBeltHXUU4Xl1Q4>
Subject: Re: [spring] Ben Campbell's No Objection on draft-ietf-spring-oam-usecase-09: (with COMMENT)
X-BeenThere: spring@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: "Stacked Tunnels for Source Routing \(STATUS\)." <spring.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/spring>, <mailto:spring-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/spring/>
List-Post: <mailto:spring@ietf.org>
List-Help: <mailto:spring-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/spring>, <mailto:spring-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 14 Dec 2017 23:34:04 -0000

HI,

Thanks for your response. Additional comments inline. I removed sections that do not seem to need further discussion.

Thanks!

Ben.

> On Dec 14, 2017, at 8:33 AM, Ruediger.Geib@telekom.de wrote:
> 

[…]

> 
> Substantive Comments:
> 
> 
> - [BC] General: I note there has been discussion about why this draft is Informational rather than something else. There's an explanation in the shepherd's writeup. It would be helpful to have the same explanation as a note in the draft. (People rarely read the shepherd's report once an RFC is
> published.)
> 
>   [RG] I'd suggest to add this explanation as a last section of the Introduction:
> 
>   [I-D.ietf-mpls-spring-lsp-ping] discusses SR OAM applicability and
>   MPLS traceroute enhancements adding functionality to the use cases
>   described by this document.
> 
>    NEW: The document describes both use cases and a standalone monitoring framework. The monitoring system re-uses existing IETF OAM protocols and leverage Segment Routing (Source Routing) to allow a single device to both sends and receives its own probing packets. As a consequence, there are no new interoperability considerations. Standard Track is not required and Informational status seems appropriate.

That helps, thanks! You might consider changing “seems appropriate” to “is appropriate."

> 
> - [BC]  3, last paragraph: " Further options, like deployment of a PMS connecting to the MPLS domain by a tunnel only require more thought, as this implies security aspects." I have trouble parsing that. Is it intended as an open issue, or a statement that the "further options" are out of scope? Also, consider deleting the word "only".
> 
>   [RG]
> 
>   OLD: Further options, like
>   deployment of a PMS connecting to the MPLS domain by a tunnel only
>   require more thought, as this implies security aspects.  MPLS so far
>   separates networks securely by avoiding tunnel access to MPLS
>   domains.
> 
>    NEW: The option to connect a PMS to an MPLS domain by a tunnel may be attractive to some operators. MPLS so far separates networks securely by avoiding tunnel access to MPLS domains. Tunnel based access of a PMS to an MPLS domain is out of scope of this document, as it implies additional security aspects.

That helps, thanks!


> 
> 
> - [BC]  4.1, 2nd to last paragraph:
> I'm not sure what to make of the "In theory at least," prefix. Normally IETF RFCs are about what (we hope) works in _practice_.
> 
>   [RG]
>   OLD: In theory at least, a single PMS is able to monitor data plane
>   availability of all LSPs in the domain.  The PMS may be a router, .....
> 
>    NEW: A single PMS can detect the complete MPLS control- and data plane topology. A reliable deployment requires two separated PMS. Scalable permanent surveillance of a set of LSPs could require deployment of several PMS. The PMS may be a router, …..

That works, but you might consider changing the first two sentences along the following lines:
“While a single PMS can detect … , a reliable deployment requires two separated PMS."


> 
> 
> -- [BC]  10, last paragraph: I don't understand the intent of this paragraph.
> 
>   [RG]
> 
>   OLD:    A PMS may operate routine measurements.  If these are automated, care
>   must be taken to avoid unintended traffic insertion.  On the other
>   hand, large scale operation based on operator configuration itself is
>   a source of unintended misconfigurations and should be avoided.
> 
>   NEW: A PMS may be configured to permanently surveil a set of LSPs. Changes of the Segment Routing topology may occur at any time. The PMS design must stop a permanent LSP measurement, as soon as it detects that its locally stored SR domain topology information related to an LSP to be monitored is stale.

Sorry, I’m still not following it. Is the point that a PMS may continuously monitor a set of LSPs, but it needs to stop doing that for a particular LSP if that LSP changes?

[…]