Re: [ANCP] I-D Action: draft-ietf-ancp-mib-an-10.txt

Hi Stefaan,

see inline.

Regards
Markus

-----Ursprüngliche Nachricht-----
Von: DE CNODDER, STEFAAN (STEFAAN) [mailto:stefaan.de_cnodder@alcatel-lucent.com] 
Gesendet: Mittwoch, 20. März 2013 10:11
An: Freudenberger, Markus
Cc: ancp@ietf.org
Betreff: RE: [ANCP] I-D Action: draft-ietf-ancp-mib-an-10.txt

Hi Markus,

Thanks for the feedback.  For issue 1, the trap could be generated because the row is deleted from both tables.  This means that the ancpAnSessionConfigSessionId in the trap has in this case no relevant information, and might point to a wrong or non-existing entry in the table.  So I am not sure if adding this to the trap is good.
MF: Ok but the same would then also happen with the other objects in the trap as they are referenced then also by an non-exisiting table entry. However, the agent implementation in the AN must ensure the information to be delivered in the trap.

For issue 2, the state "down" is actually detailed in the next sentences: it can be down because the session is deleted, or the session is a state other than the estab state.  The estab state you find back in ancpAnCurrentSessionState.
MF: Ok but the status "other(1)" can mean anything. It is not defined that this necessarily only mean that the session is down.

regards,
Stefaan

-----Original Message-----
From: ancp-bounces@ietf.org [mailto:ancp-bounces@ietf.org] On Behalf Of Markus.Freudenberger@t-systems.com
Sent: dinsdag 19 maart 2013 13:23
To: internet-drafts@ietf.org; i-d-announce@ietf.org
Cc: ancp@ietf.org
Subject: Re: [ANCP] I-D Action: draft-ietf-ancp-mib-an-10.txt

Hi all,

Issue 1:
there are two objects for identifying an ancp sessions in the access node.    
1. ancpAnSessionConfigSessionId
2. ancpAnCurrentSessionAnInstance
The first one identifies rows in the ancpAnSessionConfigTable and in the ancpAnCurrentSessionTable wehreas the second one is a number from ANCP Adjacency Message itself and is an attribute in the ancpAnCurrentSessionTable.

Proposal:
Add the attribute ancpAnSessionConfigSessionId as a varbind in the ancpSessionUp/ancpSessionDown traps as this is the clear reference to both tables ancpAnSessionConfigTable and ancpAnCurrentSessionTable. 

Issue 2:
The definition of the ancpSessionDown trap says: "This notification is generated whenever an ANCP session goes down." bute the state 'down' is not explicitely defined in the status attribute "ancpAnCurrentSessionState" which triggers the generation of the trap.

Proposal:
Define the state 'down' in the attribute ancpAnCurrentSessionState or make the relation from the different states in the ancpAnCurrentSessionState to the phrase 'down' resp. 'operational down' more clear in the description of the ancpSessionDown trap.

Thanks and regards
Markus

-----Ursprüngliche Nachricht-----
Von: ancp-bounces@ietf.org [mailto:ancp-bounces@ietf.org] Im Auftrag von internet-drafts@ietf.org
Gesendet: Dienstag, 29. Januar 2013 09:06
An: i-d-announce@ietf.org
Cc: ancp@ietf.org
Betreff: [ANCP] I-D Action: draft-ietf-ancp-mib-an-10.txt

A New Internet-Draft is available from the on-line Internet-Drafts directories.
 This draft is a work item of the Access Node Control Protocol Working Group of the IETF.

	Title           : Access Node Control Protocol (ANCP) MIB module for Access Nodes
	Author(s)       : Stefaan De Cnodder
                          Moti Morgenstern
	Filename        : draft-ietf-ancp-mib-an-10.txt
	Pages           : 39
	Date            : 2013-01-29

Abstract:
   This memo defines a portion of the Management Information Base (MIB)
   for use with network management protocols.  In particular it defines
   objects for managing access nodes that are using the Access Node
   Control Protocol (ANCP).

The IETF datatracker status page for this draft is:
https://datatracker.ietf.org/doc/draft-ietf-ancp-mib-an

There's also a htmlized version available at:
http://tools.ietf.org/html/draft-ietf-ancp-mib-an-10

A diff from the previous version is available at:
http://www.ietf.org/rfcdiff?url2=draft-ietf-ancp-mib-an-10

Internet-Drafts are also available by anonymous FTP at:
ftp://ftp.ietf.org/internet-drafts/

_______________________________________________
ANCP mailing list
ANCP@ietf.org
https://www.ietf.org/mailman/listinfo/ancp
_______________________________________________
ANCP mailing list
ANCP@ietf.org
https://www.ietf.org/mailman/listinfo/ancp