Martin Vigoureux has entered the following ballot position for draft-ietf-pce-association-group-09: No Objection
When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.) Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html for more information about IESG DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-pce-association-group/ ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- Hi, thanks for this document, here are a couple of comments/questions. The PCEP ASSOC-Type-List TLV is optional. It MAY be carried within an OPEN object sent by a PCEP speaker in an Open message to a PCEP peer so as to indicate the list of supported Association types. This is said twice. (First paragraph of section 4.1) and then in 4.1.1: A PCEP speaker MAY include an ASSOC-Type-List TLV within an OPEN object in an Open message sent to a PCEP peer in order to advertise a set of one or more supported association types. The use of ASSOC-Type-List TLV is OPTIONAL. It doesn't hurt, but you might want to consider saying this only once. Also I note OPTIONAL vs optional Sending ASSOC-Type-List TLV is optional but it might be mandatory to send some to-be-defined Association types. Isn't that somehow conflicting? The PCEP OP-CONF-ASSOC-RANGE TLV is optional. OPTIONAL? Could you clarify the difference between a PCEP speaker does not recognize the ASSOCIATION object and a PCE peer is ... unable to process the ASSOCIATION I see that the errors thrown are different. Nits: s/protections LSPs/protection LSPs/ s/The Assoc-type MAY appear more than once/A given Assoc-type MAY appear more than once/ s/to uniquely identifying/to uniquely identify/ _______________________________________________ Pce mailing list [email protected] https://www.ietf.org/mailman/listinfo/pce
