Last Call Review of draft-ietf-lsr-isis-area-proxy-10
review-ietf-lsr-isis-area-proxy-10-genart-lc-yee-2023-12-26-00

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://wiki.ietf.org/en/group/gen/GenArtFAQ>.

Document: draft-ietf-lsr-isis-area-proxy-10
Reviewer: Peter Yee
Review Date: 2023-12-26
IETF LC End Date: 2023-12-22
IESG Telechat date: 2024-01-04

Summary: This is an experimental addition to IS-IS routing to allow the use of
Level 1 networks for transit of Level 2 network traffic without burdening the
Level 2 networks with full knowledge of the Level 1 network topology. At least
I think that's what this document covers. Routing is far outside of my
wheelhouse, so I can't really provide any routing-aware comments despite some
time spent glossing over some of the normative references. There are a few
truly minor nits in the document. [Ready with nits]

Major issues: None

Minor issues: None

Nits/editorial comments:

Page 4, second paragraph, second sentence: change "MPLS based" to "MPLS-based".

Page 4, section 1.1: in the text version that I read, the boilerplate contains
the link "(https://tools.ietf.org/html/bcp14)". While this is found as a
hyperlink in the PDF and HTML versions, I don't believe it should appear in the
text version. I don't know if this is a tool issue, but recent RFCs that I
downloaded did not contain it. The hyperlink appears to cause idnits to
complain about incorrect boilerplate text.

Page 6, 1st paragraph, 1st sentence: change "i.e.  Ethernets" to "i.e.,
Ethernets", which is to say, change the first space to a comma.

Page 9, last paragraph, last sentence: change "implementation-dependent" to
"implementation dependent".