Hi, Please find some review comments for this draft...
*(1) PCRpt Message* a. There is a different in RBNF between this and draft-ietf-pce-stateful-pce-02 b. <path-list>::=<path>[<path-list>]. The status of the paths needs to reported separately in case of standby but we have only one LSP object here to specify the operational status. Also LSP-ID of primary and backup would be different. c. Incase of the delegate, all constraints configured at PCC needs to be sent to PCE. The PCRpt message should support this. d. ERO is a mandatory object and incase of the LSP's first delegate message there will be no existing path and thus no path to encode in the ERO. Either we send empty ERO i.e ERO but with no sub-objects or make ERO optional. e. In case of the first delegate message how to send the destination information? ENDPOINT object is needed. f. Support for IRO, XRO should be supported in case of PCRpt message. Most of the above comments are also applicable for PCUpd message. *(2) LSP Identifier TLVs* a. The relationship between LSP-ID in LSP Object and LSP Identifier TLV should be described. b. Maybe Tunnel Destination Address can be encoded too similar to LSP_TUNNEL_IPvX sender template c. The use of tunnel-id to describe when the traffic switches from primary and secondary is not very clear to me, maybe you can describe this here or in protection document in more details. *(3) Tunnel ID TLV* What is the need of this TLV? The description is the same as the tunnel-id already present in the LSP Identifier TLV. Regards, Dhruv
_______________________________________________ Pce mailing list [email protected] https://www.ietf.org/mailman/listinfo/pce
