[bess] Genart early review of draft-ietf-bess-evpn-unequal-lb-18

Mallory Knodel via Datatracker <noreply@ietf.org> Thu, 16 November 2023 18:13 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: bess@ietf.org
Delivered-To: bess@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id EE803C151547; Thu, 16 Nov 2023 10:13:29 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Mallory Knodel via Datatracker <noreply@ietf.org>
To: gen-art@ietf.org
Cc: bess@ietf.org, draft-ietf-bess-evpn-unequal-lb.all@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 11.14.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <170015840996.54421.1055029335179226613@ietfa.amsl.com>
Reply-To: Mallory Knodel <mknodel@cdt.org>
Date: Thu, 16 Nov 2023 10:13:29 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/bess/npd8M_eZ3DiTpdVgEcbPsH2__H8>
Subject: [bess] Genart early review of draft-ietf-bess-evpn-unequal-lb-18
X-BeenThere: bess@ietf.org
X-Mailman-Version: 2.1.39
List-Id: BGP-Enabled ServiceS working group discussion list <bess.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/bess>, <mailto:bess-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/bess/>
List-Post: <mailto:bess@ietf.org>
List-Help: <mailto:bess-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/bess>, <mailto:bess-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 16 Nov 2023 18:13:30 -0000

Reviewer: Mallory Knodel
Review result: Ready with Issues

I am the assigned Gen-ART reviewer for this draft. Please resolve these
comments along with any other comments you may receive.

For more information, please see the FAQ at
<https://wiki.ietf.org/en/group/gen/GenArtFAQ>.

Document: draft-ietf-bess-evpn-unequal-lb
Reviewer: Mallory Knodel
Review Date: 16 Nov 2023

Summary: This draft is basically ready for publication, but has nits that
should be fixed before publication.

Major issues: Section 3 as the Solution Overview seems out of step with the
remaining sections in that it properly describes the relationships between 4, 5
and 6, but it appears that 7-10 are additional over arching considerations that
might benefit from being extracted from the discussion of direct solutions.
Suggesting perhaps that 4, 5 and 6 be treated under the solution space, whereas
the remaining substantive sections 7-10 be presented as additional
considerations and tradeoffs but not direct descriptions of full solutions to
the problems outlined in the introduction.

Minor issues: Not all acronyms are properly expanded in order of their
first-time use which hinders readability. Seems 12. Operational Considerations
is superfluous and plenty of document dependencies also do not have this
section.

Nits/editorial comments: The focus of my review did not expose any
nits/editorial comments though I believe there are some that have persisted
across the various versions that I compared and I would encourage the authors
to do a full copy edit ahead of IESG submission.