Re: [Lsr] AD Review of draft-ietf-isis-yang-isis-cfg-35
"Acee Lindem (acee)" <acee@cisco.com> Thu, 05 September 2019 19:09 UTC
Return-Path: <acee@cisco.com>
X-Original-To: lsr@ietfa.amsl.com
Delivered-To: lsr@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 67F73120AF7; Thu, 5 Sep 2019 12:09:45 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.499
X-Spam-Level:
X-Spam-Status: No, score=-14.499 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-0.001, URIBL_BLOCKED=0.001, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=cisco.com header.b=Woa8A9/z; dkim=pass (1024-bit key) header.d=cisco.onmicrosoft.com header.b=U8YyKA1Q
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 C1IXzX-D3Eeb; Thu, 5 Sep 2019 12:09:41 -0700 (PDT)
Received: from rcdn-iport-2.cisco.com (rcdn-iport-2.cisco.com [173.37.86.73]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 87C421200C3; Thu, 5 Sep 2019 12:09:41 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=55845; q=dns/txt; s=iport; t=1567710581; x=1568920181; h=from:to:cc:subject:date:message-id:references: in-reply-to:mime-version; bh=7k1DEvNx33jI+bfJtOf/liHVUGrjVXYA2BJUjAx1dV4=; b=Woa8A9/zfvNzBmoAPe6ylM6SZApmsZrjGq5yhtHcdMoQUjxdFkqikARv 3GllE1n5jpU1TNhEXq7WB0SBAWnM9kmKKFpTYp5Pme5oAyzc8h+WwFKru lncZjVJpEBmBFe4g7pX79zMgg8xPiBHBU8RZPH90svUObzIFchkmW4OgP w=;
IronPort-PHdr: 9a23:Lo2qYBAb5B7GFETlpDbhUyQJPHJ1sqjoPgMT9pssgq5PdaLm5Zn5IUjD/qgw3kTRU9Dd7PRJw6rNvqbsVHZIwK7JsWtKMdRXUgMdz8AfngguGsmAXETwIfPCZC0hF8MEX1hgrDm2
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A0DbAQDbXHFd/4UNJK1lGwEBAQEDAQEBBwMBAQGBZ4EWL1ADbVYgBAsqhCGDRwOKdIJciWGOC4JSA1QJAQEBDAEBLQIBAYQ/AheCHyM4EwIDCQEBBAEBAQIBBgRthS4MhUoBAQEBAxIRHQEBNwEPAgEIDgMDAQIhAQYDAgICHxEUCQgCBAENBRsHgwABgR1NAx0BAp94AoE4iGFzgTKCfQEBBYUVDQuCFgmBNIt4GIF/gRABJx+CTD6CGoJKBhCCVTKCJo82hSGJEo4KQQqCH5Bqg34bgjSHPI8GjXqKA45aAgQCBAUCDgEBBYFpIYFYcBU7KgGCQYJCg3KKU3OBKY4nAQE
X-IronPort-AV: E=Sophos;i="5.64,471,1559520000"; d="scan'208,217";a="629431017"
Received: from alln-core-11.cisco.com ([173.36.13.133]) by rcdn-iport-2.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 05 Sep 2019 19:09:13 +0000
Received: from XCH-ALN-011.cisco.com (xch-aln-011.cisco.com [173.36.7.21]) by alln-core-11.cisco.com (8.15.2/8.15.2) with ESMTPS id x85J9DaB023218 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Thu, 5 Sep 2019 19:09:13 GMT
Received: from xhs-rcd-003.cisco.com (173.37.227.248) by XCH-ALN-011.cisco.com (173.36.7.21) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Thu, 5 Sep 2019 14:09:13 -0500
Received: from xhs-rtp-002.cisco.com (64.101.210.229) by xhs-rcd-003.cisco.com (173.37.227.248) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Thu, 5 Sep 2019 14:09:11 -0500
Received: from NAM05-DM3-obe.outbound.protection.outlook.com (64.101.32.56) by xhs-rtp-002.cisco.com (64.101.210.229) with Microsoft SMTP Server (TLS) id 15.0.1473.3 via Frontend Transport; Thu, 5 Sep 2019 15:09:11 -0400
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=EDZ6fSkoy05OZ+PGSEECSYa9bXRiqhHPwbfDX9Yf5NOk1ckgr0gge13jNz/0snGklwh03+ymF7z71TK+2dSF18ICreVIsTtlLCD7z0IvNiB8ryR2Lwx6mzn4PwXW3tEzBEuBpp1tSDngZimAXJ0S9qnpDuyCUH0kojpwJM19xRVeKBZ3Aie3B76bBhuSOtkwmq/xEgwje06vod372ZoJgzOksCrKxF4xK7XHPH13RqWlGPg6dLz5vx0++411kfYjoe705sNoE9JrRdMJqRQLhCrAbUCQ3CgKvFuZjEDcSFkUblZyriX0kOTp5v6GRhsH6pG59V/T7yzKpeb8AAYmZw==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=7k1DEvNx33jI+bfJtOf/liHVUGrjVXYA2BJUjAx1dV4=; b=efk0cfob/Iai3kGQzzsdXaAUV/y2rArrUvaFbvbgy4hWzZ4nax4yT0yPlWTFenNbV10+bRklp2hRQxiMGy3p1gz2EwRTDqexUbV8MBgLz+VrqJnGhJVpZa2IOA/c/+BEDS2N7XLuNqbP7PrJBysxE5wLyzxFg9v5Bj0GxtutpERdEmcHmlgiPUnog+CljqicoGzgL8e5Hw/HIMzs1jGrU0Dqy5XsZjUeg9fPjbf6KJDAqs6AnH+kzzX9/1VDY/BQE4HZp7cAWR9mzlx5T7exF7On+vIblo/w4rw4VP5GlzqQlSVR++aIysddkAdICHcD3ScKkF5ggFV0jrUxFd9VWg==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=cisco.com; dmarc=pass action=none header.from=cisco.com; dkim=pass header.d=cisco.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cisco.onmicrosoft.com; s=selector2-cisco-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=7k1DEvNx33jI+bfJtOf/liHVUGrjVXYA2BJUjAx1dV4=; b=U8YyKA1QFLkI02sWyO3ttZSPaKzvRqTELhryWvequdsAUDH91aqqTJQvTvZf5F864w+dsHISYZBUaolgkj1nwNSERYN8hAR3wsbWrh10GAKHOd83nr6XZjt/Wyej/4TvgBhhZpfCofQ+c3f7jm47Jbye41iBQ8/u2pDA2tTukvI=
Received: from MN2PR11MB4221.namprd11.prod.outlook.com (52.135.38.14) by MN2PR11MB4223.namprd11.prod.outlook.com (52.135.37.76) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2220.20; Thu, 5 Sep 2019 19:09:10 +0000
Received: from MN2PR11MB4221.namprd11.prod.outlook.com ([fe80::cdc1:a2cf:eb3:a420]) by MN2PR11MB4221.namprd11.prod.outlook.com ([fe80::cdc1:a2cf:eb3:a420%6]) with mapi id 15.20.2220.022; Thu, 5 Sep 2019 19:09:10 +0000
From: "Acee Lindem (acee)" <acee@cisco.com>
To: Alvaro Retana <aretana.ietf@gmail.com>, "draft-ietf-isis-yang-isis-cfg@ietf.org" <draft-ietf-isis-yang-isis-cfg@ietf.org>, "stephane.litkowski@orange.com" <stephane.litkowski@orange.com>
CC: Yingzhen Qu <yingzhen.ietf@gmail.com>, "lsr@ietf.org" <lsr@ietf.org>, "lsr-chairs@ietf.org" <lsr-chairs@ietf.org>
Thread-Topic: AD Review of draft-ietf-isis-yang-isis-cfg-35
Thread-Index: AQHVGx1XJvorFxTZhUGqUCu6BrErNKavk6QAgDRQRwCAOd2WgA==
Date: Thu, 05 Sep 2019 19:09:09 +0000
Message-ID: <9AD5F98B-69BB-40F9-A0C1-79FBFF84A74F@cisco.com>
References: <CAMMESsxQGGj_PmmjeRqBfTgU=Z2=Eu8Yn9FXLHgEm1PorTaUqA@mail.gmail.com> <19628_1561638987_5D14B84B_19628_476_1_9E32478DFA9976438E7A22F69B08FF924C25B532@OPEXCAUBMA3.corporate.adroot.infra.ftgroup> <CAMMESszTQodYdTUc1y=vO_ppROOTWJiY9pxrsNdtDgLU3PfkZg@mail.gmail.com>
In-Reply-To: <CAMMESszTQodYdTUc1y=vO_ppROOTWJiY9pxrsNdtDgLU3PfkZg@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: spf=none (sender IP is ) smtp.mailfrom=acee@cisco.com;
x-originating-ip: [2001:420:c0c4:1007::54]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 923e2b1e-19c9-4e6c-df8a-08d732348846
x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600166)(711020)(4605104)(1401327)(2017052603328)(7193020); SRVR:MN2PR11MB4223;
x-ms-traffictypediagnostic: MN2PR11MB4223:
x-ms-exchange-purlcount: 2
x-microsoft-antispam-prvs: <MN2PR11MB4223C62AA8431F25598AE214C2BB0@MN2PR11MB4223.namprd11.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-forefront-prvs: 015114592F
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(4636009)(396003)(136003)(376002)(39860400002)(346002)(366004)(51444003)(189003)(199004)(8936002)(6512007)(8676002)(81156014)(81166006)(256004)(54896002)(6306002)(9326002)(14444005)(6436002)(71200400001)(71190400001)(2501003)(53946003)(236005)(2616005)(7736002)(46003)(4326008)(478600001)(476003)(76176011)(14454004)(486006)(446003)(25786009)(186003)(53546011)(6506007)(11346002)(102836004)(5660300002)(6116002)(6486002)(110136005)(66574012)(53936002)(6246003)(99286004)(54906003)(33656002)(36756003)(64756008)(66476007)(66446008)(66556008)(76116006)(66946007)(229853002)(2201001)(316002)(2906002)(86362001); DIR:OUT; SFP:1101; SCL:1; SRVR:MN2PR11MB4223; H:MN2PR11MB4221.namprd11.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1;
received-spf: None (protection.outlook.com: cisco.com does not designate permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam-message-info: HnAVHLPEbubdN/QeWUX85H0raiMLLpAKWRcTguvM2v9PyDYMch/vBmJRCrw7LAAjfx8lQl7VWy4Ee9/KH42z88VRXhvl1N6ikDf6rmUgoH2NbLyiSe1D3X4Wmqt7k5sHIlSt7njzBu7d2dEvsoe40Btmca92D4kh2R6g1cr83qToskWiTYxytD58Z6pgMhBpc8niOGhB89qbwn5rsWf2HUvDSFNU8zpaSDjG0kwT45aJVqtsMi03pEmZpZ5EQO//aV4oJ0DBpP2qhwH6Ma2nv1HhCHK+/mu7hhk1LdsGxD4VGPM1y2RXfJkugM5boV2vmjZ6Bck75dICo7pfOI5FF5pymi1W0ZOeFzjLo6r/neeK1ixlGj6TCamol0rfbeASkgs6dQLRCJSe9+P/txe1KV2YNvQl+Tw/mLsJnjeNDQE=
x-ms-exchange-transport-forked: True
Content-Type: multipart/alternative; boundary="_000_9AD5F98B69BB40F9A0C179FBFF84A74Fciscocom_"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 923e2b1e-19c9-4e6c-df8a-08d732348846
X-MS-Exchange-CrossTenant-originalarrivaltime: 05 Sep 2019 19:09:09.9440 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5ae1af62-9505-4097-a69a-c1553ef7840e
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: NszPeVOrS4UdFMjZrczx3Cj/pOuz/m1spOjpT6KC8EEtpXsOOazPWuR6UTZu6wwW
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR11MB4223
X-OriginatorOrg: cisco.com
X-Outbound-SMTP-Client: 173.36.7.21, xch-aln-011.cisco.com
X-Outbound-Node: alln-core-11.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/lsr/Hqw0OjR4arFtPPVnK2cBNDrxH6s>
Subject: Re: [Lsr] AD Review of draft-ietf-isis-yang-isis-cfg-35
X-BeenThere: lsr@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Link State Routing Working Group <lsr.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/lsr>, <mailto:lsr-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/lsr/>
List-Post: <mailto:lsr@ietf.org>
List-Help: <mailto:lsr-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/lsr>, <mailto:lsr-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 05 Sep 2019 19:09:46 -0000
Hi Alvaro, I see you approved for IESG review. Stephane and I have some more updates in -36 version. See inline. From: Alvaro Retana <aretana.ietf@gmail.com> Date: Tuesday, July 30, 2019 at 3:29 PM To: "draft-ietf-isis-yang-isis-cfg@ietf.org" <draft-ietf-isis-yang-isis-cfg@ietf.org>, Stephane Litkowski <stephane.litkowski@orange.com> Cc: Yingzhen Qu <yingzhen.ietf@gmail.com>, "lsr@ietf.org" <lsr@ietf.org>, "lsr-chairs@ietf.org" <lsr-chairs@ietf.org> Subject: RE: AD Review of draft-ietf-isis-yang-isis-cfg-35 Resent-From: <alias-bounces@ietf.org> Resent-To: Yingzhen Qu <yingzhen.ietf@gmail.com>, Christian Hopps <chopps@chopps.org>, Acee Lindem <acee@cisco.com> Resent-Date: Tuesday, July 30, 2019 at 3:29 PM On June 27, 2019 at 8:36:46 AM, stephane.litkowski@orange.com<mailto:stephane.litkowski@orange.com> (stephane.litkowski@orange.com<mailto:stephane.litkowski@orange.com>) wrote: Stephane: Hi! Sorry it took me while to get to your reply. Thanks for your comments. We are working on updating the document accordingly. Please find some replies inline that may require more discussion. I have stripped the comments that will be fixed in the next revision. Just leaving some text with answers. Thanks! Alvaro. ... 474 Some parameters like "overload bit" and "route preference" are not 475 modeled to support a per level configuration. If an implementation 476 supports per level configuration for such parameter, this 477 implementation SHOULD augment the current model by adding both 478 level-1 and level-2 containers and SHOULD reuse existing 479 configuration groupings. [major] "...SHOULD augment the current model by adding both level-1 and level-2 containers" What other way would that be done? I think that Normative language is not needed in this case. [SLI] Using YANG there are multiple ways to do things. People may create new containers, use different namings… and we want to keep the modeling consistency even in the future augmentations. Ok…why not use MUST then? Are there cases where it would be ok to not be consistent? IOW, the current wording doesn’t guarantee consistency… This is a MUST in the -36 version. ... 978 sequence-number-skipped: This notification is sent when the system 979 receives a PDU with its own system ID and different contents. The 980 system has to reissue the LSP with a higher sequence number. [nit] That's the last thing I would have guessed that this action would have been called... Maybe it's just me... [SLI] This is inherited from RFC4444 I guess operators are familiar with this SNMP Trap misnomer. 982 authentication-type-failure: This notification is sent when the 983 system receives a PDU with the wrong authentication type field. 985 authentication-failure: This notification is sent when the system 986 receives a PDU with the wrong authentication information. [minor] Why do we need both of these? Given that they both provide the same information (including the raw PDU), and that authentication-type-failure is a specific case of receiving "a PDU with the wrong authentication information" [SLI] This is inherited from RFC4444 You used this reply in several places. Inheriting things from rfc4444 doesn’t mean it is ok…or the best solution… In either case, most are minor comments, so no big deal. I just want to make sure that some of the points were considered. In one case the wrong type of authentication is used, e.g., none vs auth-trailer and in the other the neighbors agree on authentication type but authentication fails. ... 1239 feature nsr { 1240 description 1241 "Non-Stop-Routing (NSR) support."; 1242 } [minor] Reference? [SLI] NSR is a local well known and deployed mechanism. We may enhance the description if required, however we cannot really provide a reference. Yes, please do. See the description in draft-ietf-ospf-yang. This is included in the -36 version. ... 3995 grouping tlv242-router-capabilities { 3996 container router-capabilities { 3997 list router-capability { 3998 leaf flags { 3999 type bits { 4000 bit flooding { 4001 position 0; 4002 description 4003 "If the S bit is set, the IS-IS Router CAPABILITY 4004 TLV MUST be flooded across the entire routing 4005 domain. If the S bit is clear, the TLV MUST NOT 4006 be leaked between levels. This bit MUST NOT 4007 be altered during the TLV leaking."; 4008 } [major] This is a description of the behavior (copied from rfc7981!), not a description of the field. [SLI] Yes, but this provides an accurate description on the conditions of the flag setting. But this document is not Normative in the behavior above…. If anything, Normative language should not be used unless making it clear that it is a direct quote. This is represented as an RFC 7981 quote in the -36 version. ... 4540 notification lsp-too-large { 4541 uses notification-instance-hdr; 4542 uses notification-interface-hdr; 4544 leaf pdu-size { 4545 type uint32; 4546 description "Size of the LSP PDU"; 4547 } 4548 leaf lsp-id { 4549 type lsp-id; 4550 description "LSP ID"; 4552 } 4553 description 4554 "This notification is sent when we attempt to propagate 4555 an LSP that is larger than the dataLinkBlockSize for the 4556 circuit. The notification generation must be throttled 4557 with at least 5 seconds between successive 4558 notifications."; 4559 } ... [major] "must be throttled" Why is this text not Normative? It seems to me that throttling is a good practice...in fact, it may be a good idea to specify it for all notifications. There are 12 total instances of the same text. [SLI] The text is mirrored from RFC4444. I think this was the only “major” point what you replied with rfc4444…. Just leaving it here…. No further comments. ;-) We didn’t change this notification throttling from RFC 4444. I’m not sure if it should be specified in the YANG model or be something that an implementation provides via configuration for all notifications. Thanks, Acee and Stephane
- [Lsr] AD Review of draft-ietf-isis-yang-isis-cfg-… Alvaro Retana
- Re: [Lsr] AD Review of draft-ietf-isis-yang-isis-… stephane.litkowski
- Re: [Lsr] AD Review of draft-ietf-isis-yang-isis-… Alvaro Retana
- Re: [Lsr] AD Review of draft-ietf-isis-yang-isis-… Acee Lindem (acee)
- Re: [Lsr] AD Review of draft-ietf-isis-yang-isis-… Alvaro Retana
- Re: [Lsr] AD Review of draft-ietf-isis-yang-isis-… Alvaro Retana
- Re: [Lsr] AD Review of draft-ietf-isis-yang-isis-… Acee Lindem (acee)
- Re: [Lsr] AD Review of draft-ietf-isis-yang-isis-… Alvaro Retana