Last Call Review of draft-ietf-pce-pcep-stateful-pce-gmpls-21
review-ietf-pce-pcep-stateful-pce-gmpls-21-genart-lc-bryant-2023-06-02-00

Request Review of draft-ietf-pce-pcep-stateful-pce-gmpls
Requested revision No specific revision (document currently at 23)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2023-05-29
Requested 2023-05-15
Authors Young Lee , Haomian Zheng , Oscar Gonzalez de Dios , Victor Lopez , Zafar Ali
I-D last updated 2023-06-02
Completed reviews Genart Last Call review of -21 by Stewart Bryant (diff)
Secdir Last Call review of -21 by Ivaylo Petrov (diff)
Opsdir Telechat review of -22 by Susan Hares (diff)
Rtgdir Early review of -19 by Susan Hares (diff)
Assignment Reviewer Stewart Bryant
State Completed
Request Last Call review on draft-ietf-pce-pcep-stateful-pce-gmpls by General Area Review Team (Gen-ART) Assigned
Posted at https://mailarchive.ietf.org/arch/msg/gen-art/672eYBLYznTc_CzvJ_Pi9r6fiUY
Reviewed revision 21 (document currently at 23)
Result Ready
Completed 2023-06-02
review-ietf-pce-pcep-stateful-pce-gmpls-21-genart-lc-bryant-2023-06-02-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-pce-pcep-stateful-pce-gmpls-21
Reviewer: Stewart Bryant
Review Date: 2023-06-02
IETF LC End Date: 2023-05-29
IESG Telechat date: 2023-06-08

Summary:

I reviewed this from a GenArt perspective (a reader new to the area) and have
one request : that early in the document a few sentences be added (copied from
another text) clarifying the difference between stateless and stateful PCE.

This document is otherwise ready to pass to the RFC Editor team.

I note that id-nits picks up a number of errors regarding references (RFC 2119
language and technical references), but the ones I checked seem correct, so I
am not sure what is happening.

Major issues: None

Minor issues: The absence of an early description of stateful PCE as noted
above.

Nits/editorial comments:

   Further discussion of concept of a stateful PCE can be found in
SB> of the concept

   For passive stateful PCEs, Path Computation Request (PCReq)/ Path
   Computation Reply (PCRep) messages are used to request for path
   computation.
SB> grammar problem