Re: [mpls] wg last call on draft-ietf-mpls-p2mp-te-mib-06.txt - EXTENDED
"Adrian Farrel" <adrian@olddog.co.uk> Thu, 24 April 2008 18:35 UTC
Return-Path: <mpls-bounces@ietf.org>
X-Original-To: mpls-archive@megatron.ietf.org
Delivered-To: ietfarch-mpls-archive@core3.amsl.com
Received: from core3.amsl.com (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 561F73A6CFA; Thu, 24 Apr 2008 11:35:10 -0700 (PDT)
X-Original-To: mpls@core3.amsl.com
Delivered-To: mpls@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id B8B073A6C5B for <mpls@core3.amsl.com>; Thu, 24 Apr 2008 11:35:09 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.101
X-Spam-Level:
X-Spam-Status: No, score=-1.101 tagged_above=-999 required=5 tests=[AWL=-0.515, BAYES_00=-2.599, FAKE_REPLY_C=2.012, STOX_REPLY_TYPE=0.001]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id UatQjKh1f5WI for <mpls@core3.amsl.com>; Thu, 24 Apr 2008 11:35:08 -0700 (PDT)
Received: from asmtp1.iomartmail.com (asmtp1.iomartmail.com [62.128.201.248]) by core3.amsl.com (Postfix) with ESMTP id 7A10A3A6C5C for <mpls@ietf.org>; Thu, 24 Apr 2008 11:35:08 -0700 (PDT)
Received: from asmtp1.iomartmail.com (localhost.localdomain [127.0.0.1]) by asmtp1.iomartmail.com (8.12.11.20060308/8.12.8) with ESMTP id m3OIZBgr018460; Thu, 24 Apr 2008 19:35:11 +0100
Received: from your029b8cecfe (dsl-sp-81-140-15-32.in-addr.broadbandscope.com [81.140.15.32]) (authenticated bits=0) by asmtp1.iomartmail.com (8.12.11.20060308/8.12.11) with ESMTP id m3OIZ7q2018437; Thu, 24 Apr 2008 19:35:10 +0100
Message-ID: <04f201c8a639$eaad1ee0$0300a8c0@your029b8cecfe>
From: Adrian Farrel <adrian@olddog.co.uk>
To: benjamin.niven-jenkins@bt.com
Date: Thu, 24 Apr 2008 19:34:59 +0100
MIME-Version: 1.0
X-Priority: 3
X-MSMail-Priority: Normal
X-Mailer: Microsoft Outlook Express 6.00.2900.3138
X-MimeOLE: Produced By Microsoft MimeOLE V6.00.2900.3198
Cc: mpls@ietf.org
Subject: Re: [mpls] wg last call on draft-ietf-mpls-p2mp-te-mib-06.txt - EXTENDED
X-BeenThere: mpls@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
Reply-To: Adrian Farrel <adrian@olddog.co.uk>
List-Id: Multi-Protocol Label Switching WG <mpls.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/mpls>, <mailto:mpls-request@ietf.org?subject=unsubscribe>
List-Post: <mailto:mpls@ietf.org>
List-Help: <mailto:mpls-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mpls>, <mailto:mpls-request@ietf.org?subject=subscribe>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Sender: mpls-bounces@ietf.org
Errors-To: mpls-bounces@ietf.org
Hi Ben, Many thanks for reading. > Section 2 presents references differently to the rest of the document > section 2 uses the format RFCxxx [RFCxxx] but elsewhere the format used is > [RFCxxx]. Also STD 58 has no reference. > > Section 2: In the list of documents that describe SMIv2 why is STD58 > listed > three times? The precise words in section 2 are thrust upon us. They form boilerplate for MIB module I-Ds and we dare not whisper of changing this text. Oddly (?) the three listed RFCs combine to form STD 58. See http://www.rfc-editor.org/rfc-index.html > Section 4 refers to a P2MP path being made up of LSPs (plural) - I think > there is an implicit assumption by the authors that a P2MP LSP is modelled > as a set of sub-LSPs that when joined together at the branch LSRs form the > complete P2MP LSP tree. I would suggest that this assumption is stated at > the beginning of section 4 and that RFC4875 for a more detailed > description. > > I suggest the authors could use something like the following text "This > document models a P2MP LSP as comprising of multiple source-to-leaf (S2L) > sub-LSPs. These S2L sub-LSPs are set up between the ingress and egress > LSRs > and are appropriately combined by the branch LSRs to result in a P2MP TE > LSP. See [RFC4875] for a more detailed description of S2L LSPs and how > they > are combined to form a P2MP TE LSP" Good catch. I'll add something similar and rework the section a little for more clarity. > Section 5.2, 4th paragraph - I would suggest the authors make it explicit > whether LSR B sends a single or multiple Resv messages to LSR A. Yes. Actually, the same question applies to Path messages. I have added a reference to [RFC4875] and pointed out that on the upstream segment there could be one or two Path and Resv messages depending on implementation. > As the P2MP TE MIB can be used for both P2MP MPLS & GMPLS LSPs and as P2MP > LSPs are always unidirectional the document should probably specify that > when using the MIB to configure P2MP GMPLS LSPs that gmplsTunnelDirection > should always be set to forward(0). Yes. That is also a good catch as well. Added material to the discussion of GMPLs in section 4. > Purely editorial comments: > ToC: Two sections 4.3 & no 4.4. > > Section 4.2 3rd paragraph - I think you mean "supplementary to" not > "supplementary for" Or do you mean the text supplants/supercedes/replaces > that in RFC3812? > > Section 4.2.2 mplsTunnelMaxHops s/mximum/maximum/ s/legac/legacy/ > > Section 5.1 step 2 s/-- This table entry is created by configuration no > signaling/-- This table entry is created by configuration not signaling/ > > Section 5.2 4th paragraph, 1st line s/LSR/LSR B/ > > Section 5.2 the paragraph above the mplsTeP2mpTunelTable example > s/speicific/specific/ > > Section 5.2 the paragraph above mplsTunnelHopTable s/spearated/separated/ All excellent. > Section 8 paragraph 11 the "even then" is redundant and should be removed > leaving the following sentence: "Even if the network itself is secure (for > example by using IPSec), there is no control as to who on the secure > network > is allowed to access and GET/SET (read/change/create/delete) the objects > in > this MIB module." > > I would also suggest the authors consider s/there is no control/there is > still no control/ I've made those changes and we'll see what happens. I think these two are errors in the boilerplate :-( Many thanks, Adrian _______________________________________________ mpls mailing list mpls@ietf.org https://www.ietf.org/mailman/listinfo/mpls
- Re: [mpls] wg last call on draft-ietf-mpls-p2mp-t… Adrian Farrel