Re: [Isis-wg] comments on draft-litkowski-isis-yang-isis-cfg-01

[<[e-mail]>]

Hi,

Thanks for your comment.
It seems that your are IOS CLI familiar :)
As I pointed to OSPF wg meeting, Yang datamodel does not require to be a CLI mirror.

Inline answers ...

From: Wunan (Eric) [mailto:[e-mail]]
Sent: Tuesday, July 22, 2014 15:26
To: LITKOWSKI Stephane SCE/IBNF; [e-mail]
Cc: Lizhenbin; Yangang
Subject: comments on draft-litkowski-isis-yang-isis-cfg-01


Hi Stephane,



Some comments below. Please check.



1. +--rw isis

Should be a list here with a key like [name] or [local-id] here?



[SLI] The name of the protocol instance comes from the "routing-protocol* [name]" container in core routing model.





2. +--rw nsap-address in isis-cfg container,

"nsap-address" is used here but "system-id" is used isis-state. Suggest to use same one.

And one isis can have at least three NSAP since area address can be different.



[SLI] NSAP address type is AREA+SYSTEMID+NSEL while system-id type is only systemid that's why I needed two types.

I agree that we may extend to multiple area ... this is just a first shot , so some parts are missing ...





3. +--rw ipv4-router-id?

    +--rw ipv6-router-id?

Not sure the use here.



[SLI] Definition of address of TLV134 and 140



4. Missing vrf association ?



[SLI] No the protocol definition is within a routing-instance in the core datamodel .



5. Missing nsf ?



[SLI] NSF is cisco :( you may  refer to GR or NSR stuffs ? HA features are not there yet.



6. Missing traffic engineering ?



[SLI] Only wide metrics are there , but not traffic-eng. I was wondering if TE should be part of a separate datamodel (using augmentation) and so can be used for both OSPF and ISIS protocols ... point to discuss ...



7. Missing Timer related cfg ?



[SLI] Some timers are there ... some may be missing. Which timer are your referering to ?



8. Missing redistribution related cfg ?



[SLI] Route-filters are part of the core data model. But maybe we need to augment stuffs ... I did not look at it yet.



9. Multi-topology should allow to configure more besides ipv4-uni, ipv6-uni, ipv4-multi, ipv6-multi

These are commonly used configuration which should not be omitted.



[SLI] Please provide more details on what you want.



10. isis route configuration and operation should be included in isis-cfg and isis-state ?



[SLI] Don't understand the comment ...



11. interfaces and TE state should be included in isis-state



[SLI] Interface state is not there yet ... for TE, it must be part of the TE modeling discussion ...



_________________________________________________________________________________________________________________________

Ce message et ses pieces jointes peuvent contenir des informations confidentielles ou privilegiees et ne doivent donc
pas etre diffuses, exploites ou copies sans autorisation. Si vous avez recu ce message par erreur, veuillez le signaler
a l'expediteur et le detruire ainsi que les pieces jointes. Les messages electroniques etant susceptibles d'alteration,
Orange decline toute responsabilite si ce message a ete altere, deforme ou falsifie. Merci.

This message and its attachments may contain confidential or privileged information that may be protected by law;
they should not be distributed, used or copied without authorisation.
If you have received this email in error, please notify the sender and delete this message and its attachments.
As emails may be altered, Orange is not liable for messages that have been modified, changed or falsified.
Thank you.