§4.3.2.2
<https://datatracker.ietf.org/doc/html/draft-ietf-pce-pcep-bfd-parameters-00#section-4.3.2.2>
Multiplier:
1000..255000
This should be 1..255.
§4.3.2.3
<https://datatracker.ietf.org/doc/html/draft-ietf-pce-pcep-bfd-parameters-00#section-4.3.2.3>
LSP-SBFD-Discriminator sub-TLV
<https://datatracker.ietf.org/doc/html/draft-ietf-pce-pcep-bfd-parameters-00#name-lsp-sbfd-discriminator-sub->
Under what circumstances should this TLV be optional? S-BFD really
needs you to know the discriminator in order to work. In the case where
it's not present, was there another way that we're able to be signaled
"S-BFD is supported, but you'll learn the discriminator somewhere
else"? If so, it might be worth discussing that in the document.
Aside from these two nits, the document is clear for someone not
familiar with PCE. IMO, the document could do with a round of
simplifying some of the text since some of the procedure feels
redundantly stated. However, this might be a style choice consistent
with other PCE documents and thus is only a personal preference.
-- Jeff
_______________________________________________
Pce mailing list -- [email protected]
To unsubscribe send an email to [email protected]
