[Sidrops] Opsdir last call review of draft-ietf-sidrops-8210bis-06

Bo Wu via Datatracker <noreply@ietf.org> Mon, 02 May 2022 05:14 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: sidrops@ietf.org
Delivered-To: sidrops@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id B794CC157B5C; Sun, 1 May 2022 22:14:33 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Bo Wu via Datatracker <noreply@ietf.org>
To: ops-dir@ietf.org
Cc: draft-ietf-sidrops-8210bis.all@ietf.org, last-call@ietf.org, sidrops@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 8.1.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <165146847373.49076.4463961222416701917@ietfa.amsl.com>
Reply-To: Bo Wu <lana.wubo@huawei.com>
Date: Sun, 01 May 2022 22:14:33 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/sidrops/G4LU61YH426mk4PCMdt7E3uEmI0>
Subject: [Sidrops] Opsdir last call review of draft-ietf-sidrops-8210bis-06
X-BeenThere: sidrops@ietf.org
X-Mailman-Version: 2.1.34
List-Id: A list for the SIDR Operations WG <sidrops.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/sidrops>, <mailto:sidrops-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/sidrops/>
List-Post: <mailto:sidrops@ietf.org>
List-Help: <mailto:sidrops-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/sidrops>, <mailto:sidrops-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 02 May 2022 05:14:33 -0000

Reviewer: Bo Wu
Review result: Has Nits

Hi all,

 I have been selected as the Operational Directorate reviewer for this
 draft.

Review summary:
I believe this document is "Ready with nits". And it would be better
readability if the draft had added the reason for "obsolete" rather than
"update".

comments below:

This draft obsoletes RFC8210 and section 1.2 list the following changes:
1, new sections 5.12 ASPA PDU type
2, section 11 two ROA PDU race conditions
3, The protocol version number incremented  from 1 (one) to 2 (two)
4, the Section 7 has been updated accordingly

suggestions:
- Adds the reason why choosing "obsolete" RFC 8210 instead of "update", since
the protocol version has been updated from 1 to 2 - The title "Protocol Version
Negotiation" to be added to section 7 description. -  Some major changes to
other sections to be added:
   5.1 Flags adds ASPA PDU specific processing
   5.8. End of Data adds clarification text on version 1 changes
   14. IANA Considerations