[spring] Barry Leiba's No Objection on draft-ietf-spring-srv6-network-programming-19: (with COMMENT)
Barry Leiba via Datatracker <noreply@ietf.org> Mon, 21 September 2020 17:17 UTC
Return-Path: <noreply@ietf.org>
X-Original-To: spring@ietf.org
Delivered-To: spring@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id AEB7C3A08F8; Mon, 21 Sep 2020 10:17:12 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Barry Leiba via Datatracker <noreply@ietf.org>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-spring-srv6-network-programming@ietf.org, spring-chairs@ietf.org, spring@ietf.org, Bruno Decraene <bruno.decraene@orange.com>, Joel Halpern <jmh@joelhalpern.com>, jmh@joelhalpern.com
X-Test-IDTracker: no
X-IETF-IDTracker: 7.17.0
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Barry Leiba <barryleiba@computer.org>
Message-ID: <160070863224.16553.3215584446210310666@ietfa.amsl.com>
Date: Mon, 21 Sep 2020 10:17:12 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/spring/RtEjE5uYqEwIZlEQSYRrf0re2kM>
Subject: [spring] Barry Leiba's No Objection on draft-ietf-spring-srv6-network-programming-19: (with COMMENT)
X-BeenThere: spring@ietf.org
X-Mailman-Version: 2.1.29
List-Id: "Source Packet Routing in NetworkinG \(SPRING\)" <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: Mon, 21 Sep 2020 17:17:13 -0000
Barry Leiba has entered the following ballot position for draft-ietf-spring-srv6-network-programming-19: No Objection When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.) Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html for more information about IESG DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-spring-srv6-network-programming/ ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- Throughout the document: It should be “an SID”, “an FIB”, “an RIR”, and some others, not “a”, because one reads these as “ess-eye-dee” and “eff-eye-bee”, not as the expansions thereof. — Section 3.1 — An SRv6 endpoint behavior MAY require additional information I think this is a statement of fact, so “may” (or “might”), not “MAY”. — Section 3.2 — Nit: The plural of “SID” is “SIDs”, without an apostrophe. The provider historically deployed IPv6 and assigned infrastructure address from a portion of the fc00::/7 prefix Nit: “addresses”. In another example, a large mobile and fixed line service provider Nit: hyphenate “fixed-line”. IPv6 address consumption in both these examples is minimum, Nit: “minimal” (also, put a comma before “respectively”). A remote node uses the IANA behavior codepoint to map the received SID (B:N:FUNCT) to a behavior. I don’t know what “IANA behavior codepoint” means. As the rest of the sentence makes it clear that this is mapping to “a behavior”, maybe it’s better to say “registered codepoint”, or some such? Or, as you say a couple of paragraphs down, “Endpoint Behavior codepoint”? In any case, please be consistent about “IANA behavior”, “IANA Behavior”, and “IANA Endpoint Behavior”. My preference would be to avoid saying “IANA” in these, but use your judgment on what’s most understandable and clear. o Assign an SRv6 Locator 2001:db8:bbbb:3::/64 to the Router 3 in their SR Domain What is “the Router 3” (and router 4)? There’s no introduction nor diagram that says. Also, please be consistent in capitalization of these. — Section 3.3 — routing protocol specific aspects that are outside the scope of this Nit: “routing-protocol-specific” needs to be hyphenated. As that’s awkward, you might want to reword to avoid it, “are specific to the routing protocol and are outside the scope....” — Section 4 — The pseudocode describing these behaviors detail local processing Nit: “details”, singular, to match “pseudocode”. — Section 4.1 — The End behavior operates on the same FIB table (i.e. VRF, L3 relay id) associated to the packet. Is “i.e.” really what you want here? How does “VRF, L3 relay” equate to “FIB table”? — Section 4.16.1.3 — segments left to 0. The SDN controller knows that no-other node Nit: “no other” should not be hyphenated. For that matter, the word “other” isn’t needed anyway, because you have “after” in there. -as part of the decapsulation process the egress PE is required to terminates less bytes from the packet. I can’t figure this out. It looks like it should be “required to terminate”, but I don’t know what it means to “terminate less bytes”. Can you reword this? to the lookup engine in the forwarding ASIC. “ASIC” needs to be expanded. As this is the only place you use it, I suggest just using the expansion and not bothering with the abbreviation. — Section 7 — This document introduces SRv6 Endpoint and SR Policy Headend behaviors for implementation on SRv6 capable nodes in the network. As such, this document does not introduce any new security considerations. I’m not convinced of this. It seems that misuse (such as injection or alteration) of some of these Behaviors could, for example, result in packets being forwarded to nodes they were not intended to go to. Is it not important to discuss issues such as that: how these Behaviors might be attacked? Is that really fully covered in 8754 and 8402? — Section 8.1 — The presence of SIDs in the IGP do not imply any routing semantics Nit: “does not”, to match the singular subject “the presence”. an IPv6 address is solely governed by the, non-SID-related, IGP Nit: remove both commas. not governed neither influenced in any way by a SID advertisement Nit: make it “neither governed nor influenced” build TI-LFA [I-D.ietf-rtgwg-segment-routing-ti-lfa] based FRR solutions Nit: there needs to be a hyphen in there, but the citation gets in the way. Make it, “build FRR solutions based on TI-LFA [I-D.ietf-rtgwg-segment-routing-ti-lfa]”. — Section 8.4 — For example, a BGP-LS advertisement of H.Encaps behavior would describe the capability of node N to perform a H.Encaps behavior, specifically it would describe how many SIDs could be pushed by N without significant performance degradation. This is a comma splice. Split the sentence before “specifically” (and put a comma after “specifically”).
- [spring] Barry Leiba's No Objection on draft-ietf… Barry Leiba via Datatracker
- Re: [spring] Barry Leiba's No Objection on draft-… Joel M. Halpern
- Re: [spring] Barry Leiba's No Objection on draft-… Pablo Camarillo (pcamaril)
- Re: [spring] Barry Leiba's No Objection on draft-… Barry Leiba
- Re: [spring] Barry Leiba's No Objection on draft-… Pablo Camarillo (pcamaril)