[Gen-art] Re: GEN-ART review of draft-ietf-ccamp-loose-path-reopt-01
"Adrian Farrel" <adrian@olddog.co.uk> Mon, 06 February 2006 16:33 UTC
Received: from localhost.cnri.reston.va.us ([127.0.0.1] helo=megatron.ietf.org) by megatron.ietf.org with esmtp (Exim 4.32) id 1F69Iy-0006xu-Ra; Mon, 06 Feb 2006 11:33:32 -0500
Received: from odin.ietf.org ([132.151.1.176] helo=ietf.org) by megatron.ietf.org with esmtp (Exim 4.32) id 1F69Iw-0006xR-Sk for gen-art@megatron.ietf.org; Mon, 06 Feb 2006 11:33:31 -0500
Received: from ietf-mx.ietf.org (ietf-mx [132.151.6.1]) by ietf.org (8.9.1a/8.9.1a) with ESMTP id LAA01856 for <gen-art@ietf.org>; Mon, 6 Feb 2006 11:31:41 -0500 (EST)
Received: from protext01.itu.ch ([156.106.192.41]) by ietf-mx.ietf.org with esmtp (Exim 4.43) id 1F69Ux-0006z7-Tc for gen-art@ietf.org; Mon, 06 Feb 2006 11:45:56 -0500
Received: from protext01.itu.ch ([156.106.192.41]) by protext01.itu.ch with Microsoft SMTPSVC(5.0.2195.6713); Mon, 6 Feb 2006 17:32:46 +0100
Received: From mail5.itu.ch ([156.106.192.21]) by protext01.itu.ch (WebShield SMTP v4.5 MR1a); id 1139243565449; Mon, 6 Feb 2006 17:32:45 +0100
Received: from Puppy ([156.106.204.73]) by mail5.itu.ch (8.13.5/8.13.5) with SMTP id k16GWfXU084877; Mon, 6 Feb 2006 17:32:42 +0100 (MET)
Message-ID: <029a01c62b3b$8461bae0$49cc6a9c@Puppy>
From: Adrian Farrel <adrian@olddog.co.uk>
To: JP Vasseur <jvasseur@cisco.com>, Alex Zinin <zinin@psg.com>
References: <31E5D26B8A12D889312D466C@B50854F0A9192E8EC6CDA126> <FB85FC92-E66E-4277-9D7D-6FA5D95A677C@cisco.com> <1D8A9BF8-A9CF-4D9F-BF7B-6822B80C2DFB@cisco.com>
Date: Mon, 06 Feb 2006 16:36:02 -0000
MIME-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: 7bit
X-Priority: 3
X-MSMail-Priority: Normal
X-Mailer: Microsoft Outlook Express 6.00.2800.1158
X-MimeOLE: Produced By Microsoft MimeOLE V6.00.2800.1165
X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-2.0.2 (mail5.itu.ch [156.106.192.21]); Mon, 06 Feb 2006 17:32:45 +0100 (MET)
X-OriginalArrivalTime: 06 Feb 2006 16:32:46.0121 (UTC) FILETIME=[F5F50190:01C62B3A]
X-Spam-Score: 0.0 (/)
X-Scan-Signature: a492040269d440726bfd84680622cee7
Content-Transfer-Encoding: 7bit
Cc: raymond_zhang@infonet.com, Kireeti Kompella <kireeti@juniper.net>, gen-art@ietf.org, Bill Fenner <fenner@research.att.com>, Yuichi Ikejiri <y.ikejiri@ntt.com>, "JP Vasseur (Cisco)" <jpv@cisco.com>
Subject: [Gen-art] Re: GEN-ART review of draft-ietf-ccamp-loose-path-reopt-01
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.5
Precedence: list
Reply-To: Adrian Farrel <adrian@olddog.co.uk>
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www1.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www1.ietf.org/pipermail/gen-art>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www1.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
Sender: gen-art-bounces@ietf.org
Errors-To: gen-art-bounces@ietf.org
This WG chair is OK with the changes. Looking at the I-D tracker, this I-D is pending AD write-up, so it hasn't gone out for IESG review yet. I would suggest an immediate submission of the new version of the I-D so that the IESG does not waste its time on the points that Harald caught. Thanks, Adrian ----- Original Message ----- From: "JP Vasseur" <jvasseur@cisco.com> To: "Alex Zinin" <zinin@psg.com> Cc: "Harald Tveit Alvestrand" <harald@alvestrand.no>; <gen-art@ietf.org>; "JP Vasseur (Cisco)" <jpv@cisco.com>; "Yuichi Ikejiri" <y.ikejiri@ntt.com>; <raymond_zhang@infonet.com>; "Kireeti Kompella" <kireeti@juniper.net>; "Adrian Farrel" <adrian@olddog.co.uk>; "Alex Zinin" <zinin@psg.com>; "Bill Fenner" <fenner@research.att.com> Sent: Monday, February 06, 2006 4:15 PM Subject: Re: GEN-ART review of draft-ietf-ccamp-loose-path-reopt-01 > Hi Alex, > > Could you let me know if you are ok with the changes (incorporating > Harald's suggestions made during GEN-ART review) ? If so, I'll repost > that new revision. > > Thanks. > > JP. > -------------------------------------------------------------------------- ------ > > On Feb 2, 2006, at 11:22 AM, JP Vasseur wrote: > > > Hi Harald, > > > > Many Thanks for the comments ... and sorry for responding so > > late ... See in line, > > > > On Nov 3, 2005, at 11:34 AM, Harald Tveit Alvestrand wrote: > > > >> <I'm assuming people know about gen-art by now. If not - ask.> > >> > >> Document: draft-ietf-ccamp-loose-path-reopt-01 > >> Reviewer: Harald Alvestrand > >> Date: November 3, 2005 > >> > >> Summary: Technology ready for Proposed, presentation could use work > >> > >> Once I understood what this document's driving at, it's a > >> refreshingly simple idea: Let intermediate nodes tell the headend > >> node that a better (G)MPLS path might be available, and let the > >> headend node have a way to get the intermediate nodes to look for > >> one. > >> > >> I think this is well captured in the last half of the abstract: > >> > >> This document proposes a > >> mechanism that allows a TE LSP head-end LSR to trigger a new > >> path re- > >> evaluation on every hop having a next hop defined as a loose or > >> abstract hop and a mid-point LSR to signal to the head-end LSR > >> that a > >> better path exists (compared to the current path in use) or that > >> the > >> TE LSP must be reoptimized because of some maintenance required on > >> the TE LSP path. > >> > >> and that the abstract would actually be better if the first part > >> was moved to the introduction (which in fact says much the same > >> thing, but with more words). > >> > > > > Indeed, I moved most of the first half of the abstract to the > > Introduction section. > > > >> Minor technical: > >> > >> The document does not say explicitly that it's only applicable to > >> LSPs set up and maintained via RSVP-TE. (Is it RSVP-TE, or just > >> RSVP?) > > > > OK, I clarified. > > > >> > >> More editorials: > >> > >> - The "notice" that is section 1 seems oddly placed. It might be > >> better placed in section 7, where the requirements for section 1 > >> being true are stated. > > > > Yes, good point. I moved the text that used to be in the notice > > section to the section renamed "Applicability and Interoperability" > > > >> > >> - The document needs a terminology section, or absent that, a > >> consistent policy of acronym expansion on first use; ERO is used 6 > >> times before it's expanded in the last sentence of the first para > >> of section 1. Other acronyms not expanded are TE, LSP, RSVP, LSR, > >> IGP - these are commonly used across many (G)MPLS documents, so > >> it's possible that you could point to a terminology section from > >> another RFC and be done. > > > > Yes, thanks, a Terminology section has been added. > > > >> > >> - I believe section 3 is purely tutorial and contains no new > >> protocol. It would be good if it clearly said so, and said which > >> document contained the normative description of the procedure. > >> > > > > Indeed, I added some text to clarify. > > > >> - Section 4 repeats the description of the mechanism given in the > >> introduction. I believe this is superfluous. > >> > > > > You're right but unless strong objection I would prefer to keep > > that section since it incorporates an example that has been > > introduced in section 3. > > > >> - The order in which the two mechanisms are introduced in section > >> 2 and 4 was confusing to me at first read. I think it would flow > >> better if the midpoint to headend signalling was mentioned first, > >> and the headend to midpoint mechanism was defined afterwards, > >> saying something like > >> > >> - A head-end LSR to trigger on every LSR whose next hop is a > >> loose hop or an abstract node the re-evaluation of the current > >> path in order to detect a potential more optimal path, > >> which may > >> result in the mid-point LSR using the mechanism above to > >> signal > >> the existence of such a more optimal path > >> > >> (Note: The English of the paragraph reads oddly, given that the > >> bullets do not form complete sentences without the introductory > >> text; it's possible to do this better, I think.) > >> > > > > I kept the same order (because the first mechanism is likely to be > > the one more commonly used - that said, they're not exclusive) but > > I reworded a bit since indeed clarify could be improved. Thanks. > > > >> - The description in section 6.3 also obscures the linkage between > >> the two functions by calling them "modes"; I'd prefer "functions" > >> - because use of the "head-end requesting function" makes the > >> midpoint invoke the "mid-point explicit notification function". > >> > > > > Indeed, thanks (fixed). > > > >> - The enumeration of reasons to perform a reoptimization query > >> (timer, knowledge of link state change, operator command) seems > >> like either too much or too little; there seems to be no protocol > >> impact whatsoever of these mechanisms, and there could concievably > >> be other reasons for sending the queries.... I suggest inserting > >> some "for example" statements around them, so that it's clear that > >> the nodes are free to exercise these procedures whenever they feel > >> like it. > >> > > > > Good point indeed, thanks. > > > >> - There's a minor inconsistency between section 8, where a head- > >> end may (lowercase) decide to ignore notifications from another > >> domain, and section 6.3.2, where it MUST perform a reoptimization > >> on receiving sub-codes 7 and 8. I suggest changing the MUST to a > >> SHOULD; this also avoids the silly state of having gotten a > >> notification for a path it's decided to tear down...... > >> > > > > Also a good point ... Thanks, this has been changed to a SHOULD. > > > >> - Some speling erors were noted, but I didn't have time to write > >> them down. > >> > > > > I made another pass and hopefully caught them. > > > >> Nice document! > >> > >> > > > > Sorry for the proprietary format ... but if you want to have a look > > at the changes and let me know whether they fully address your points: > > > > <draft-ietf-ccamp-loose-path-reopt-02.doc> > > > > Thanks, > > > > JP. > > > >> > >> > >> > >> > >> > >> > >> > > > > _______________________________________________ Gen-art mailing list Gen-art@ietf.org https://www1.ietf.org/mailman/listinfo/gen-art
- [Gen-art] GEN-ART review of draft-ietf-ccamp-loos… Harald Tveit Alvestrand
- [Gen-art] Re: GEN-ART review of draft-ietf-ccamp-… Adrian Farrel
- [Gen-art] Re: GEN-ART review of draft-ietf-ccamp-… Bill Fenner
- [Gen-art] Re: GEN-ART review of draft-ietf-ccamp-… Harald Tveit Alvestrand
- [Gen-art] Re: GEN-ART review of draft-ietf-ccamp-… Adrian Farrel
- [Gen-art] Re: GEN-ART review of draft-ietf-ccamp-… JP Vasseur
- [Gen-art] Re: GEN-ART review of draft-ietf-ccamp-… JP Vasseur
- [Gen-art] Re: GEN-ART review of draft-ietf-ccamp-… Harald Tveit Alvestrand
- [Gen-art] Re: GEN-ART review of draft-ietf-ccamp-… JP Vasseur
- [Gen-art] Re: GEN-ART review of draft-ietf-ccamp-… Harald Tveit Alvestrand
- [Gen-art] Re: GEN-ART review of draft-ietf-ccamp-… JP Vasseur
- [Gen-art] Re: GEN-ART review of draft-ietf-ccamp-… Adrian Farrel
- [Gen-art] Re: GEN-ART review of draft-ietf-ccamp-… Alex Zinin