Hi, Thanks for posting the adopted draft.
I think we need to make the following changes so catch all of the IANA issues associated with being Experimental. Cheers, Adrian === New section... 6.2. Experimental Error-Types and Error-Values This experiment uses a single Experimental Use error-type [I-D.farrel-pce-experimental-errors] to indicate 'PCEP-LS Error' to cover all experimental error cases. The value used needs to be consistent between all partners in the experiment, and implementations would ideally make it configurable. The following error-values apply to this experimental error-type. Error-Value | Meaning ------------+-------------------------------------------------------- 1 | LS Synchronisation Error: Error in processing the LSRpt 2 | LS Synchronisation Error: Internal PCC error 3 | Mandatory object missing: LS object missing 4 | Invalid Operation: Attempted LS Report if LS remote | capability was not advertised --- New section... 6.3. Experimental PCEP TLV Types This document uses a number of experimental PCEP TLVs. Values are chosen from the Experimental Use range. The values used need to be consistent between all partners in the experiment, and implementations would ideally make them configurable. For clarity in the document, the TLVs are indicated using the tags EXPx for different values of x, as follows. Value | Meaning -------+-------------------------------- EXP5 | LS-CAPABILITY TLV EXP7 | ROUTING-UNIVERSE TLV EXP8 | Local Node Descriptors TLV EXP9 | Remote Node Descriptors TLV EXP10 | Link Descriptors TLV EXP11 | Prefix Descriptors TLV EXP12 | Node Attributes TLV EXP13 | Link Attributes TLV EXP14 | Prefix Attributes TLV EXP15 | ROUTE-DISTINGUISHER TLV --- Throughout the document, make the following changes TBD5 --> EXP5 TBD7 --> EXP7 TBD8 --> EXP8 TBD9 --> EXP9 TBD10 --> EXP10 TBD11 --> EXP11 TBD12 --> EXP12 TBD13 --> EXP13 TBD14 --> EXP14 TBD15 --> EXP15 --- 6.2 OLD The LS reports sent by PCC MAY carry the remote link-state (and TE) information learned via existing means like IGP and BGP-LS only if both PCEP Speakers set the R (remote) Flag in the "LS Capability" TLV to 'Remote Allowed (R Flag = 1)'. If this is not the case and LS reports carry remote link-state (and TE) information, then a PCErr with error-type 19 (Invalid Operation) and error-value TBD1 (Attempted LS Report if LS remote capability was not advertised) and it will terminate the PCEP session. NEW The LS reports sent by PCC MAY carry the remote link-state (and TE) information learned via existing means like IGP and BGP-LS only if both PCEP Speakers set the R (remote) Flag in the "LS Capability" TLV to 'Remote Allowed (R Flag = 1)'. If this is not the case and LS reports carry remote link-state (and TE) information, then a PCErr with the experimental error-type (PCEP-LS Error) and error-value (Attempted LS Report if LS remote capability was not advertised) and it will terminate the PCEP session. END --- 6.3 OLD If the PCC encounters a problem which prevents it from completing the LS synchronization, it MUST send a PCErr message with error-type TBD2 (LS Synchronization Error) and error-value 2 (indicating an internal PCC error) to the PCE and terminate the session. NEW If the PCC encounters a problem which prevents it from completing the LS synchronization, it MUST send a PCErr message with the experimental error-type (PCEP-LS Error) and error-value 2 (LS Synchronization Error: Internal PCC error) to the PCE and terminate the session. END --- 6.3 OLD The PCE does not send positive acknowledgments for properly received LS synchronization messages. It MUST respond with a PCErr message with error-type TBD2 (LS Synchronization Error) and error-value 1 (indicating an error in processing the LSRpt) if it encounters a problem with the LS Report it received from the PCC and it MUST terminate the session. NEW The PCE does not send positive acknowledgments for properly received LS synchronization messages. It MUST respond with a PCErr message with with the experimental error-type (PCEP-LS Error) and error-value 1 (LS Synchronization Error: Error in processing the LSRpt) if it encounters a problem with the LS Report it received from the PCC and it MUST terminate the session. END --- 8.1 OLD The Message-Type field of the PCEP common header for the LSRpt message is set to [TBD3]. NEW The Message-Type field of the PCEP common header for the LSRpt message is set to a value from the range of Experimental Use message types. The value used needs to be consistent between all partners in the experiment, and implementations would ideally make it configurable. END --- 8.1 OLD The LS object is a mandatory object which carries LS information of a node/prefix or a link. Each LS object has a unique LS-ID as described in Section 9.3. If the LS object is missing, the receiving PCE MUST send a PCErr message with Error-type=6 (Mandatory Object missing) and Error-value=[TBD4] (LS object missing). NEW The LS object is a mandatory object which carries LS information of a node/prefix or a link. Each LS object has a unique LS-ID as described in Section 9.3. If the LS object is missing, the receiving PCE MUST send a PCErr message with the experimental error-type (PCEP-LS Error) and error-value 3 (Mandatory Object missing: LS object missing). END --- 9.3 OLD LS Object-Class is TBD6. Four Object-Type values are defined for the LS object so far: NEW LS Object-Class is assigned a value from the Experimental Use range of object classes. The value used needs to be consistent between all partners in the experiment, and implementations would ideally make it configurable. Four Object-Type values are defined for the LS object: END --- 9.3 OLD The following values are defined (some of the initial values are the same as [I-D.ietf-idr-rfc7752bis]): NEW The following values are defined (some of the values are the same as [I-D.ietf-idr-rfc7752bis]): END --- 9.3.6 OLD 9.3.6. Node Descriptors Sub-TLVs The Node Descriptors TLV (Local and Remote) carries one or more Node Descriptor Sub-TLV follows the format of all PCEP TLVs as defined in [RFC5440], however, the Type values are selected from a new PCEP-LS sub-TLV IANA registry (see Section 13.6). Type values are chosen so that there can be commonality with BGP-LS [I-D.ietf-idr-rfc7752bis]. This is possible because the "BGP-LS Node Descriptor, Link Descriptor, Prefix Descriptor, and Attribute TLVs" registry marks 0-255 as reserved. Thus the space of the sub-TLV values for the Type field can be partitioned as shown below - Range | ---------------+--------------------------------------------- 0 | Reserved - must not be allocated. | 1 .. 255 | New PCEP sub-TLV allocated according to the | registry defined in this document. | 256 .. 65535 | Per BGP registry defined by | [I-D.ietf-idr-rfc7752bis]. | Not to be allocated in this registry. All Node Descriptors TLVs defined for BGP-LS can then be used with PCEP-LS as well. One new PCEP sub-TLVs for Node Descriptor are defined in this document. +----------+-------------------+----------+----------------+ | Sub-TLV | Description | Length |Value defined in| +----------+-------------------+----------+----------------+ | 24 | SPEAKER-ENTITY-ID | Variable | [RFC8232] | +----------+-------------------+----------+----------------+ A new sub-TLV type (24) is allocated for SPEAKER-ENTITY-ID sub-TLV. The length and value fields are as per [RFC8232]. NEW 9.3.6. Node Descriptors Sub-TLV The Node Descriptors TLV (Local and Remote) carries one or more Node Descriptor Sub-TLV follows the format of all PCEP TLVs as defined in [RFC5440]. Sub-TLV values in the range 256 to 65535 match those assigned for BGP-LS [I-D.ietf-idr-rfc7752bis]. A sub-TLV type (24) is used for SPEAKER-ENTITY-ID sub-TLV. The length and value fields are as per [RFC8232]. END --- Delete sections 13 and 14 --- Add [I-D.farrel-pce-experimental-errors] to the references. _______________________________________________ Pce mailing list Pce@ietf.org https://www.ietf.org/mailman/listinfo/pce