[RTG-DIR] FW: RtgDir review: draft-ietf-grow-bmp-local-rib

"Yemin (Amy)" <amy.yemin@huawei.com> Wed, 31 March 2021 08:00 UTC

Return-Path: <amy.yemin@huawei.com>
X-Original-To: rtg-dir@ietfa.amsl.com
Delivered-To: rtg-dir@ietfa.amsl.com
Received: from localhost (localhost []) by ietfa.amsl.com (Postfix) with ESMTP id A02283A1F04 for <rtg-dir@ietfa.amsl.com>; Wed, 31 Mar 2021 01:00:34 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.919
X-Spam-Status: No, score=-1.919 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([]) by localhost (ietfa.amsl.com []) (amavisd-new, port 10024) with ESMTP id rUByj_RNCNgk for <rtg-dir@ietfa.amsl.com>; Wed, 31 Mar 2021 01:00:30 -0700 (PDT)
Received: from frasgout.his.huawei.com (frasgout.his.huawei.com []) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 87AA93A1F07 for <rtg-dir@ietf.org>; Wed, 31 Mar 2021 01:00:29 -0700 (PDT)
Received: from fraeml738-chm.china.huawei.com (unknown []) by frasgout.his.huawei.com (SkyGuard) with ESMTP id 4F9JTz4kc1z684Q8 for <rtg-dir@ietf.org>; Wed, 31 Mar 2021 15:53:43 +0800 (CST)
Received: from dggema717-chm.china.huawei.com ( by fraeml738-chm.china.huawei.com ( with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256) id 15.1.2106.2; Wed, 31 Mar 2021 10:00:27 +0200
Received: from dggema769-chm.china.huawei.com ( by dggema717-chm.china.huawei.com ( with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2106.2; Wed, 31 Mar 2021 16:00:25 +0800
Received: from dggema769-chm.china.huawei.com ([]) by dggema769-chm.china.huawei.com ([]) with mapi id 15.01.2106.013; Wed, 31 Mar 2021 16:00:24 +0800
From: "Yemin (Amy)" <amy.yemin@huawei.com>
To: "rtg-dir@ietf.org" <rtg-dir@ietf.org>
Thread-Topic: RtgDir review: draft-ietf-grow-bmp-local-rib
Thread-Index: AdcmA9DElVSeZZgsVkGCenWwkiJBvw==
Date: Wed, 31 Mar 2021 08:00:24 +0000
Message-ID: <21f021480c27492ea822a72b3ce64125@huawei.com>
Accept-Language: zh-CN, en-US
Content-Language: zh-CN
X-MS-Has-Attach: yes
x-originating-ip: []
Content-Type: multipart/mixed; boundary="_002_21f021480c27492ea822a72b3ce64125huaweicom_"
MIME-Version: 1.0
X-CFilter-Loop: Reflected
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtg-dir/Hssfx1whCpjbGWCC8HULR7jYw7Q>
Subject: [RTG-DIR] FW: RtgDir review: draft-ietf-grow-bmp-local-rib
X-BeenThere: rtg-dir@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Routing Area Directorate <rtg-dir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/rtg-dir>, <mailto:rtg-dir-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/rtg-dir/>
List-Post: <mailto:rtg-dir@ietf.org>
List-Help: <mailto:rtg-dir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/rtg-dir>, <mailto:rtg-dir-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 31 Mar 2021 08:00:35 -0000

Resending to RTG-DIR. 

发件人: Loa Andersson [mailto:loa@pi.nu] 
发送时间: 2021年3月31日 14:05
收件人: Yemin (Amy) <amy.yemin@huawei.com>om>; rtg-ads@ietf.org
抄送: draft-ietf-grow-bmp-local-rib.all@ietf.org
主题: RtgDir review: draft-ietf-grow-bmp-local-rib


I have been selected as the Routing Directorate reviewer for this draft. 
The Routing Directorate seeks to review all routing or routing-related drafts as they pass through IETF last call and IESG review, and sometimes on special request. The purpose of the review is to provide assistance to the Routing ADs. For more information about the Routing Directorate, please see ​ http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir

Although these comments are primarily for the use of the Routing ADs, it would be helpful if you could consider them along with any other IETF Last Call comments that you receive, and strive to resolve them through discussion or by updating the draft.

Document: draft-ietf-grow-bmp-local-rib
Reviewer: Loa Andersson
Review Date: 2021-03-31
IETF LC End Date: 2021-04-08
Intended Status: Standards Track


This document is basically ready for publication, but has nits that should be considered prior to publication (see the included Word-file)

I have one minor concern about this document that I think should be resolved before publication.


The document is well written, but have a lot of nits that makes the reading slow, most of this is captured on the included Word-document.

Major Issues:
"No major issues found."

Minor Issues:

I have one minor issue (though I'm uncertain how to classify it).

This minor concern is in "4.2.  Peer Flags"

This document removes the "locally sourced routes" comment under the L flag description in section 4.2 of [RFC7854]. This a text change in an existing RFC, I think this need to be done more careful than saying that a comment is dropped.

I think this would do:

"The second sub-bullet under the second bullet in section 4.2 "Per-Peer
  Header" will now read:

*  The L flag, if set to 1, indicates that the message reflects the post-policy Adj-RIB-In (i.e., its path attributes reflect the application of inbound policy). It is set to 0 if the message reflects the pre-policy Adj-RIB-In. See Section 5 for further detail.  This flag has no significance when used with route mirroring messages (Section 4.7).

If locally sourced routes are communicated using BMP, they MUST be conveyed using the Loc-RIB instance peer type."

Note: I have not checked if we need to change anything in Section 5.

See the included word file.


Loa Andersson                        email: loa@pi.nu
Senior MPLS Expert                          loa.pi.nu@gmail.com
Bronze Dragon Consulting             phone: +46 739 81 21 64