[Gen-art] Genart last call review of draft-ietf-idr-bgp-open-policy-18
Gyan Mishra via Datatracker <noreply@ietf.org> Wed, 22 December 2021 01:01 UTC
Return-Path: <noreply@ietf.org>
X-Original-To: gen-art@ietf.org
Delivered-To: gen-art@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 286513A07AD; Tue, 21 Dec 2021 17:01:20 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Gyan Mishra via Datatracker <noreply@ietf.org>
To: gen-art@ietf.org
Cc: draft-ietf-idr-bgp-open-policy.all@ietf.org, idr@ietf.org, last-call@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 7.41.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <164013488006.27197.13873644386825552452@ietfa.amsl.com>
Reply-To: Gyan Mishra <hayabusagsm@gmail.com>
Date: Tue, 21 Dec 2021 17:01:20 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/bnQ6_d1J8TSN6zw-579JvK8bxkQ>
Subject: [Gen-art] Genart last call review of draft-ietf-idr-bgp-open-policy-18
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.29
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: <https://mailarchive.ietf.org/arch/browse/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: Wed, 22 Dec 2021 01:01:20 -0000
Reviewer: Gyan Mishra Review result: Ready with Nits I am the assigned Gen-ART reviewer for this draft. The General Area Review Team (Gen-ART) reviews all IETF documents being processed by the IESG for the IETF Chair. Please treat these comments just like any other last call comments. For more information, please see the FAQ at <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>. Document: draft-ietf-idr-bgp-open-policy-?? Reviewer: Gyan Mishra Review Date: 2021-12-21 IETF LC End Date: 2021-12-17 IESG Telechat date: Not scheduled for a telechat Summary: This draft provides a new BGP open role capability and OTC path attribute to detect and mitigate route leaks automatically. I have been following this draft on IDR and supported through Adoption and WGLC. This document has matured and is ready for publication. The new BGP role capabilities mismatch code 2 subcode 8 discussed on ML seems to have multiple implementations deployed and one confined by Cisco. I agree that the authors should request a new subcode for the role mismatch notification. Major issues: None Minor issues: None Nits/editorial comments: Comment related to Gao-Rexford model. The Gao-Rexford Model only has 3 peer types North bound upstream Provider, Southbound Customer and lateral same tier level peer. With the role capabilities, RS and RS-Client is added which makes it slightly different but almost identical. In describing the role types would it make sense to have a graphical depiction of Gao-Redford model with the role capabilities on adjacent peers to help explain the role relationship for local and remote-as. Just an idea to help explain the role capabilities. In the role correctness section scenario where the peer receives multiple role capabilities to send role mismatch notification. What if there is a timing issue and the multiple are received after the BGP open and peer is established possible sequence of events issue. Is it possible the peer may not get a mismatch notification if the peer establishes prior to getting a different capabilities where a mismatch or problem exists that is missed that could result in a route leak. I am thinking of possibly false positive or negative or negative during BGP open capabilities exchange
- [Gen-art] Genart last call review of draft-ietf-i… Gyan Mishra via Datatracker
- Re: [Gen-art] Genart last call review of draft-ie… Alvaro Retana
- Re: [Gen-art] Genart last call review of draft-ie… Gyan Mishra
- Re: [Gen-art] Genart last call review of draft-ie… Alvaro Retana
- Re: [Gen-art] Genart last call review of draft-ie… Sriram, Kotikalapudi (Fed)
- Re: [Gen-art] Genart last call review of draft-ie… Gyan Mishra
- Re: [Gen-art] Genart last call review of draft-ie… Gyan Mishra
- Re: [Gen-art] Genart last call review of draft-ie… Sriram, Kotikalapudi (Fed)
- Re: [Gen-art] [Idr] Genart last call review of dr… Brian Dickson
- Re: [Gen-art] Genart last call review of draft-ie… Alexander Azimov
- Re: [Gen-art] Genart last call review of draft-ie… Alvaro Retana
- Re: [Gen-art] [Idr] Genart last call review of dr… Gyan Mishra
- Re: [Gen-art] Genart last call review of draft-ie… Gyan Mishra
- Re: [Gen-art] Genart last call review of draft-ie… Alexander Azimov
- Re: [Gen-art] Genart last call review of draft-ie… Alvaro Retana
- Re: [Gen-art] Genart last call review of draft-ie… Sriram, Kotikalapudi (Fed)
- Re: [Gen-art] Genart last call review of draft-ie… Gyan Mishra
- Re: [Gen-art] Genart last call review of draft-ie… Lars Eggert