Re: [trill] [Idr] WGLC for draft-ietf-idr-ls-trill-03
Donald Eastlake <d3e3e3@gmail.com> Tue, 03 April 2018 04:11 UTC
Return-Path: <d3e3e3@gmail.com>
X-Original-To: trill@ietfa.amsl.com
Delivered-To: trill@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 778D71270A3; Mon, 2 Apr 2018 21:11:59 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.449
X-Spam-Level:
X-Spam-Status: No, score=-2.449 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_ENVFROM_END_DIGIT=0.25, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7, 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=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 9WkFvfabNh7S; Mon, 2 Apr 2018 21:11:57 -0700 (PDT)
Received: from mail-it0-x229.google.com (mail-it0-x229.google.com [IPv6:2607:f8b0:4001:c0b::229]) (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 0C2871200FC; Mon, 2 Apr 2018 21:11:56 -0700 (PDT)
Received: by mail-it0-x229.google.com with SMTP id h143-v6so21458670ita.4; Mon, 02 Apr 2018 21:11:56 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=4eYeR8OcCSLopcvEGey/6MV7IHTq+0DrxzRXn1bvtGc=; b=EtzQ/+Lx59fjdC0tRaK6nQcxqFyuvM45c5Pok2RkcOsTZG/gcVRcL6SR9ZNUU3HnzI tajrN3+cE/vrYIFcYu4yQQZgXbThfd2+Wss53cMdQfIoPKTpIAckU1bLhBuvDSzjLJ8e ilFHiEXMCrYIjSOl5Djz2ydE63juqeCX4D1y78NNQh4Uz3W8/9gBri4WG8pN0RnOZgyF CUtKP1epDB7WOEGOFWZ23Sdnhd4KHJJnljiwvUtmTlgTtb3qIq8m5CU8xdjJ+T5sDs9L ObCeJnpN6I031RGUhgV22Zae/Y+/HRYcj3jw5yrH9kVQsj2dyDDbyspT6/v4sAuX5qRu VPEg==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=4eYeR8OcCSLopcvEGey/6MV7IHTq+0DrxzRXn1bvtGc=; b=n064jkaz5gy9FVjxMjvqIpih+fbLMGzFlSu33ryhrEpwBMoSX3evkeqrL8TK7Mb1iC mOPi9UN6/iDj4a2uLgOFOZxGXjEJW4UritR30apFUNIBiJ0xP2XRRvhqxCyWNckI5RV4 aM+B+8VsJpZH6ege7MIvs4MddwboROWoH152Qv0xuh8TY6WKuRlncHZCf5t1Cw8AdGO4 c9x2c8K0Rh+QZiRQbGVKNNjGcwrOoO+0/Vh9u/aoi2ThEvdkwDlX6glKN2E9TAf33wTf slpDeEnXxaw34QSZXyWAwVWyUJSCLx2c1cRdYWnZPYD1xe5UAtKcOYgNc3h03xKLujou dVlQ==
X-Gm-Message-State: AElRT7FXOl2r8dV6wEF5xk+FiYRfei1h9qsPEaaI6sdcYMoq+BY9Kdb+ O5KTJAwGgQBjUzLlLS51WKSm0wq7zHs0dEm7oNQ=
X-Google-Smtp-Source: AIpwx4+AMPpaiwwTY4+07Ccwd3lstTsea9xyOjKwH8uo1BRh3GF6Xa8JsD928bv+LHl0suEGkf7YlARjxkeRw9grN4I=
X-Received: by 2002:a24:df04:: with SMTP id r4-v6mr3475845itg.105.1522728715906; Mon, 02 Apr 2018 21:11:55 -0700 (PDT)
MIME-Version: 1.0
Received: by 10.107.58.193 with HTTP; Mon, 2 Apr 2018 21:11:40 -0700 (PDT)
In-Reply-To: <74FFA21E-B80B-4DA4-B83D-A8D5F58AC8D3@juniper.net>
References: <F78B2461-A433-41FF-8997-4AEE8B0BE1A7@juniper.net> <74FFA21E-B80B-4DA4-B83D-A8D5F58AC8D3@juniper.net>
From: Donald Eastlake <d3e3e3@gmail.com>
Date: Tue, 03 Apr 2018 00:11:40 -0400
Message-ID: <CAF4+nEHx5zWCGx2EEJeQYDArpXA2fDMqb0+H1TSOo22jcuKDAw@mail.gmail.com>
To: "John G. Scudder" <jgs@juniper.net>
Cc: "idr@ietf. org" <idr@ietf.org>, draft-ietf-idr-ls-trill@ietf.org, trill IETF mailing list <trill@ietf.org>, Hannes Gredler <hannes@gredler.at>, Adrian Farrel <adrian@olddog.co.uk>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: <https://mailarchive.ietf.org/arch/msg/trill/LSWyVijFqVbh-bRIhYm48a2dwk8>
Subject: Re: [trill] [Idr] WGLC for draft-ietf-idr-ls-trill-03
X-BeenThere: trill@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: "Developing a hybrid router/bridge." <trill.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/trill>, <mailto:trill-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/trill/>
List-Post: <mailto:trill@ietf.org>
List-Help: <mailto:trill-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/trill>, <mailto:trill-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 03 Apr 2018 04:11:59 -0000
Hi John, Thanks for these comments, See below. On Wed, Mar 21, 2018 at 11:32 AM, John G. Scudder <jgs@juniper.net> wrote: > Hi All, > > This WGLC collected exactly one expression of support other than from > co-authors, from zhuangyan.zhuang@huawei.com. I'm afraid we don't have > consensus to progress the document right now, or evidence that the document > has received much in the way of review. > > I do note several problems that will need to be resolved: > > 1. There are six front-page authors. There should be five or fewer, see the > final bullet of > https://trac.ietf.org/trac/idr/wiki/Checklist%20for%20writing%20a%20BGP-related%20draft OK. > 2. The IANA section has > > IANA is requested to assign one Node Flag bit for "Layer 3 Gateway" > from the BGP-LS registry of BGP-LS Attribute TLVs. > > however, the registry BGP-LS Node Descriptor, Link Descriptor, Prefix > Descriptor, and Attribute TLVs actually has no provision for registering new > flag bits. https://tools.ietf.org/html/rfc7752#section-3.3.1.1 simply lists > them as "Reserved for future use". Possibly address this by requesting IANA > to create the registry within the "Border Gateway Protocol - Link State > (BGP-LS) Parameters" and allocate your bit from there. You would also have > to make your draft update RFC 7752. I've cc'd Adrian and Hannes, the > Designated Experts for that group of registries, in case they have further > comment. Creating a registry in this document seems like the appropriate course of action. > 3. Not precisely a problem, but are there implementations of the draft? As > you know, if there aren't, by IDR WG convention we will stall until we do > have some, even if we do complete a WGLC. I am not aware of any right now. > 4. I'd kind of prefer you remove tables 1 and 2. They aren't authoritative > and table 1 is actually obsolete. OK. > 5. Section 3.3.1.1 has > > +----------+----------------------------+-----------+ > | Bit | Description | Reference | > +----------+----------------------------+-----------+ > | 'G' | Layer 3 Gateway Bit | [RFC7176] | > | Reserved | Reserved for future use | | > +----------+----------------------------+-----------+ > > I think the reference must be wrong. RFC 7176 ("Transparent Interconnection > of Lots of Links (TRILL) Use of IS-IS") doesn't include the string "gateway" > at all, so if the ref is correct, it's at best obscure. > > But in any case, per #2 above, probably this is not the right table in the > right place. I'll see what I can do to improve this. > 6. I have read section 3.3.1.2 several times and don't understand it. Other > than "what does it mean?" I wonder what the intent of table 6 is. It kind of > resembles the table in section 5.2 of RFC 7176? I'm confused. I think Section 3.3.1.2 is enumerating TRILL link state information that can be transported in BGP-LS using the opaque node attribute TLV as of some point in time (perhaps when the -00 draft was created). However, additional possible TRILL link state information has been specified since [RFC7176]. In any case, I think this subsection should be recast in a more general way and the Table 6 should probably be eliminated. > 7. Is this a typo? > > o Does any fixed length TLV correspond to the TLV Length field in > this document ? > > Do you mean "every"? I believe it should be "every". > 8. In this: > > opaque TLV support the range of ISIS-TLV/SUB-TLV shown in Table > 3, and link TLVs support the range in Figure 8. > > there is no figure 8 in the document. I think it is just trying to say that you check that items of link state information are inside the right envelope; but, it is not very clear and refers to Figure 8 when it probably means Table 7. I'll try to improve it. > 9. I don't know if the security section will fly, but then again I don't > know if it won't. Once the doc is revised to address the above I'll ask the > Sec Dir for a review. OK. Thanks, Donald =============================== Donald E. Eastlake 3rd +1-508-333-2270 (cell) 155 Beaver Street, Milford, MA 01757 USA d3e3e3@gmail.com > Thanks, > > --John > > > On Oct 19, 2017, at 3:33 PM, John Scudder <jgs@juniper.net> wrote: > > Hi All, > > An IDR working group last call has been requested for > draft-ietf-idr-ls-trill-03. Please reply to the list with your comments. As > usual note we cannot advance the draft without participation from the group. > Please get your comments in before November 3, 2017. > > Authors, please confirm that any relevant IPR has been disclosed. > > https://urldefense.proofpoint.com/v2/url?u=https-3A__tools.ietf.org_html_draft-2Dietf-2Didr-2Dls-2Dtrill-2D03&d=DwICAg&c=HAkYuh63rsuhr6Scbfh0UjBXeMK-ndb3voDTXcWzoCI&r=hLt5iDJpw7ukqICc0hoT7A&m=YQIHKfaMAUOrl3hLdgq6nw1PcXtmtpICF3XlGCVFreQ&s=6fwhGLRqq-HwpYC9ZJyaR60B_kl8F-1BV0Gd5B7XPl8&e= > > Thanks, > > --John > > _______________________________________________ > Idr mailing list > Idr@ietf.org > https://urldefense.proofpoint.com/v2/url?u=https-3A__www.ietf.org_mailman_listinfo_idr&d=DwICAg&c=HAkYuh63rsuhr6Scbfh0UjBXeMK-ndb3voDTXcWzoCI&r=hLt5iDJpw7ukqICc0hoT7A&m=YQIHKfaMAUOrl3hLdgq6nw1PcXtmtpICF3XlGCVFreQ&s=1Hl0Amflm0UF1PXY9sJgdlNF1i0jXiSk9xReayA2hF0&e= > >
- Re: [trill] WGLC for draft-ietf-idr-ls-trill-03 Sujay Gupta
- [trill] WGLC for draft-ietf-idr-ls-trill-03 John Scudder
- Re: [trill] WGLC for draft-ietf-idr-ls-trill-03 Donald Eastlake
- Re: [trill] WGLC for draft-ietf-idr-ls-trill-03 Haoweiguo
- Re: [trill] WGLC for draft-ietf-idr-ls-trill-03 Susan Hares
- Re: [trill] [Idr] WGLC for draft-ietf-idr-ls-tril… John G. Scudder
- Re: [trill] [Idr] WGLC for draft-ietf-idr-ls-tril… Donald Eastlake
- Re: [trill] [Idr] WGLC for draft-ietf-idr-ls-tril… John G. Scudder