[OSPF] tsv-dir review of draft-ietf-ospf-encapsulation-cap-06
Joe Touch <touch@strayalpha.com> Thu, 24 August 2017 04:39 UTC
Return-Path: <touch@strayalpha.com>
X-Original-To: ospf@ietfa.amsl.com
Delivered-To: ospf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 0F2981329CF; Wed, 23 Aug 2017 21:39:10 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.99
X-Spam-Level:
X-Spam-Status: No, score=-1.99 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, T_SPF_PERMERROR=0.01] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=strayalpha.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 7U6AOxg6q_cV; Wed, 23 Aug 2017 21:39:08 -0700 (PDT)
Received: from server217-3.web-hosting.com (server217-3.web-hosting.com [198.54.115.226]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 41D211323F7; Wed, 23 Aug 2017 21:39:08 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=strayalpha.com; s=default; h=Content-Transfer-Encoding:Content-Type: In-Reply-To:MIME-Version:Date:Message-ID:From:Cc:References:To:Subject:Sender :Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id:List-Help: List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=tQbvlt+1F6NizoNJeuxKQv2ZPtahYYRAD1aOs1U0h1A=; b=wZtU7XaUNC+g789dFLyiBKcN72 EIOmmQCp+3dWT85s2Yv/Q2ixqWp2cVnfXDhdr7Wk4fSuQ58PvOJFR5qqBUUJX0InOGBqNoome/xsd l+kcpayTT76T3wpreHfPsnDYWjl0VmNyTFxVJouvYh046PqZm6kEwa18XTzinufBsX4HRjHfJAs7f p7qUmSsArFBfMQjIog4gAH7FZZhpcO0j/hVVkWhEWrl6QWxyk9Hf0XjUc4DkkPGz2/CIsPpk/ar45 TBC06CJKo4OVgUPXWcCHMCUUwbpxsdu0rSQQhyMuOrBOJx1+BmBqziSoHeGm63uRbt6Yn2S9A7msN ybcVRVaA==;
Received: from cpe-172-250-240-132.socal.res.rr.com ([172.250.240.132]:52647 helo=[192.168.1.189]) by server217.web-hosting.com with esmtpsa (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128) (Exim 4.87) (envelope-from <touch@strayalpha.com>) id 1dkjv8-001aJr-Lr; Thu, 24 Aug 2017 00:39:07 -0400
To: IETF discussion list <ietf@ietf.org>, ospf@ietf.org
References: <150280426212.21081.15543071828535131713.idtracker@ietfa.amsl.com>
Cc: TSV Dir <tsv-dir@ietf.org>
From: Joe Touch <touch@strayalpha.com>
Message-ID: <7ca66dbe-1437-482d-a785-d5e48a558ccb@strayalpha.com>
Date: Wed, 23 Aug 2017 21:39:04 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0
MIME-Version: 1.0
In-Reply-To: <150280426212.21081.15543071828535131713.idtracker@ietfa.amsl.com>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
Content-Language: en-US
X-OutGoing-Spam-Status: No, score=-1.0
X-AntiAbuse: This header was added to track abuse, please include it with any abuse report
X-AntiAbuse: Primary Hostname - server217.web-hosting.com
X-AntiAbuse: Original Domain - ietf.org
X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12]
X-AntiAbuse: Sender Address Domain - strayalpha.com
X-Get-Message-Sender-Via: server217.web-hosting.com: authenticated_id: touch@strayalpha.com
X-Authenticated-Sender: server217.web-hosting.com: touch@strayalpha.com
X-Source:
X-Source-Args:
X-Source-Dir:
X-From-Rewrite: unmodified, already matched
Archived-At: <https://mailarchive.ietf.org/arch/msg/ospf/4wHcD5shQjqlyVjcb6XzzI7b8Fc>
Subject: [OSPF] tsv-dir review of draft-ietf-ospf-encapsulation-cap-06
X-BeenThere: ospf@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: The Official IETF OSPG WG Mailing List <ospf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ospf>, <mailto:ospf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ospf/>
List-Post: <mailto:ospf@ietf.org>
List-Help: <mailto:ospf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ospf>, <mailto:ospf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 24 Aug 2017 04:39:10 -0000
Hi, all, I am the assigned TSV-ART reviewer for draft-ietf-ospf-encapsulation-cap For background on TSV-ART, please see the FAQ at <https://trac.ietf.org/trac/tsv/wiki/TSV-Directorate>. Please resolve these comments along with any other Last Call comments you may receive. Joe ------ I've reviewed this document as part of the transport area directorate's ongoing effort to review key IETF documents. These comments were written primarily for the transport area directors, but are copied to the document's authors for their information and to allow them to address any issues raised. When done at the time of IETF Last Call, the authors should consider this review together with any other last-call comments they receive. Please always CC tsv-dir@ietf.org if you reply to or forward this review. "This draft has serious issues, described in the review, and needs to be rethought." Although I found no transport issues, there are many fundamental problems that need to be corrected before this document should proceed. These are summarized below. Sec1: The introduction jumps too quickly into a list of situations in which tunnels are used, two of which are emerging (currently only drafts). There are many better and more established examples (going back to the M-Bone, the 6-Bone, etc., to VPNs, network virtualization, etc.). Terms are used before defined (what is an ingress? are you referring to the router or host at which a tunnel is attached, or the input to the tunnel itself?). The last two sentences should begin the intro, which should expand and explain what is meant (e.g., by "egress tunneling" and the reason for wanting to encode tunnels as LSAs. Sec2: The terminology section is incomplete. The terms tunnel, ingress, egress, egress tunneling, etc. are not defined in RFC7770. Only the LSA terminology is relevant from that RFC. Other terms need to be defined in this doc or used from others. Sec3: This section is written as if the "Encapsulation Capability TLV" is already defined (in RFC7770), rather than being defined herein. The section title doesn't match this name, which is confusing. The term TLV is itself confusing, as this is really a single TLV of the RI Opaque LSA set of types, which itself is represented as a (sub)TLV list. I appreciate that this terminology was made confusing in previous, established documents, but a bit of explanation would go a long way - as would showing the top-level OSPF RI Opaque LSA and where the new TBD1 value would appear - before diving into the structure inside this new "TLV". Sec 4: the discussion should clarify what happens if the length field is longer than the fields indicated by the sub-TLVs (is this an error or padding to be ignored; does the latter present a covert channel). additionally, this section refers to RFC5512 - which is being obsoleted by ietf-idr-tunnel-encaps - and this pending ID is used as the primary reference in Secs 5.1 and 5.2. References to RFC5512 should be replaced with that ID (which I'll assume will be published concurrently). Sec 5: this section refers to the concept of an "invalid" sub-TLV; given the previous sentence explains that unknown sub-TLVs are silently ignored, then what exactly is an invalid sub-TLV? It further isn't clear why 2 octets of type and 2 octets of length are needed (granted it provides alignment, but is that really important for this sort of application-layer protocol?) Sec 5.1 and 5.2: it is unclear why the two fundamental sub-TLVs of this TLV are defined elsewhere; in specific, the example of the sub-TLV should appear here (actually for each of the sub-TLVs). Sec 5.3 infers IP address version from length, when the version could (should) easily encoded either as different types (you have 65K of them!) or using an additional few bits (e.g., carved out of the excesses noted in Sec 5). Sec 5.4 - I had to thread through several different sections of the IDR ID to figure out what Color meant and how it is used. Granted that is a different problem in a different doc, IMO this doc should include enough of a summary that this sort of 'traceback' should not be needed to understand what is being described. Further, the other doc doesn't even explain Color sufficiently, IMO. Secs 5.6 and 5.7 - There are many rules for how the outer encapsulation header is composed, and many fields it may affect beyond QoS and UDP destination port. This section is incomplete in describing values for the outer header(s) or relationships to the inner header(s). Sec 6 - I would encourage moving this section earlier, before the details of how the LSA is composed. Sec 7 - the document should describe where experimental values can/should be safely used and how collisions are handled; it should also explain what happens when a reserved value is seen in an LSA. ----
- [OSPF] tsv-dir review of draft-ietf-ospf-encapsul… Joe Touch
- Re: [OSPF] tsv-dir review of draft-ietf-ospf-enca… Acee Lindem (acee)
- [OSPF] 答复: tsv-dir review of draft-ietf-ospf-enca… Xuxiaohu
- Re: [OSPF] tsv-dir review of draft-ietf-ospf-enca… bruno.decraene