Hi Jon,
Trimming the mail to the points which need commenting...
> ~ a new object for corrupted messages (note that corrupted messages are
> different from unknown messages and this cannot be derived from number of
> error messages sent either).
> Jon> Happy to have this - please could you propose some text?
pcePcepPeerNumCorruptRcvd OBJECT-TYPE
SYNTAX Counter32
MAX-ACCESS read-only
STATUS current
DESCRIPTION
"The number of corrupted PCEP message received from this peer. "
::= { pcePcepPeerEntry xx }
pcePcepSessNumCorruptRcvd OBJECT-TYPE
SYNTAX Counter32
MAX-ACCESS read-only
STATUS current
DESCRIPTION
"The number of corrupted PCEP message received on this session. "
::= { pcePcepSessEntry yy }
> - Section 5.1
> pcePcepEntityEntry OBJECT-TYPE
> SYNTAX PcePcepEntityEntry
> MAX-ACCESS not-accessible
> STATUS current
> DESCRIPTION
> "An entry in this table represents a PCEP entity."
> INDEX { pcePcepEntityIndex }
> ::= { pcePcepEntityTable 1 }
>
> ~ I think the description should not say 'this table' while describing
> an entry. Also true for pcePcepSessEntry.
>
> Jon> Not sure I understand - why?
OLD:
"An entry in this table represents a PCEP entity."
NEW:
"This entry represents a PCEP entity."
Just my preference, entirely up to you if you would like to change.
(snip)
Thanks!
Dhruv
_______________________________________________
Pce mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/pce
