[Pce] Comments on draft-crabbe-pce-stateful-pce-mpls-te-00

[Dhruv Dhody <[e-mail]>]

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