[Gen-art] Gen-ART telechat review of draft-ietf-idr-ls-distribution-10

Alexey Melnikov <alexey.melnikov@isode.com> Sun, 10 May 2015 16:18 UTC

Return-Path: <alexey.melnikov@isode.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com []) by ietfa.amsl.com (Postfix) with ESMTP id 60A741A1DE2 for <gen-art@ietfa.amsl.com>; Sun, 10 May 2015 09:18:21 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 2.695
X-Spam-Level: **
X-Spam-Status: No, score=2.695 tagged_above=-999 required=5 tests=[BAYES_20=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FH_RELAY_NODNS=1.451, HELO_MISMATCH_COM=0.553, RDNS_NONE=0.793, SPF_PASS=-0.001] autolearn=no
Received: from mail.ietf.org ([]) by localhost (ietfa.amsl.com []) (amavisd-new, port 10024) with ESMTP id F6r8Rlv-g4Z0 for <gen-art@ietfa.amsl.com>; Sun, 10 May 2015 09:18:20 -0700 (PDT)
Received: from waldorf.isode.com (unknown []) by ietfa.amsl.com (Postfix) with ESMTP id 3685F1A1B8C for <gen-art@ietf.org>; Sun, 10 May 2015 09:18:20 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1431274699; d=isode.com; s=selector; i=@isode.com; bh=UbQrShePNaeGA0xRWgbxN7XxFYY2nkuoCcNCcZIvbnY=; h=From:Sender:Reply-To:Subject:Date:Message-ID:To:Cc:MIME-Version: In-Reply-To:References:Content-Type:Content-Transfer-Encoding: Content-ID:Content-Description; b=Zwkij3rQ7OjsqJDQq2n8MoBVF3dFL+kYmNIH4imOu12ka/ZIMbtsd4nJr+uc/WxmU3ZDie 67ic7OIkVM/wpjVtRDBtXausu8V05v3K2r6uPpdKKiXAZWFDs2VlmPM/9L2VCOC+6vpTSg hjumQfedIhGQByhd9qlUMHCEs96pZCo=;
Received: from [] (cpc5-nmal20-2-0-cust24.19-2.cable.virginm.net []) by waldorf.isode.com (submission channel) via TCP with ESMTPSA id <VU-ExwAZW67F@waldorf.isode.com>; Sun, 10 May 2015 17:18:18 +0100
X-SMTP-Protocol-Errors: PIPELINING
Message-ID: <554F84C4.2090303@isode.com>
Date: Sun, 10 May 2015 17:18:12 +0100
From: Alexey Melnikov <alexey.melnikov@isode.com>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/20130620 Thunderbird/17.0.7
To: draft-ietf-idr-ls-distribution.all@tools.ietf.org, General Area Review Team <gen-art@ietf.org>
MIME-Version: 1.0
Content-Type: text/plain; charset="UTF-8"; format="flowed"
Content-transfer-encoding: quoted-printable
Archived-At: <http://mailarchive.ietf.org/arch/msg/gen-art/5dCbdyM76fbrBvoTsEqWmS-dxFA>
Subject: [Gen-art] Gen-ART telechat review of draft-ietf-idr-ls-distribution-10
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sun, 10 May 2015 16:18:21 -0000


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 wait for direction from your document shepherd
or AD before posting a new version of the draft.

Document: draft-ietf-idr-ls-distribution-10.txt
Reviewer: Alexey Melnikov
Review Date: 2015-05-10
IETF LC End Date: 2015-04-08
IESG Telechat date: N/A

My apologies for the late review of this document.

Summary: Ready with nits

Minor (but some of these might be more serious):

In 6.2.2:

If an implementation of BGP-LS detects a malformed attribute, then it
    SHOULD use the ’Attribute Discard’ action as per
    [I-D.ietf-idr-error-handling] Section 2.

This needs to be a Normative reference. Or you can keep it as 
Informative, if you change the sentence not to use RFC 2119 language.

In - does this need a new IANA registry? (I am fine if you think 
you don't).

In - what is "subset of the FQDN"?


       The TE Default Metric TLV carries the TE-metric for this link.
       The length of this TLV is fixed at 4 octets.

I am probably showing my ignorance, but is the term "TE-metric" defined 
somewhere? The description below suggests it has substructure, which I 
don't know anything about.

        If a source protocol (e.g.
        IS-IS) does not support a Metric width of 32 bits then the high
        order octet MUST be set to zero.

Best Regards,