Re: [mpls] [PWE3] Working Group Last Call on draft-ietf-mpls-retire-ach-tlv

"Andrew G. Malis" <amalis@gmail.com> Fri, 28 June 2013 12:36 UTC

Return-Path: <amalis@gmail.com>
X-Original-To: mpls@ietfa.amsl.com
Delivered-To: mpls@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 8B8D921F9D5B; Fri, 28 Jun 2013 05:36:44 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.599
X-Spam-Level:
X-Spam-Status: No, score=-102.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, HTML_MESSAGE=0.001, NO_RELAYS=-0.001, 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 BaWnEWNaYqdl; Fri, 28 Jun 2013 05:36:38 -0700 (PDT)
Received: from mail-ob0-x233.google.com (mail-ob0-x233.google.com [IPv6:2607:f8b0:4003:c01::233]) by ietfa.amsl.com (Postfix) with ESMTP id 4ED3221F9F08; Fri, 28 Jun 2013 05:34:48 -0700 (PDT)
Received: by mail-ob0-f179.google.com with SMTP id xk17so1896017obc.10 for <multiple recipients>; Fri, 28 Jun 2013 05:34:47 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc:content-type; bh=vwedOUcpIJdS1x5KkWVyaADfzQIweQuLgWLsrI9EV+E=; b=CkBwN6kPRItgJ9mBO+0XiCYwDsgBzvqCKSb67gMfouTBGiXeRgB/d866wYWC3R9rTf dRysr+l3JBsBwJ0xnkWc20e33NJeQwJUDQN3ug4U+KrroCcqnp9KH4Weo6UUaSGQ3Vaz IQzfVuGrYwRrugHz9crQ5INOBPzxHI1XBDLLDDKrj8anWJK3MZ10qbagisVRZwjYwh56 iw96DwVUHNIY//ma9rJlRzGRyWWCbDZM1eHmjor6peKkJJeQBYqBBTzhiJ6Ofd/LqDPI k3l3aJbTXGNwA+JtRxdVAebi8YJ9LFKpgghO8LKHUqCpMZbrdFJUspnYacOgs3HS3XjW hGlA==
X-Received: by 10.182.27.102 with SMTP id s6mr6163772obg.42.1372422887795; Fri, 28 Jun 2013 05:34:47 -0700 (PDT)
MIME-Version: 1.0
Received: by 10.76.144.65 with HTTP; Fri, 28 Jun 2013 05:34:27 -0700 (PDT)
In-Reply-To: <001401ce73fa$8f45d620$add18260$@olddog.co.uk>
References: <51B9BE97.2090103@pi.nu> <51CD64A1.4010408@pi.nu> <001401ce73fa$8f45d620$add18260$@olddog.co.uk>
From: "Andrew G. Malis" <amalis@gmail.com>
Date: Fri, 28 Jun 2013 08:34:27 -0400
Message-ID: <CAK+d4xtosDGkosKqhxvbdVJkm6eTvuZDvxcmU49eYEyhsWozVw@mail.gmail.com>
To: Adrian Farrel <adrian@olddog.co.uk>
Content-Type: multipart/alternative; boundary="089e01184c8cb5773304e0361afe"
X-Mailman-Approved-At: Mon, 01 Jul 2013 05:25:09 -0700
Cc: "mpls@ietf.org" <mpls@ietf.org>, "mpls-chairs@tools.ietf.org" <mpls-chairs@tools.ietf.org>, "pwe3@ietf.org" <pwe3@ietf.org>, "pwe3-chairs@tools.ietf.org" <pwe3-chairs@tools.ietf.org>, Loa Andersson <loa@pi.nu>
Subject: Re: [mpls] [PWE3] Working Group Last Call on draft-ietf-mpls-retire-ach-tlv
X-BeenThere: mpls@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Multi-Protocol Label Switching WG <mpls.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mpls>, <mailto:mpls-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/mpls>
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>
X-List-Received-Date: Fri, 28 Jun 2013 12:36:44 -0000

Adrian,

Looks good to me. Just from an English language aspect (here I am lecturing
a Brit about English :-) I suggest either removing the final comma from
your text or alternatively adding a comma after
"ACH messages".

Cheers,
Andy


On Fri, Jun 28, 2013 at 8:25 AM, Adrian Farrel <adrian@olddog.co.uk> wrote:

> > However TLVs might be carried in the GACh messages, the disticnction
> > is not entirely clear in the draft, can the authors please make
> > this clear.
>
> Happily.
>
> The current version defines "ACH TLV" as "TLV constructs that can be
> carried in
> messages on the G-ACh by placing them in the ACH between the fixed header
> fields
> and the G-ACh message."
>
> Thus, retiring ACH TLVs, obviously means only removing this concept and
> not any
> other TLV-related concept.
>
> I would propose to change the last sentence of Section 1 as follows:
>
> OLD
>    This document states that ACH TLVs as specified in RFC 5586 are not
>    useful and might be harmful.  It updates RFC 5586 by deprecating the
>    ACH TLV and updating the associated IANA registries as described in
>    Section 4 of this document.
> NEW
>    This document states that ACH TLVs as specified in RFC 5586 are not
>    useful and might be harmful.  It updates RFC 5586 by deprecating the
>    ACH TLV and updating the associated IANA registries as described in
>    Section 4 of this document.  This document makes no comment about the
>    use of TLVs in other places.  In particular, proposals to use TLVs
>    within ACH messages, or as an appendage to ACH messages are not in
>    scope of this document.
> END
>
> Does that work?
>
> Thanks,
> Adrian
>
> _______________________________________________
> pwe3 mailing list
> pwe3@ietf.org
> https://www.ietf.org/mailman/listinfo/pwe3
>