Re: [CCAMP] WG Last Call: WSON documents - draft-ietf-ccamp-general-constraint-encode
Lou Berger <lberger@labn.net> Tue, 29 October 2013 19:07 UTC
Return-Path: <lberger@labn.net>
X-Original-To: ccamp@ietfa.amsl.com
Delivered-To: ccamp@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 26CB911E8110 for <ccamp@ietfa.amsl.com>; Tue, 29 Oct 2013 12:07:49 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.203
X-Spam-Level:
X-Spam-Status: No, score=-102.203 tagged_above=-999 required=5 tests=[AWL=0.396, BAYES_00=-2.599, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id p6RgauyfHu6z for <ccamp@ietfa.amsl.com>; Tue, 29 Oct 2013 12:07:38 -0700 (PDT)
Received: from qproxy1-pub.mail.unifiedlayer.com (qproxy1-pub.mail.unifiedlayer.com [173.254.64.10]) by ietfa.amsl.com (Postfix) with SMTP id AA40911E8263 for <ccamp@ietf.org>; Tue, 29 Oct 2013 12:07:35 -0700 (PDT)
Received: (qmail 24361 invoked by uid 0); 29 Oct 2013 19:07:11 -0000
Received: from unknown (HELO box313.bluehost.com) (69.89.31.113) by qproxy1.mail.unifiedlayer.com with SMTP; 29 Oct 2013 19:07:11 -0000
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=labn.net; s=default; h=Content-Transfer-Encoding:Content-Type:In-Reply-To:References:Subject:To:MIME-Version:From:Date:Message-ID; bh=Wq67fQhBd+YXERLPWqWSoOHoHghD/ddNXC1Yo3CCjPs=; b=lQD/qIJyDz6mIEfHHDY6QVYQH04fteX39BkbxamlcKu7GaKj0WrViuRhZva/0APWBjQySh+l+t/ZY9kkzPvKOAg1kVy0KyXRJ7Vcg9SrY3PXu/5xftLK/jrcgpsiewai;
Received: from box313.bluehost.com ([69.89.31.113]:52861 helo=[127.0.0.1]) by box313.bluehost.com with esmtpa (Exim 4.80) (envelope-from <lberger@labn.net>) id 1VbE0W-0005Td-Kz; Tue, 29 Oct 2013 12:27:12 -0600
Message-ID: <526FFDF8.1060101@labn.net>
Date: Tue, 29 Oct 2013 14:27:04 -0400
From: Lou Berger <lberger@labn.net>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.0.1
MIME-Version: 1.0
To: CCAMP <ccamp@ietf.org>, draft-ietf-ccamp-general-constraint-encode@tools.ietf.org
References: <524AF9A9.3040006@labn.net> <5266E138.8080605@labn.net>
In-Reply-To: <5266E138.8080605@labn.net>
X-Enigmail-Version: 1.5.2
Content-Type: text/plain; charset="ISO-8859-1"
Content-Transfer-Encoding: 7bit
X-Identified-User: {1038:box313.bluehost.com:labnmobi:labn.net} {sentby:smtp auth 69.89.31.113 authed with lberger@labn.net}
Subject: Re: [CCAMP] WG Last Call: WSON documents - draft-ietf-ccamp-general-constraint-encode
X-BeenThere: ccamp@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Discussion list for the CCAMP working group <ccamp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ccamp>, <mailto:ccamp-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ccamp>
List-Post: <mailto:ccamp@ietf.org>
List-Help: <mailto:ccamp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ccamp>, <mailto:ccamp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 29 Oct 2013 19:07:50 -0000
Authors, I have some comments on this document. A number are strictly editorial. Note that I'm the document shepherd, see RFC 4858 for more information. - Please address my general comments on the WSON document set - I'm a bit surprised that this document does *not* reference the rwa-info draft. Given that it provides the context for this document, it should be there, and probably as a normative reference. - Your references to [Switch] seem almost normative in nature. I suspect that this is not your intent as it is listed as an informative reference. Please revisit your references and ensure that the draft does not depend on any informative references. (Either change the text, or move the reference to normative.) Perhaps these are places where a reference to rwa-info are more appropriate. - An editorial comment on section ordering: Section 2 is ordered as follows: 2. Encoding.......................................................6 2.1. Link Set Field............................................6 2.2. Label Set Field...........................................8 2.3. Available Labels Sub-TLV.................................11 2.4. Shared Backup Labels Sub-TLV.............................11 2.5. Connectivity Matrix Sub-TLV..............................12 2.6. Port Label Restriction sub-TLV...........................13 Is there a logical order to these sections? They don't seem to be aligned with the rwa-info draft which has: 4. Node Information (General).....................................8 4.1. Connectivity Matrix.......................................9 4.2. Shared Risk Node Group...................................10 6. Link Information (General)....................................17 6.6. Port Label (Wavelength) Restrictions.....................18 7. Dynamic Components of the Information Model...................21 7.1. Dynamic Link Information (General).......................22 Perhaps it makes sense from a readability standpoint to realign to match rwa-info? - Section 2 says: A type-length-value (TLV) encoding of the general connectivity and label restrictions and availability extensions is given in this section. This encoding is designed to be suitable for use in the GMPLS routing protocols OSPF [RFC4203] and IS-IS [RFC5307] and in the PCE protocol PCEP [PCEP]. Note that the information distributed in [RFC4203] and [RFC5307] is arranged via the nesting of sub-TLVs within TLVs and this document makes use of such constructs... Yet there are no types or (sub)TLVs actually defined in this document. As such, I think this document should not use the term "sub-TLV" for the information elements it defines. In most cases this just means substituting "the <XYZ_information_element> sub-TLV" with just "<XYZ_information_element>". Section 2 can be revised to read something along the lines of: This section provides encodings for the information elements defined in [RWA-INFO] that have general applicability. The encodings are designed to be suitable for use in the GMPLS routing protocols OSPF [RFC4203] and IS-IS [RFC5307] and in the PCE protocol PCEP [PCEP]. Note that the information distributed in [RFC4203] and [RFC5307] is arranged via the nesting of sub-TLVs within TLVs and this document defines elements to be used within such constructs... - section 2.2. Labels are variable in lengh and need not be 4 bytes long. This needs to be represented and accounted for in the encodings defined in this section. - Section 2.2.1 and 2.2.2 says "Num Labels (not used)" and "Num Labels is not used in this particular format since the Length parameter is sufficient to determine the number of labels in the list." Is there a reason to not set the Num Labels Field in these cases? Recall that labels are themselves variable length fields. - section A.2 s/With the Grid/Using the label format defined in [RFC 6205], with the Grid... s/Num Wavelengths/Num Labels (multiple places) +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ | 0 | Num Wavelengths = 40 | Length = 20 bytes | +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ shouldn't Num Labels either be 0 (if unused) or = 7 (assuming num labels used in this case)? That's it on this one, Lou On 10/22/2013 4:34 PM, Lou Berger wrote: > > All, > Given the recent draft submission deadline and only one comment being > received to date, we'd like to extend the WG more time for review. > > These drafts represent significant work by the authors and WG, so please > review and let the WG know what you think (positive or negative)! > > Please have all comments in by October 29. > > Thank you, > Lou (and Deborah) > > On 10/1/2013 12:34 PM, Lou Berger wrote: >> All, >> >> This mail begins working group last call on the WSON documents. As >> there are 6 documents in this set, the last call will be three weeks. >> The documents included in the last call are: >> >> http://tools.ietf.org/html/draft-ietf-ccamp-rwa-info-18 >> (Informational, IPR Disclosed) >> >> http://tools.ietf.org/html/draft-ietf-ccamp-general-constraint-encode-11 >> (Standards Track, IPR Disclosed) >> >> http://tools.ietf.org/html/draft-ietf-ccamp-rwa-wson-encode-21 >> (Standards Track) >> >> http://tools.ietf.org/html/draft-ietf-ccamp-gmpls-general-constraints-ospf-te-05 >> (Standards Track, IPR Disclosed) >> >> http://tools.ietf.org/html/draft-ietf-ccamp-wson-signal-compatibility-ospf-12 >> (Standards Track, IPR Disclosed) >> >> http://tools.ietf.org/html/draft-ietf-ccamp-wson-signaling-06 (Standards >> Track) Also has one open issue that will need to be resolved as part of >> LC, see http://trac.tools.ietf.org/wg/ccamp/trac/ticket/52. >> >> This working group last call ends on October 22. Comments should be >> sent to the CCAMP mailing list. Please remember to include the >> technical basis for any comments. >> >> Positive comments, e.g., "I've reviewed this document and believe it is >> ready for publication", are welcome! >> >> Please note that we're still missing some IPR statements. Any >> forthcoming publication request will be delayed by late IPR >> statements/disclosures. >> >> >> Thank you, >> Lou (and Deborah) >> _______________________________________________ >> CCAMP mailing list >> CCAMP@ietf.org >> https://www.ietf.org/mailman/listinfo/ccamp >> >> >> >> > _______________________________________________ > CCAMP mailing list > CCAMP@ietf.org > https://www.ietf.org/mailman/listinfo/ccamp > > > >
- [CCAMP] WG Last Call: WSON documents - rwa-info, … Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - rwa-in… Margaria, Cyril (Coriant - DE/Munich)
- Re: [CCAMP] WG Last Call: WSON documents - rwa-in… Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - rwa-in… Leeyoung
- Re: [CCAMP] WG Last Call: WSON documents - rwa-in… Giovanni Martinelli (giomarti)
- Re: [CCAMP] WG Last Call: WSON documents - rwa-in… Giovanni Martinelli (giomarti)
- Re: [CCAMP] WG Last Call: WSON documents - rwa-in… Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - rwa-in… Leeyoung
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - rwa-in… Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - rwa-in… Leeyoung
- Re: [CCAMP] WG Last Call: WSON documents - rwa-in… Leeyoung
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Leeyoung
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Leeyoung
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Leeyoung
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Leeyoung
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Leeyoung
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Leeyoung
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Leeyoung
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Leeyoung
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Leeyoung
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Leeyoung
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Leeyoung
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Lou Berger
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Giovanni Martinelli (giomarti)
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Giovanni Martinelli (giomarti)
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Giovanni Martinelli (giomarti)
- Re: [CCAMP] WG Last Call: WSON documents - draft-… Lou Berger