Re: Gen-ART LC review ofdraft-ietf-mpls-return-path-specified-lsp-ping-12

t.p. <daedulus@btconnect.com> Thu, 29 August 2013 15:34 UTC

Return-Path: <daedulus@btconnect.com>
X-Original-To: ietf@ietfa.amsl.com
Delivered-To: ietf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id EBCAF21E805A; Thu, 29 Aug 2013 08:34:32 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.073
X-Spam-Level:
X-Spam-Status: No, score=-3.073 tagged_above=-999 required=5 tests=[AWL=-0.474, BAYES_00=-2.599]
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 rll+h33EFdzL; Thu, 29 Aug 2013 08:34:22 -0700 (PDT)
Received: from db9outboundpool.messaging.microsoft.com (mail-db9lp0250.outbound.messaging.microsoft.com [213.199.154.250]) by ietfa.amsl.com (Postfix) with ESMTP id 2637D21F9CD1; Thu, 29 Aug 2013 08:34:20 -0700 (PDT)
Received: from mail162-db9-R.bigfish.com (10.174.16.250) by DB9EHSOBE029.bigfish.com (10.174.14.92) with Microsoft SMTP Server id 14.1.225.22; Thu, 29 Aug 2013 15:34:19 +0000
Received: from mail162-db9 (localhost [127.0.0.1]) by mail162-db9-R.bigfish.com (Postfix) with ESMTP id 3069B4200C4; Thu, 29 Aug 2013 15:34:19 +0000 (UTC)
X-Forefront-Antispam-Report: CIP:157.56.254.181; KIP:(null); UIP:(null); IPV:NLI; H:DBXPRD0711HT001.eurprd07.prod.outlook.com; RD:none; EFVD:NLI
X-SpamScore: -17
X-BigFish: PS-17(zz9371I936eI542I1432I168aJzz1f42h208ch1ee6h1de0h1fdah2073h1202h1e76h1d1ah1d2ah1fc6hzz1de098h1033IL17326ah186068h8275bh8275dh1de097hz2dh2a8h5a9h839h947hd24hf0ah1177h1179h1288h12a5h12a9h12bdh137ah139eh13b6h1441h1504h1537h162dh1631h1758h17f1h184fh1898h18e1h1946h19b5h19ceh1ad9h1b0ah1d0ch1d2eh1d3fh1dfeh1dffh1e1dh1e23h304l1d11m1155h)
Received: from mail162-db9 (localhost.localdomain [127.0.0.1]) by mail162-db9 (MessageSwitch) id 1377790456452990_6643; Thu, 29 Aug 2013 15:34:16 +0000 (UTC)
Received: from DB9EHSMHS022.bigfish.com (unknown [10.174.16.239]) by mail162-db9.bigfish.com (Postfix) with ESMTP id 4326A1C017A; Thu, 29 Aug 2013 15:34:16 +0000 (UTC)
Received: from DBXPRD0711HT001.eurprd07.prod.outlook.com (157.56.254.181) by DB9EHSMHS022.bigfish.com (10.174.14.32) with Microsoft SMTP Server (TLS) id 14.16.227.3; Thu, 29 Aug 2013 15:34:13 +0000
Received: from DBXPRD0611HT004.eurprd06.prod.outlook.com (157.56.254.85) by pod51017.outlook.com (10.255.178.34) with Microsoft SMTP Server (TLS) id 14.16.353.4; Thu, 29 Aug 2013 15:34:11 +0000
Message-ID: <012001cea4cd$1845c680$4001a8c0@gateway.2wire.net>
From: "t.p." <daedulus@btconnect.com>
To: Roni Even <ron.even.tlv@gmail.com>, draft-ietf-mpls-return-path-specified-lsp-ping.all@tools.ietf.org
References: <F73A3CB31E8BE34FA1BBE3C8F0CB2AE255C28238@szxeml558-mbs.china.huawei.com><F73A3CB31E8BE34FA1BBE3C8F0CB2AE255C284D5@szxeml558-mbs.china.huawei.com><04cf01cea499$22eb4a80$68c1df80$@gmail.com><F73A3CB31E8BE34FA1BBE3C8F0CB2AE255C285CA@szxeml558-mbs.china.huawei.com> <04d701cea4a1$f8fdde00$eaf99a00$@gmail.com>
Subject: Re: Gen-ART LC review ofdraft-ietf-mpls-return-path-specified-lsp-ping-12
Date: Thu, 29 Aug 2013 16:33:04 +0100
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.1106
X-MimeOLE: Produced By Microsoft MimeOLE V6.00.2800.1106
X-Originating-IP: [157.56.254.85]
X-OriginatorOrg: btconnect.com
X-FOPE-CONNECTOR: Id%0$Dn%*$RO%0$TLS%0$FQDN%$TlsDn%
Cc: gen-art@ietf.org, ietf@ietf.org
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ietf>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 29 Aug 2013 15:34:38 -0000

Roni

I find it difficult to know whether to agree with you or not since there
is another "gorilla" operating in this namespace, namely
draft-andersson-mpls-lsp-ping-upd-00
which sort of gives the option of taking out the sub-TLV types of a TLV
Type into a separate registry which can then be referenced by the
specifications of several TLV Types.  This I-D formally calls for no
action by IANA, except that it implies a reorganisation of the
registries which IANA might well already have executed - certainly, the
registries no longer look the same as they did a few months ago although
that might also be a coincidence.

You make reference to the registry set up by RFC6426 but you should be
aware that what is now in the IANA registries is not what was there when
that RFC was approved; rather, it has changed as alluded to above..

If
draft-andersson-mpls-lsp-ping-upd-00
is approved, and my perception is that IANA assumes it has been, then
this, return-path I-D would appear to be barking up the wrong tree and
needs revising to reflect the new IANA structure, namely that it needs
to update
the title which already exists
Sub-TLVs for TLV Types 1 and 16
to become
Sub-TLVs for TLV Types 1 and 16 and 21
whereupon things start to fall into place

draft-andersson-mpls-lsp-ping-upd-00
also updates RFC4379 which, if accepted, means, I think, that this I-D
need not.

All very technically sound but we seem to have our processes upside
down.

There is of course a lengthy history to this that is not material to the
current discussion.

Tom Petch

----- Original Message -----
From: "Roni Even" <ron.even.tlv@gmail.com>
To: "'Mach Chen'" <mach.chen@huawei.com>;
<draft-ietf-mpls-return-path-specified-lsp-ping.all@tools.ietf.org>
Cc: <gen-art@ietf.org>; <ietf@ietf.org>
Sent: Thursday, August 29, 2013 11:24 AM
Subject: RE: Gen-ART LC review
ofdraft-ietf-mpls-return-path-specified-lsp-ping-12


> Hi,
> This text is OK if one wants to implement this draft.
> My concern was about the consistency of the IANA registration so that
if
> someone defines a new TLV type 1 based on RFC4379, IANA will know that
it
> must update also the registry for TLV type 21. If you see no such
problem, I
> have no concerns
> Roni
>
> > -----Original Message-----
> > From: Mach Chen [mailto:mach.chen@huawei.com]
> > Sent: 29 August, 2013 1:05 PM
> > To: Roni Even; draft-ietf-mpls-return-path-specified-lsp-
> > ping.all@tools.ietf.org
> > Cc: ietf@ietf.org; gen-art@ietf.org
> > Subject: RE: Gen-ART LC review of
> draft-ietf-mpls-return-path-specified-lsp-
> > ping-12
> >
> > Hi Roni,
> >
> > How about this:
> >
> > " No assignments of sub-TLVs in the range of 0-16383 and 32768-49161
> >     are made directly for TLV Type 21, sub-TLVs in these ranges are
> >     copied from the assignments made for TLV Type 1 and kept the
same
> >     as that for TLV Type 1. All sub-TLVs in these ranges (include
existing
> >     and future defined) defined for TLV Type 1 apply to TLV Type 21.
> >     Assignments of sub-..."
> >
> > Best regards,
> > Mach
> >
> > > -----Original Message-----
> > > From: Roni Even [mailto:ron.even.tlv@gmail.com]
> > > Sent: Thursday, August 29, 2013 5:21 PM
> > > To: Mach Chen;
> > > draft-ietf-mpls-return-path-specified-lsp-ping.all@tools.ietf.org
> > > Cc: ietf@ietf.org; gen-art@ietf.org
> > > Subject: RE: Gen-ART LC review of
> > > draft-ietf-mpls-return-path-specified-lsp-ping-12
> > >
> > > Hi,
> > > I am not sure you responded to my latest email.
> > >
> > > Having the policy for TLV type 1 here is not enough in my view
since I
> > > only look at RFC4379 and create a new TLV type I will not know
that I
> > > have to register it also for the type 21 if it will not be
mentioned
> > >
> > > As for the vendor specific see my other email Roni
> > >
> > > > -----Original Message-----
> > > > From: Mach Chen [mailto:mach.chen@huawei.com]
> > > > Sent: 29 August, 2013 11:33 AM
> > > > To: Roni Even; draft-ietf-mpls-return-path-specified-lsp-
> > > > ping.all@tools.ietf.org
> > > > Cc: ietf@ietf.org; gen-art@ietf.org
> > > > Subject: RE: Gen-ART LC review of
> > > draft-ietf-mpls-return-path-specified-lsp-
> > > > ping-12
> > > >
> > > > Hi Roni,
> > > >
> > > > Thanks for your detailed review and comments!
> > > >
> > > > Please see my reply inline...
> > > >
> > > > > From: Roni Even [mailto:ron.even.tlv@gmail.com]
> > > > > Sent: Wednesday, August 28, 2013 9:06 PM
> > > > > To:
> > > > >
draft-ietf-mpls-return-path-specified-lsp-ping.all@tools.ietf.org
> > > > > Cc: ietf@ietf.org; gen-art@ietf.org
> > > > > Subject: Gen-ART LC review of
> > > > > draft-ietf-mpls-return-path-specified-lsp-ping-12
> > > > >
> > > > > I am the assigned Gen-ART reviewer for this draft. For
background
> > > > > on Gen-ART, please see the FAQ at
> > > > > <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.
> > > > > Please resolve these comments along with any other Last Call
> > > > > comments you may receive.
> > > > > Document: draft-ietf-mpls-return-path-specified-lsp-ping-12
> > > > > Reviewer: Roni Even
> > > > > Review Date:2013-8-28
> > > > > IETF LC End Date: 2013-9-4
> > > > > IESG Telechat date:
> > > > >
> > > > > Summary: This draft is almost ready for publication as a
standard
> > > > > track
> > > RFC.
> > > > >
> > > > >
> > > > > Major issues:
> > > > > Minor issues:
> > > > > I am not clear on the sub-TLV in section 6.2 1. If a new
sub-TLV
> > > > > is defined for TLV type 1 do they need also to be added to TLV
type
> 21.
> > > > > This should be clear, and if there is some relation I think it
> > > > > should be reflected in the IANA registry for TLV type 1
> > > >
> > > > Yes, type 21 TLV intends to reuse existing and future defined
> > > > sub-TLVs for type TLV 1. And in Section 3.3, it has already
stated
> this, it
> > says:
> > > >
> > > > "The Target FEC sub-TLVs defined in [RFC4379] provide a good way
to
> > > >    identify a specific return path.  The Reply Path TLV can
carry any
> > > >    sub-TLV defined for use in the Target FEC Stack TLV that can
be
> > > >    registered."
> > > >
> > > > So, for Section 6.2, to make it cleaner and more explicit, how
about
> > > > this
> > > > change:
> > > >
> > > > Old:
> > > >
> > > > " No assignments of sub-TLVs in the range of 0-16383 and
32768-49161
> > > >    are made directly for TLV Type 21, sub-TLVs in these ranges
are
> > > >    copied from the assignments made for TLV Type 1. Assignments
of
> > > sub-..."
> > > >
> > > > New:
> > > >
> > > > " No assignments of sub-TLVs in the range of 0-16383 and
32768-49161
> > > >    are made directly for TLV Type 21, sub-TLVs in these ranges
are
> > > >    copied from the assignments (including existing and future
> allocations)
> > > >    made for TLV Type 1. Assignments of sub-..."
> > > >
> > > >
> > > > > 2. For the vendor or private use why a difference policy than
the
> > > > > rest of the sub-TLV registry
> > > >
> > > > This document does not make any changes to the "Vendor and
Private
> > use"
> > > > definition, range and policy as defined in RFC4379. In RFC4379,
it's
> > > policy is
> > > > defined different from other ranges.
> > > >
> > > > >
> > > > > Nits/editorial comments:
> > > > > 1. In section 3.4 I assume that "TC" is traffic class. It will
be
> > > > > good to expand and have reference.
> > > >
> > > > OK, will fix it when all last call comments received.
> > > >
> > > > Best regards,
> > > > Mach
>
>