Re: [Idr] Fwd: I-D Action: draft-sas-idr-maxprefix-outbound-02.txt

[Ben Maddison <[e-mail]>]

Hi Melchior,

On 03/16, Melchior Aelmans wrote:
> Hi IDR,
> 
> Some time ago we updated the maxprefix outbound draft after some discussion
> on this list.
> Can you please review and comment?
> 
I have given the document a reasonably careful read, and have some
comments: some minor, others less-so.

Line numbers refer to the XML source on datatracker, which hopefully
correspond to your working version.

Section 2:
    - misspelled "discarded" (line 120)
    - "Sent" should be "Sends" (line 123)
    - "uses option (b)" should be "uses option (C)" (line 130)

Section 3:
    - The phrase "... or announces more prefixes than through local
      configuration allowed to." doesn't parse for me. (line 155)
      Perhaps:
        """
        Possible reasons for an AutomaticStop event include:
        1.  A BGP speaker receives an UPDATE message with a number of
            prefixes from a given peer such that the total prefixes
            received exceeds the maximum number of (inbound) prefixes
            configured (either "Pre-Policy" or "Post-Policy"); or
        2.  The Phase 3 Routing Decision function (RFC4271 Section 9.1.3)
            results in the total number of prefixes in the Adj-RIB-Out
            for that session exceeding the maximum number of (outbound)
            prefixes configured.
        In each case, the local system automatically disconnects the
        peer.
        """

Section 4:
    - In the inserted section 9.5.1 language, the requirement that "the
      session MUST be terminated" (line 179) seems to contradict the
      possible options enumerated in section 2.

Section 5.1:
    - "measurements" should be "measures" (line 238)

Hope this is helpful.

Cheers,

Ben