[Idr] Secdir last call review of draft-ietf-idr-bgp-sendholdtimer-13

Carl Wallace via Datatracker <noreply@ietf.org> Sun, 28 July 2024 15:37 UTC

MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Carl Wallace via Datatracker <noreply@ietf.org>
To: secdir@ietf.org
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <172218105300.1558539.17525621904498066822@dt-datatracker-659f84ff76-9wqgv>
Date: Sun, 28 Jul 2024 08:37:33 -0700
Message-ID-Hash: 3MBRSLV63SNY77CP53NN26WCC3SOVMLF
CC: draft-ietf-idr-bgp-sendholdtimer.all@ietf.org, idr@ietf.org, last-call@ietf.org
Reply-To: Carl Wallace <carl@redhoundsoftware.com>
Subject: [Idr] Secdir last call review of draft-ietf-idr-bgp-sendholdtimer-13
Archived-At: <https://mailarchive.ietf.org/arch/msg/idr/9LcIOtwYjC1Gtz7t-WRjKSdsiQU>

Reviewer: Carl Wallace
Review result: Ready

This draft defines the SendHoldtimer and SendHoldTimer_Expires for the BGP
state machine. The draft is straightforward and well written but I had one
minor nit. In section 3.3, there is a NEW block to be added that begins "If the
SendHoldTimer_Expires (Event 29), the local system:". This seems to be
incomplete. From similar text in 4271, one fix would be to put the word
"occurs" before the comma.

[Idr] Secdir last call review of draft-ietf-idr-b… Carl Wallace via Datatracker
[Idr] Re: Secdir last call review of draft-ietf-i… Job Snijders