[Pce] Martin Vigoureux's No Objection on draft-ietf-pce-association-group-09: (with COMMENT)

[Martin Vigoureux via Datatracker <[e-mail]>]

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/