Re: [Teas] I-D Action: draft-ietf-teas-yang-rsvp-16.txt etc

t petch <ietfa@btconnect.com> Tue, 10 August 2021 09:42 UTC

Return-Path: <ietfa@btconnect.com>
X-Original-To: teas@ietfa.amsl.com
Delivered-To: teas@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 7C4303A0B5B; Tue, 10 Aug 2021 02:42:21 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -0.029
X-Spam-Level:
X-Spam-Status: No, score=-0.029 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, FAKE_REPLY_A1=1.871, MSGID_FROM_MTA_HEADER=0.001, RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=no autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=btconnect.onmicrosoft.com
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 l8k2ykZP7X26; Tue, 10 Aug 2021 02:42:16 -0700 (PDT)
Received: from EUR05-AM6-obe.outbound.protection.outlook.com (mail-am6eur05on2129.outbound.protection.outlook.com [40.107.22.129]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 253DD3A0B52; Tue, 10 Aug 2021 02:42:16 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ku/FKQB0PVuXwrOua9ruQXBgm6F8NqF3AQdgr15m35x+3mo5A5pfapB3xXItFqAQ2baPFxGPO/E/CgxW+6dOApylON8AqMNd5b2cGB2IqzfyUQI5HUqJqNaPOZUrKpmNfvXgeUaTroWl6mQrSM7hLctsFnylPVHEolEI+bukGlmatbixCSv1M4PMOxBteBGJ1qIrFacCK8IE8hQRBwn8FEK/e5pZLKb/crOIH4Hur3p34vY9YiqwvC8DxldA7FbYkjhpnSBRX2MiWSyFE887c/dYffmb9Dwe1D2l/n3T6PSIoWuEnanjWzkhBRT3zjluF8lAByN5Q78KK0gCUY88Ww==
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=sLHTmC2Fq6nKOqfVt3o1Ki8BW6rMT9wNWQ1YAjOFqZg=; b=SboHOdjww73XnDXmihQja8fDu4szcFMPSZPtB6DghE2bFruK66I48FUc3nYceco7Nly/CSDoXawyWapmP0Fh942eueO88qFiEtiT8MTsm0IIlvkjkQVIgtXv3F23ZqPJC9et+aLLzV1SuqW6qNzD/m/SpeVnZTL+fjg7bLAwNwssBeFo1sP9KRccQ5TQVnSZEgkJd2MqY2hP4WfZW4ggoz1PNo91yjfBAeoTdBG4XBoJXkjumA4nyh/IAWH8m8JG6fkvPsuJsDXUEnce2JjbXLwhH7H7sfCkjvfHP87467Ot7+ic9ouF4IlrINQO1c1PLOhJAtw2aoYscrBsvvNv0w==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=btconnect.com; dmarc=pass action=none header.from=btconnect.com; dkim=pass header.d=btconnect.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=btconnect.onmicrosoft.com; s=selector2-btconnect-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=sLHTmC2Fq6nKOqfVt3o1Ki8BW6rMT9wNWQ1YAjOFqZg=; b=olwC00y1bJGQDEDs54u6hWmY/0GjADXpITJOfMce2a0Q74YcACE0Xy7WNNJEVLJfGDmEk26oEgorLaAbY21viK+Hew+1Oe2ESc6ZPkt2laDsdmzyylX7r1FqOtECbpcPpPqsKSftI73dGSkw3obxEaRiVUPenIRSl5Q62ChjJ4Q=
Authentication-Results: ietf.org; dkim=none (message not signed) header.d=none;ietf.org; dmarc=none action=none header.from=btconnect.com;
Received: from DB7PR07MB5546.eurprd07.prod.outlook.com (2603:10a6:10:73::23) by DB7PR07MB4572.eurprd07.prod.outlook.com (2603:10a6:5:2b::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4415.9; Tue, 10 Aug 2021 09:42:10 +0000
Received: from DB7PR07MB5546.eurprd07.prod.outlook.com ([fe80::51be:6d5a:9b3f:ac8c]) by DB7PR07MB5546.eurprd07.prod.outlook.com ([fe80::51be:6d5a:9b3f:ac8c%5]) with mapi id 15.20.4415.012; Tue, 10 Aug 2021 09:42:10 +0000
From: t petch <ietfa@btconnect.com>
To: Tarek Saad <tsaad=40juniper.net@dmarc.ietf.org>, teas@ietf.org, teas-chairs@ietf.org
Message-ID: <611249EA.4090700@btconnect.com>
Date: Tue, 10 Aug 2021 10:42:02 +0100
User-Agent: Mozilla/5.0 (Windows NT 5.1; rv:38.0) Gecko/20100101 Thunderbird/38.5.0
Content-Type: text/plain; charset=windows-1252; format=flowed
Content-Transfer-Encoding: 7bit
X-ClientProxiedBy: LO4P123CA0308.GBRP123.PROD.OUTLOOK.COM (2603:10a6:600:197::7) To DB7PR07MB5546.eurprd07.prod.outlook.com (2603:10a6:10:73::23)
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
Received: from [192.168.1.65] (86.146.121.231) by LO4P123CA0308.GBRP123.PROD.OUTLOOK.COM (2603:10a6:600:197::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.4394.17 via Frontend Transport; Tue, 10 Aug 2021 09:42:09 +0000
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id: e59808ea-dd39-4615-4c19-08d95be31fe2
X-MS-TrafficTypeDiagnostic: DB7PR07MB4572:
X-Microsoft-Antispam-PRVS: <DB7PR07MB4572E333400AB88525563AEAA2F79@DB7PR07MB4572.eurprd07.prod.outlook.com>
X-MS-Oob-TLC-OOBClassifiers: OLM:9508;
X-MS-Exchange-SenderADCheck: 1
X-MS-Exchange-AntiSpam-Relay: 0
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: 7ZFKBrIJvuqLcerdmdV27d5i91Jrff4m9rKyT3uoVHSnxpHgLX7VhaMQToljE5DTzbSSYMQaHix7Dyc/16zFHVyZWuhuD9F8q8FT1m4rXtGcXdjITsosl+5HCOAMZDf7ZD10WLi/W3S9eHeItr3EWZyyBzYp2ljsW5If/A2YrErVkIdXvt78r7I2bCvnb965ijDa7eqrulWxspzih0xk4kHPxzsIyZy+0dcFD7Cng3rQEDpbD3qnHjHnKFpzgujqgJmHex7fINq28P/OVM+BNqYltZlULfbFkWU6n72Wt4SYHCV05Eat+WMTk3AtVwzpQ7aENMiKIWASAU1UFciui5njVjHE97R5Kwu8XfV+lPidAySmzUph5gPzvYGA3dedFPoiCeS6H3wmn0ckC1FXUPR/TkFXZUftfuliryA3Xkni5rxJ58dhOJbbeg+Uuu/nJv3r8g/mchw55XW4q124NGNdOHtXLj5K0oNuJ4bF2rQBK6FryBSz3Gvdgsg61CjEai5NazWnzXOZ2ksnHgykUDbaYmv/WtsJDCGP7vIQ7ghzCylFb3hQEI6cYr0V1ntc0xAtwCWvVjXRyQJ0msKsZxcIVsgj9Q1NiniDTg8JMP7tt1lR8g6NnHsD23fh5XcGPbXepLe/4MNqIV3c2AGMGTYm4b0bz7lz8QHAaxEluSWjEFxwZ5zjYJLxJ/mWQPgcqPeBBHie30cCO/ewLzS2nvswxXl7KFOxHau3vNtewyD145Tw0SKOajzXfB7i7mE93c6dgDYvCHILA2gxVB8nEQ==
X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:DB7PR07MB5546.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFS:(366004)(66946007)(8676002)(6666004)(186003)(6486002)(38100700002)(38350700002)(66476007)(66556008)(2906002)(956004)(2616005)(8936002)(33656002)(508600001)(36756003)(16576012)(86362001)(53546011)(26005)(5660300002)(87266011)(316002)(52116002)(83380400001); DIR:OUT; SFP:1102;
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?Windows-1252?Q?YVy+zbib0oytemZN+snHMODPltk2wlCikp1yp+yjDVTAYXdfUumqAex1?= =?Windows-1252?Q?q6EtCCROqEZkm5rsOirNRVRPA55cX7I3BGy1Q1uTVrby+ZeZbeKu74ed?= =?Windows-1252?Q?WUe3Kz7T7IxxsgA6jhLqhRzdNMluiml0BpU/fMBEzb/k0NQNdEW8F+XT?= =?Windows-1252?Q?W1o3s9uEsfFYtcdK7XouDStC6JxDpldmTpgvuAqqPLs3srpvPFwPZPkM?= =?Windows-1252?Q?QfC+0KMa5x+Co8aPvylsnz6UFGUkEVwGkEkL0JS666ghbnOOyXV1fS6D?= =?Windows-1252?Q?/p2M/Xdp4WGq9p08gGoGnwyPFjbP4pKJe8CLTp5VzLhmuN/2ENjdKQCZ?= =?Windows-1252?Q?KP+sr9bq1J4059PLl1YobmTYv9DS4lQRlFmZDCaURioypfXr8CB8maGF?= =?Windows-1252?Q?NK4oTPtUSu0y9yOIQypP68Kf2LvIaXZjdY0/cBHDmKW9v9JZq7jAsxum?= =?Windows-1252?Q?I2Ebu8rF9A1qpO8EkhQrG286Ngtne0ZVbV/+P/MzIH/SiMwODtmiehV8?= =?Windows-1252?Q?/2DQ5lTdjym2civFG8LOsRQf2wuayazRtznKf9ReuW9dFrFmKMZCgVoO?= =?Windows-1252?Q?ce166CpemQfMvBoqWMyT50psS0NV6qbQy5I3WrwRy2CrV2OlX/XWeWkA?= =?Windows-1252?Q?V9QS6PT9JBn2qXT7GRPJ9jDN9+szjIuA6z9jcmXLUbSEXiOtcFA6GNkG?= =?Windows-1252?Q?t7KIUrKKgYw0tnipPQJBvN+3YxLvjKbs7Oisea+JpIiNDj9SNo9ijKlc?= =?Windows-1252?Q?k5bxT69242mzY7SSVNvDE2eorxPz+CXFeSYSFRTr2u126Lq4uZz8FKF7?= =?Windows-1252?Q?Qrp+Csplh83dRL3NOAjibyXFUuQOk+nT8GUaAtNFdTFcowSpBfBsfrCL?= =?Windows-1252?Q?gcX3+XPBxD+WP9zF2nz7t5yoJvLTz/iLyQNgEo6ZWUSEKUS5rrR5XD5t?= =?Windows-1252?Q?FC+KquspIYsiJY6NjNFB8EaaTkF+v1V69a/PZR11ZuyaK1tnOG0cBSR/?= =?Windows-1252?Q?MJ2mqgLCYj18O7HNZwu8a8tHLayk4OZ2SdduiLZmAkwgx30+1M9MSdco?= =?Windows-1252?Q?A75qBi7TXeiVl5oAuURgQ4G+JsBaSeMLZDcXSXcTOx8Ij2OOgdvSKDjw?= =?Windows-1252?Q?xFG7mQFU1l54KxK8I2teMIkNBPCaFxdMQC/bxHW39Fw2ZFmj62SikA1J?= =?Windows-1252?Q?hAUiBR8F9/EhiBGmfBH2KpNO7cqOD8FKQa83bfjTupq+i0maUkm1JFki?= =?Windows-1252?Q?DQWdVN47yQrfbChe6PI61PqN3OS8pgtADcKbkvOygAn1Su2heEMuzsBO?= =?Windows-1252?Q?YULLPkFVM8mbXrNyevqMI8Yncu2Rd4tolQ7jOceQB8+S6i3blDVpC+MB?= =?Windows-1252?Q?dw5y7hNTWGT4ju4HSH3/D0lZwgU10L31Iun58yUyaEHqhA56b9lU4Plm?=
X-OriginatorOrg: btconnect.com
X-MS-Exchange-CrossTenant-Network-Message-Id: e59808ea-dd39-4615-4c19-08d95be31fe2
X-MS-Exchange-CrossTenant-AuthSource: DB7PR07MB5546.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Aug 2021 09:42:10.2302 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: cf8853ed-96e5-465b-9185-806bfe185e30
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 1xHW5OJpp7A0lS1IZ+llaQ4dl1k8J6LvnJ8QMY2sYnogSubRmcPEN0XF7TL+QRNLSWo6zURhOdW5UboNJQhbJA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB7PR07MB4572
Archived-At: <https://mailarchive.ietf.org/arch/msg/teas/xZytXxXODVZ53EKHa-NgSPNx2Vk>
Subject: Re: [Teas] I-D Action: draft-ietf-teas-yang-rsvp-16.txt etc
X-BeenThere: teas@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Traffic Engineering Architecture and Signaling working group discussion list <teas.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/teas>, <mailto:teas-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/teas/>
List-Post: <mailto:teas@ietf.org>
List-Help: <mailto:teas-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/teas>, <mailto:teas-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 10 Aug 2021 09:42:22 -0000

Some final comments (pro tem.).

'epoch' has a reference of RFC5063; yes that RFC uses the term but
RFC2961 defines it and seems to me the better reference

' purged if no states associated with it."; '
not quite English - not sure what it is trying to say

              + "/rsvp:sessions/rsvp:session-ip/destination";
                    + "/rsvp:neighbors/rsvp:neighbor/address";
both would be better with another ' rsvp: '

        "WG Web:   <http://tools.ietf.org/wg/teas/>
should be https: (I have seen an AD raise a DISCUSS on this:-)

        leaf interface-based {
          type empty;
Why not boolean (or a presence container)?

        leaf hello-interval {
RFC3209 uses hello_interval but I think that you are right to change it

          units "milliseconds";
          default "9000";
RFC3209 says
"  A node periodically generates a Hello message containing a HELLO
    REQUEST object for each neighbor who's status is being tracked.  The
    periodicity is governed by the hello_interval.  This value MAY be
    configured on a per neighbor basis.  The default value is 5 ms."
Is that the same concept? If so, the values are rather different.

          reference
            "RFC3209: RSVP-TE: Extensions to RSVP for LSP Tunnels.
             RFC5495: Description of the Resource Reservation Protocol -
I am unclear what RFC5495 adds; AFAICT all the information needed is in
RFC3209

        leaf hello-misses {
          type uint32 {
            range "1..10";
Why a 'uint32' for a range 1..10?

          reference
            "RFC3209: RSVP-TE: Extensions to RSVP for LSP Tunnels.
             RFC5495: Description of the Resource Reservation Protocol -
ditto

        leaf refresh-misses {
          type uint32;
          default "9";
Where does that value come from?  RFC2205 or RFC2961 would appear to say
three.

        leaf checksum_enabled {
YANG style is usually
        leaf checksum-enabled {

        leaf patherr-state-removal {
          type empty;
As before, why empty?

The referenced RFC3473 has state-removed rather than state-removal which 
I would find clearer.

/rfc4558/RFC4558/
I note that while this RFC is referenced in the body of the text as
relating to rsvp-extended, it does not appear in the YANG module.  The
RFC seems not to change anything by way of retries or timers so I am not
clear why it is applicable.

Overall, it is timers, timeouts and retries I struggle with since I find
it difficult, impossible even, to relate the YANG definitions to those
in the RFC.  I think that all the RSVP RFC should be Normative not
Informative References.

Tom Petch


----- Original Message -----
From: "t petch" <ietfa@btconnect.com>
To: "Tarek Saad" <tsaad=40juniper.net@dmarc.ietf.org>rg>; <teas@ietf.org>rg>;
<teas-chairs@ietf.org>
Sent: Tuesday, August 10, 2021 9:16 AM
Subject: Re: [Teas] I-D Action: draft-ietf-teas-yang-rsvp-16.txt refresh
reduction