Re: [babel] Fwd: New Version Notification for draft-mahesh-babel-yang-model-00.txt

[Mahesh Jethanandani <[e-mail]>]

Hi Juliusz,

Thanks first of all for reviewing the document.

In general, you would have noticed that the description fields in the model have been cut and pasted from the information model. I am going to assume that those comments are intended for the information model also.

> On Oct 23, 2018, at 10:21 AM, Juliusz Chroboczek <[e-mail]> wrote:
> 
> Gee, thanks a lot.
> 
> The description of security.enable should mention explicitly that if set,
> unauthenticated packets are ignored on this interface.

Ok. Will add.


> 
> Interfaces.reference: is the description of "objects at any layer", which
> I understand was taken from Barbara's information model, still useful in
> Yang?  I'd expect the notion of interface to be included by reference to
> another document.

It is referencing interface definition in RFC 8343, which the model includes, using the prefix ‘if'. Therefore the type is 'if:interface-ref’. I will add an explicit reference to the RFC in the next update.

> 
> I think that external-cost has been flushed from the information model.
> Barbara?
> 
> All the seqno objects (exp-mcast-hello-seqno, route.seqno, etc.) should be
> of type uint16.  The seqnos are interpreted modulo 2^16, as described in
> Section 3.2.1 of draft-ietf-babel-rfc6126bis.

Ok. Thanks for the clarification.

> 
> All the metric objects should be of type uint16, with 0xFFFF representing
> infinity.

Ok.

> 
> I may be missing something, but I don't see an outgoing interface for the
> route object.

Would this be different or separate from the list of interfaces under the babel-interfaces-obj? The information model does not seem to call out a separate outgoing interface.

Thanks.

> 
> Thanks again,
> 
> -- Juliusz

Mahesh Jethanandani
[e-mail]