Re: [Netconf] comments on draft-ietf-netconf-subscribed-notifications-12

["Eric Voit (evoit)" <[e-mail]>]

> Sure, but can YP import the "Event Record" term from SN?

Sure.  It imports other terms.  Alex, do you want to bring it in?
 
> Also, I think that the definition could be improved.  It currently reads:
> 
>    Event record: A set of information detailing an event.

Yes.  But the word 'event' here is itself defined as:

   Event: An occurrence of something that may be of interest.  Examples
   include a configuration change, a fault, a change in status, crossing
   a threshold, or an external input to the system.

Reviewers have liked separation of the event itself from the record about it.

Eric

> Kent // contributor
> 
> 
> 
> > From: Martin Bjorklund, June 8, 2018 10:23 AM
> >
> <snip>
> > But the name of the leaf doesn't change the semantics.  The
> > description says "number of event records", so your casual user still
> > have to understand that a YANG Push notif is an event record.
> >
> > BTW, it is not clear from the YANG push document that a YANG push
> > notif really is an event record.  It uses the term "update record",
> > and use the term "event record" in just one place.  This term should
> > be imported from Subscribed-notifications, and used.
> 
> Update record has additional constraints.  Probably the way to handle this is to
> tweak the definition of update record in YANG push to the following:
> 
> "Update record: An event record which contains a representation of one or
> more datastore node updates.  In addition, an update record may contain
> which type of  update led to the datastore node update (e.g., whether the
> datastore node was added, changed, deleted).  Also included in the  update
> record may be other metadata, such as a subscription identifier of the
> subscription as part of which the update record was generated."
> 
> Eric
> 
> > /martin
> 
> _______________________________________________
> Netconf mailing list
> [e-mail]
> https://urldefense.proofpoint.com/v2/url?u=https-
> 3A__www.ietf.org_mailman_listinfo_netconf&d=DwICAg&c=HAkYuh63rsuhr6S
> cbfh0UjBXeMK-
> ndb3voDTXcWzoCI&r=9zkP0xnJUvZGJ9EPoOH7Yhqn2gsBYaGTvjISlaJdcZo&m=C
> hdyY-tKTlPquVOwbOXya3_PcGgirMepotW7UY8_V68&s=Bd-
> V3zeqTlVJmJa9OVqUUYf4I2RqZU5ce49Jd2xY37E&e=
>