Re: [mpls] MPLS-RT review of draft-mirsky-mpls-bfd-directed-03

David Allan I <david.i.allan@ericsson.com> Tue, 04 August 2015 17:27 UTC

Return-Path: <david.i.allan@ericsson.com>
X-Original-To: mpls@ietfa.amsl.com
Delivered-To: mpls@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 2E0CF1A88AE; Tue, 4 Aug 2015 10:27:33 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.2
X-Spam-Level:
X-Spam-Status: No, score=-4.2 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001] autolearn=ham
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id x2DIGh24h87T; Tue, 4 Aug 2015 10:27:31 -0700 (PDT)
Received: from usevmg20.ericsson.net (usevmg20.ericsson.net [198.24.6.45]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 32ED41A8874; Tue, 4 Aug 2015 10:27:31 -0700 (PDT)
X-AuditID: c618062d-f799e6d00000329e-2d-55c0996cc01f
Received: from EUSAAHC001.ericsson.se (Unknown_Domain [147.117.188.75]) by usevmg20.ericsson.net (Symantec Mail Security) with SMTP id 39.60.12958.C6990C55; Tue, 4 Aug 2015 12:52:28 +0200 (CEST)
Received: from EUSAAMB105.ericsson.se ([147.117.188.122]) by EUSAAHC001.ericsson.se ([147.117.188.75]) with mapi id 14.03.0210.002; Tue, 4 Aug 2015 13:27:29 -0400
From: David Allan I <david.i.allan@ericsson.com>
To: Ross Callon <rcallon@juniper.net>, Gregory Mirsky <gregory.mirsky@ericsson.com>
Thread-Topic: MPLS-RT review of draft-mirsky-mpls-bfd-directed-03
Thread-Index: AdC+PcT+J6C8Nh5rR/SbGsOgRmpbuAHpvzxAAjrXv5AAAnqCkA==
Date: Tue, 4 Aug 2015 17:27:29 +0000
Message-ID: <E6C17D2345AC7A45B7D054D407AA205C4AF39952@eusaamb105.ericsson.se>
References: <AAE428925197FE46A5F94ED6643478FEB1120B3F41@HE111644.EMEA1.CDS.T-INTERNAL.COM> <7347100B5761DC41A166AC17F22DF11221883589@eusaamb103.ericsson.se> <BY1PR0501MB1430E416B0B3AC3CE7AE2E62A5760@BY1PR0501MB1430.namprd05.prod.outlook.com>
In-Reply-To: <BY1PR0501MB1430E416B0B3AC3CE7AE2E62A5760@BY1PR0501MB1430.namprd05.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [147.117.188.11]
Content-Type: multipart/alternative; boundary="_000_E6C17D2345AC7A45B7D054D407AA205C4AF39952eusaamb105erics_"
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFupikeLIzCtJLcpLzFFi42KZXLrHWzdn5oFQg+8HdSweLu5ht1h3+RSb xa2lK1kt/q64wuLA4rFkyU8mj+tNV9k9vlz+zBbAHMVlk5Kak1mWWqRvl8CVcX3BOfaCJ64V N478ZWlgnGfXxcjJISFgIrF52jVGCFtM4sK99WxdjFwcQgJHGSU+dD2GcpYxSkze/Q2sik3A QGLP/y9gtohAqMTul82MIEXMAjsZJVZ+6GIGSQgLOEgcWdfCBFHkKPHl1C42CNtJ4s+GJrAa FgEViUn/usDivAK+Ej8+Xoba9pVRYnXTZXaQBKdAosT3Kw1gRYxA930/tQZsKLOAuMStJ/OZ IO4WkFiy5zwzhC0q8fLxP1YIW0ni4+/57BD1+RJb1t9ihVgmKHFy5hOWCYyis5CMmoWkbBaS Moi4jsSC3Z/YIGxtiWULXzPD2GcOPGZCFl/AyL6KkaO0OLUsN93IYBMjMPKOSbDp7mDc89Ly EKMAB6MSD6/Cmv2hQqyJZcWVuYcYpTlYlMR5HaPyQoUE0hNLUrNTUwtSi+KLSnNSiw8xMnFw SjUw8v8WP7Rikx3336aoPacmH99zdQN3+3+N2BPbjy1ZxpO79ueT6GuVW3a2h3p9bvnPrx5i m+B0T1TV+UqU0gzHyT9P70vrXVSnodi03HjajY5PO86aluhqKF05UTP53+bzZkUfl3LFFSzm cn0ieercj7RnOzOSFHvrnz5hK3u26drSwvnFWkc+WimxFGckGmoxFxUnAgBQsO0OnQIAAA==
Archived-At: <http://mailarchive.ietf.org/arch/msg/mpls/_IA0ehbgT2YTpaqAiOcotXMdT4I>
Cc: "mpls@ietf.org" <mpls@ietf.org>, "mpls-chairs@ietf.org" <mpls-chairs@ietf.org>, "draft-mirsky-mpls-bfd-directed@tools.ietf.org" <draft-mirsky-mpls-bfd-directed@tools.ietf.org>
Subject: Re: [mpls] MPLS-RT review of draft-mirsky-mpls-bfd-directed-03
X-BeenThere: mpls@ietf.org
X-Mailman-Version: 2.1.15
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: <https://mailarchive.ietf.org/arch/browse/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: Tue, 04 Aug 2015 17:27:33 -0000

Hi

I've read the document and believe it to be close to complete.

My only concerns are


1)      Given a configuration message can support configuring multiple tests at a time, what are the actual consequences of one or more errors. Are those requests only invalidated or is are all the sub-TLVs in a request invalidated. IMO this should be clarified. Given that there are classes of error that should invalidate the entire request, and those that do not necessarily do so, it might be easiest to make a blanket statement that any error invalidates the entire message.

2)      The IANA considerations section is missing sub-TLV types 300 and 400.

If these items were addressed, I would be supportive of the document progressing.

Cheers
D