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

"Wunan (Eric)" <eric.wu@huawei.com> Wed, 23 July 2014 18:04 UTC

Return-Path: <eric.wu@huawei.com>
X-Original-To: isis-wg@ietfa.amsl.com
Delivered-To: isis-wg@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 2EC431B2989 for <isis-wg@ietfa.amsl.com>; Wed, 23 Jul 2014 11:04:41 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.751
X-Spam-Level:
X-Spam-Status: No, score=-1.751 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, MIME_CHARSET_FARAWAY=2.45, RCVD_IN_DNSWL_MED=-2.3, RP_MATCHES_RCVD=-0.001, SPF_PASS=-0.001] autolearn=ham
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id Cx2Bh-39M5RU for <isis-wg@ietfa.amsl.com>; Wed, 23 Jul 2014 11:04:36 -0700 (PDT)
Received: from lhrrgout.huawei.com (lhrrgout.huawei.com [194.213.3.17]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id B93941B2869 for <isis-wg@ietf.org>; Wed, 23 Jul 2014 11:04:35 -0700 (PDT)
Received: from 172.18.7.190 (EHLO lhreml406-hub.china.huawei.com) ([172.18.7.190]) by lhrrg02-dlp.huawei.com (MOS 4.3.7-GA FastPath queued) with ESMTP id BHN45291; Wed, 23 Jul 2014 18:04:34 +0000 (GMT)
Received: from SZXEMA401-HUB.china.huawei.com (10.82.72.33) by lhreml406-hub.china.huawei.com (10.201.5.243) with Microsoft SMTP Server (TLS) id 14.3.158.1; Wed, 23 Jul 2014 19:04:33 +0100
Received: from SZXEMA508-MBX.china.huawei.com ([169.254.7.81]) by SZXEMA401-HUB.china.huawei.com ([10.82.72.33]) with mapi id 14.03.0158.001; Thu, 24 Jul 2014 02:04:29 +0800
From: "Wunan (Eric)" <eric.wu@huawei.com>
To: "stephane.litkowski@orange.com" <stephane.litkowski@orange.com>, "isis-wg@ietf.org" <isis-wg@ietf.org>
Thread-Topic: comments on draft-litkowski-isis-yang-isis-cfg-01
Thread-Index: Ac+l4qtZhrh5j6gJR3OwDOjb/Qkn+QAA+BmwAC3vweo=
Date: Wed, 23 Jul 2014 18:04:28 +0000
Message-ID: <0F26584357FD124DB93F1535E4B0A65036804A82@szxema508-mbx.china.huawei.com>
References: <0F26584357FD124DB93F1535E4B0A65036804905@szxema508-mbx.china.huawei.com>, <4223_1406059402_53CEC38A_4223_4491_1_9E32478DFA9976438E7A22F69B08FF92044506@OPEXCLILM34.corporate.adroot.infra.ftgroup>
In-Reply-To: <4223_1406059402_53CEC38A_4223_4491_1_9E32478DFA9976438E7A22F69B08FF92044506@OPEXCLILM34.corporate.adroot.infra.ftgroup>
Accept-Language: zh-CN, en-US
Content-Language: zh-CN
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.47.130.25]
Content-Type: multipart/alternative; boundary="_000_0F26584357FD124DB93F1535E4B0A65036804A82szxema508mbxchi_"
MIME-Version: 1.0
X-CFilter-Loop: Reflected
Archived-At: http://mailarchive.ietf.org/arch/msg/isis-wg/7_VH7bSczs5VZIWfsXFhz7634QA
Subject: Re: [Isis-wg] comments on draft-litkowski-isis-yang-isis-cfg-01
X-BeenThere: isis-wg@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: IETF IS-IS working group <isis-wg.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/isis-wg>, <mailto:isis-wg-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/isis-wg/>
List-Post: <mailto:isis-wg@ietf.org>
List-Help: <mailto:isis-wg-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/isis-wg>, <mailto:isis-wg-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 23 Jul 2014 18:04:41 -0000

Hi Stephane,



Have to say you are also familiar with IOS CLI too. ( Alright we need to stop doing advertisement here:) )



And it is correct YANG need not to be CLI mirror but it needs to contain its counterpart for fulfilling YANG's configuration objective.



Please check responses below.



Regards

Eric



________________________________
发件人: stephane.litkowski@orange.com [stephane.litkowski@orange.com]
发送时间: 2014年7月23日 4:03
收件人: Wunan (Eric); isis-wg@ietf.org
抄送: Lizhenbin; Yangang
主题: RE: comments on draft-litkowski-isis-yang-isis-cfg-01

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:eric.wu@huawei.com]
Sent: Tuesday, July 22, 2014 15:26
To: LITKOWSKI Stephane SCE/IBNF; isis-wg@ietf.org
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.

[Eric] GR. Sorry for that face here.



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 …

[Eric] Agree to be a separate model. That's will be better.



7. Missing Timer related cfg ?



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

[Eric] For example, spf delay timer.



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.

[Eric] Multi-topology for non-standard ones. This model should allow this.



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



[SLI] Don’t understand the comment …

[Eric] Does isis model need to expand the route attributes from core model?
            How about adding state and rpc operation for isis route ?



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.