Subject: [Pce] [Errata Verified] RFC7420 (4673)

From: RFC Errata System <[email-masked]>
Date: Tue, 12 Jul 2016 16:00:37 -0700 (PDT)

The following errata report has been verified for RFC7420,
"Path Computation Element Communication Protocol (PCEP) Management Information Base (MIB) Module". 

--------------------------------------
You may review the report below and at:
http://www.rfc-editor.org/errata_search.php?rfc=7420&eid=4673

--------------------------------------
Status: Verified
Type: Technical

Reported by: Mahendra Singh Negi <[email-masked]>
Date Reported: 2016-04-20
Verified by: Deborah Brungard (IESG)

Section: 4.1

Original Text
-------------
pcePcepSessState OBJECT-TYPE
       SYNTAX      INTEGER {
                      tcpPending(1),
                      openWait(2),
                      keepWait(3),
                      sessionUp(4)
                   }
       MAX-ACCESS  read-only
       STATUS      current
       DESCRIPTION
           "The current state of the session.

            The set of possible states excludes the idle state since
            entries do not exist in this table in the idle state."
       ::= { pcePcepSessEntry 3 }

Corrected Text
--------------
pcePcepSessState OBJECT-TYPE
       SYNTAX      INTEGER {
		      idle(0),
                      tcpPending(1),
                      openWait(2),
                      keepWait(3),
                      sessionUp(4)
                   }
       MAX-ACCESS  read-only
       STATUS      current
       DESCRIPTION
           "The current state of the session."
       ::= { pcePcepSessEntry 3 }	

Notes
-----
As per security consideration, if PCE needs to allow incomming connections from only known PCCs. 
Source addresses of PCCs are configured on PCE. If PCEP session on PCE goes down with configured PCCs. 
PCE needs to raise notification pcePcepSessDown (i.e. details mentioned below).
Issue is whiling sending the notification pcePcepSessDown, as session state (pcePcepSessState) defined in RFC doesn't include idle state.
I suggest to include idle(0) state for pcePcepSessState. 

   pcePcepSessDown NOTIFICATION-TYPE
       OBJECTS     {
                      pcePcepSessState,
                      pcePcepSessStateLastChange
                   }
       STATUS      current
       DESCRIPTION
           "This notification is sent when the value of
            pcePcepSessState leaves the sessionUp state."
       ::= { pcePcepNotifications 2 }

--------------------------------------
RFC7420 (draft-ietf-pce-pcep-mib-11)
--------------------------------------
Title               : Path Computation Element Communication Protocol (PCEP) Management Information Base (MIB) Module
Publication Date    : December 2014
Author(s)           : A. Koushik, E. Stephan, Q. Zhao, D. King, J. Hardwick
Category            : PROPOSED STANDARD
Source              : Path Computation Element
Area                : Routing
Stream              : IETF
Verifying Party     : IESG