Skip to main content

Last Call Review of draft-ietf-grow-bmp-local-rib-10

Request Review of draft-ietf-grow-bmp-local-rib
Requested revision No specific revision (document currently at 13)
Type Last Call Review
Team Routing Area Directorate (rtgdir)
Deadline 2021-03-29
Requested 2021-03-15
Requested by Alvaro Retana
Authors Tim Evens , Serpil Bayraktar , Manish Bhardwaj , Paolo Lucente
I-D last updated 2021-03-31
Completed reviews Rtgdir Last Call review of -10 by Loa Andersson (diff)
Secdir Last Call review of -10 by Chris M. Lonvick (diff)
Genart Last Call review of -10 by Thomas Fossati (diff)
Assignment Reviewer Loa Andersson
State Completed
Request Last Call review on draft-ietf-grow-bmp-local-rib by Routing Area Directorate Assigned
Posted at
Reviewed revision 10 (document currently at 13)
Result Has issues
Completed 2021-03-31
Resending to RTG-DIR.

发件人: Loa Andersson []
发送时间: 2021年3月31日 14:05
收件人: Yemin (Amy) <>;
主题: 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 ​

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:
Senior MPLS Expert                
Bronze Dragon Consulting             phone: +46 739 81 21 64