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

t petch <ietfa@btconnect.com> Thu, 05 August 2021 11:25 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 A89FB3A0C4A; Thu, 5 Aug 2021 04:25:10 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.901
X-Spam-Level:
X-Spam-Status: No, score=-1.901 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, MSGID_FROM_MTA_HEADER=0.001, NICE_REPLY_A=-0.001, RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=unavailable 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 WNHLOXCti-Tz; Thu, 5 Aug 2021 04:25:04 -0700 (PDT)
Received: from EUR01-DB5-obe.outbound.protection.outlook.com (mail-eopbgr150127.outbound.protection.outlook.com [40.107.15.127]) (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 856C03A0C49; Thu, 5 Aug 2021 04:25:04 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Dbrii1Ba1OoQYZ/m1Xu+Kt27gIZ9gOTSUTJ7u4VGzUeKnKS1eJn5ymUUz5DLv0rg7Pw+Gq0+WA9OyZMI0ClJ+a8ROt5yaCyBLSHKTZ7eRzkLAhryWg9271Ru4el1N51G3CJdFEQLxgUIPCq0ePtWZ/oXzqEtqMsmncsOwsGEfF8DzqsDHCOUIxPvZRTnDPuWf/C7xcIhACSwU5i5QH1xArUmtE6KpZcKddYvOIw8i4vChSh1A2WmqkbtUUpOfMpJiXPWpZr/xtHM2qSqJDaW3zx/sAcDzMSbdcOgFzhoRIFr5BwKCoe2fM+0XYUN2xscNr2bbbNHB1G6mew4rtdtjA==
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=Yb50qLV19RiEkPijgVLd6PqDWYwcOatzDdenvbNPBNY=; b=IY2kDzp3rjcSo9rPEgRhgHJ86q4qXtnM3BAA7tGrRlMee3CpKE1pMT/Di5ynXCnKOjx7wW3aqjFBwDAyL1/pCWK9dwCA70pO2Rub7S+ihfkyFL5MZ10JHOBXD85dggl3vqMk+YnzkS34GE1hjJqosZwLRhHNCT/0uIDCHtmX+NVtIQ9fUzvebZg/iUf4homIKDogn0//PnMGvwLsOgLUcjaQJZu6hI6ifXURL6yJbsKs9AsDqz/MQobfrL7I3aR9KL4+2dHVo2CcgdLIHiS9HFqV6pZNjKCcY5keI18Qqjq3XizK8QF8V6LD3VQd0doFz7iVxCXdt5ASb224Be8/gA==
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=Yb50qLV19RiEkPijgVLd6PqDWYwcOatzDdenvbNPBNY=; b=mO7Nc/6703TCN/nEilv8oCMa7nEFbQAA1+e+YxrEl4eNLjEsH4p/98Z/QV9SBfJNiLfSKDP3GHjXaBIdf/IDr8+I5OzJ8vYbm6dfbaUZpRlzL/2oyO7opzg/jr0/siX2YcHgMAynVamvAlQkimLo+oLPK+iVCgjpHPGj6ahgn5o=
Authentication-Results: dmarc.ietf.org; dkim=none (message not signed) header.d=none;dmarc.ietf.org; dmarc=none action=none header.from=btconnect.com;
Received: from DB7PR07MB5546.eurprd07.prod.outlook.com (2603:10a6:10:73::23) by DB6PR0701MB2789.eurprd07.prod.outlook.com (2603:10a6:4:24::23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4415.6; Thu, 5 Aug 2021 11:25:02 +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.4394.017; Thu, 5 Aug 2021 11:25:02 +0000
To: teas@ietf.org, teas-chairs@ietf.org, Tarek Saad <tsaad=40juniper.net@dmarc.ietf.org>
References: <161376473026.23146.6190954851706936517@ietfa.amsl.com> <BAF1F43B-EF70-452A-8084-DFCD6AF58896@juniper.net> <DB7PR07MB5546E6046169413A6B137126A2F29@DB7PR07MB5546.eurprd07.prod.outlook.com>
From: t petch <ietfa@btconnect.com>
Message-ID: <610BCA8A.7000207@btconnect.com>
Date: Thu, 5 Aug 2021 12:24:58 +0100
User-Agent: Mozilla/5.0 (Windows NT 5.1; rv:38.0) Gecko/20100101 Thunderbird/38.5.0
In-Reply-To: <DB7PR07MB5546E6046169413A6B137126A2F29@DB7PR07MB5546.eurprd07.prod.outlook.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
X-ClientProxiedBy: LO4P123CA0223.GBRP123.PROD.OUTLOOK.COM (2603:10a6:600:1a6::12) 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 LO4P123CA0223.GBRP123.PROD.OUTLOOK.COM (2603:10a6:600:1a6::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.4394.16 via Frontend Transport; Thu, 5 Aug 2021 11:25:01 +0000
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id: 81ec65dd-3c59-4f59-2f27-08d95803aa75
X-MS-TrafficTypeDiagnostic: DB6PR0701MB2789:
X-Microsoft-Antispam-PRVS: <DB6PR0701MB27890E08F24CAE6107EB8681A2F29@DB6PR0701MB2789.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: jaiHXENQVkOtA7lsmbveE2X4m4HD7LLD4t76Zl3ixxqAQ02wlxeYdrUxm6vG1KvUM6UjaNflJTivwNWBafPyEwug5/tgNWiC/omPf9G1lK6t5Ur8kPG6slsJ51UTwsjzkD9UWWHUxWxNJbP+yHPm91y5NN5BJar8YvP6XL7dmyrfFNucW9WqCM/RvFL9Jhh7VUL7zsEjlPFZx3nmInyKvpqUNKe8kbI47h83uUbV41bD4wveaXfm9seav6gfo7Hrznbp0uZ1cM6KmN6k9E50iOnr0O2Spyw4wRaKKhV6WQbb7jTdEfjp0vSdd0h54yMRsr6N4NNDpUvLMaqeMcBNAH9lyb3YDO757ssBRFWY9U+eRC6XuY6NGm0uF+WKPvsYPu/s2eZ9baJ/JUPxasg+5se2enmFvF5D1zZjJuv6hiLgTvtyTRnocVpzoBNl1SXQCdbQMpTLbPOMy0l7C1CaWzFRa/dVxEhcXN9/jvH2mObuwrmMsb///z30u22iwvu71O/PbvJhZwydibeypIYWm+DR+9vzq51Sks4tQnEpBOiNagHIbX4p6Kc4kOCIgx4dfZQHmMzzaq4NnyQXVS1YowrZwT+sfvGNKUJ77W/It8XDbeta4uxHzXXm4h4IVz9w1snQFnfZhcJpKO9qaRJlTLzPHsMI3ZK3eOQzLp0ZWgdS8wL5es7ooNxwb+x5/fiSD8Ntv9VPu0RHicwwR8ND/Gek8DElmA3XejCqRRwnynk9dCPwj/ri812s1Lv7YtiB/mGC1y9fd/dSi5KkRzrHS1MAJqreSYkbXO2Pv+yhTgRh0xDPS7dUwaD3dcGk6izT
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)(376002)(136003)(39860400002)(346002)(396003)(6486002)(956004)(66946007)(8936002)(2906002)(66556008)(2616005)(83380400001)(8676002)(26005)(53546011)(38100700002)(38350700002)(66476007)(5660300002)(86362001)(87266011)(66574015)(186003)(6666004)(52116002)(966005)(478600001)(33656002)(36756003)(16576012)(316002); DIR:OUT; SFP:1102;
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?ZjljMzcxVlk4bHo3MnozdmtuYTdDUWQvNDFhWEJha2lrVDBpN3JqU2ZwOElX?= =?utf-8?B?SGxYSGxNTTlUenlhTU9lV2lxV3ZDUnh2WWhMdklrWlZoN0Z2dFRBalM0RTdI?= =?utf-8?B?d2lhYnpzRlEyVXhLZUhjb1ViVDNxSnlyd1pEVEtxSjN0NStINVUxNWNBMUFp?= =?utf-8?B?MmkzVEdwYUlPZXpXYzd0ZXhKWU50K1lQQ3pwUmtHaTJGcjVQanZ6Z25paGF0?= =?utf-8?B?b2lScStUaEsrenFjUlVOcVI4VDZMcHh6d0RTOVNKaXF6ZFZVVmF3ZDhWOXhV?= =?utf-8?B?R0FyVjExdCtLSTR0dGlzeEdOUktuMjB5ZEQ3TzNRRXUzaEYvRWlZWGlOUlBI?= =?utf-8?B?T2t4ZEhXT1JpNTE4VXBuUmY5aXdzTEVjNytSQ0JtK0ZEd0xXRlJtMXVMQ0cw?= =?utf-8?B?dVYvM00xRWdFR3VUOEdzN0Jwdk9TM0l1V2VnOExOMktoVUhwTllpVFhncGZN?= =?utf-8?B?UVlyMGJRSmthZjh0dVZDNS95c01KVlJQR0M0Rlg3SkwxUjN4WnF5VklLZnlS?= =?utf-8?B?TDlaRnQ2ZlJ4cG1KeU5jSlZaZmRIK0Nlem5mT1RpWXRvZTBybEFGUktITGhO?= =?utf-8?B?M0wzc0VrM283bUM5VStFRHdNcy9pbmdVRVkvazU5dCtPNVE1WTRPRTMwemxP?= =?utf-8?B?b3hLYkpRRVZtei90WnlFL1pxTDlnOXFDam5jQVVabmJ2YXBGWEFLVkVjaFd0?= =?utf-8?B?T3FJYVAvQ3lWNENMK0Z6a3Vhc0hkcmF2RXFTRUdzMmdOQk9aQVNpL3gwelVF?= =?utf-8?B?Q09FaFlTeE42N3pONnVvR05LTWVXQnJFYjNDdGVIeFdVRkI0WnlUWnBMNURK?= =?utf-8?B?aCtIcWc5WW9TeXhQN2NtbVNKM1dheWhidndDYUxXbjQwSmMrTWc2TjNQT2No?= =?utf-8?B?eDMwbDY3Zm4ydkRhbGJHV3p6aDU1STd5YjlRVVZvVC9zUXY4Rkp4TDBTRXY5?= =?utf-8?B?OW5NYm5qR2lkaEk0TDhFckFUNDR2MkdTeWRDNEtnN0FpWjdFT01ReE1Ob0VP?= =?utf-8?B?cDJZbnNSVFp1MmJQRzRCeDAwYmVtNCtmTzRwQXhxbk5PeTJmSEVvdklzcDZm?= =?utf-8?B?R01UM1FZZDRkVTRpL0tRc0pmU1VyOTFzb0MyVHFnRU5lamZRTFBCaG5YblJn?= =?utf-8?B?WnVLWjZxbXZuWm5saXBsc3hXMVdVVFpGSzd4Q1dXdUpJaHFSSVQvcytUbUdV?= =?utf-8?B?L3VWRUVWQll5eVBrQ1VoTzJ3aGRYeXBsQzhZSXpLUDZQaXpPZVRKMzA0ZFYr?= =?utf-8?B?eExBR3gxSGJBaStxc1F5OUdROFp2YmtDNDBNN241SVpOVEtxblZHdHN5Zit6?= =?utf-8?B?bjA2YXVNVDlFYU9jSGtPdkUrbmRKaytSZEY0VUdqWjVyMk1xRmRkb25zNWtS?= =?utf-8?B?Um1BNm1oVmdZdmtzdE9yTHZ3Wm1PZ2RjNFQ0ekNEeERJWlJlNlR2enBUbW9I?= =?utf-8?B?Ujd2WkN5RUErYWJGZmxEZFQ4WE5YS2psTU9RKzJ3eFlzYkV4OVRrWmxMaXFX?= =?utf-8?B?bGdFK3MvRzVTQ0p2S09mYUp5eWRuR2VvSi91ZEZZRHQ5aDVvMEFqY2lwTVBI?= =?utf-8?B?WHVBV1hnWkF1Wng1d0VxM1ozem44TG93M0hsQ0xXRjZuTm16anZRYUFPSkx6?= =?utf-8?B?US9oMEZmTWFTQkJiU25sRnJKZUdJdjNOUDZWRzZxWG9DSWU1NklWNnBLUStD?= =?utf-8?B?M29SWFBkdWhYMlJlZG5lSEViNnVocTJoQ0FjVGJqTlNJVmxpbnNJcExtNkN6?= =?utf-8?Q?lONSo12eHxC3x6zDi+3cPh7x1NzoWDT5go30PMF?=
X-OriginatorOrg: btconnect.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 81ec65dd-3c59-4f59-2f27-08d95803aa75
X-MS-Exchange-CrossTenant-AuthSource: DB7PR07MB5546.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Aug 2021 11:25:01.8987 (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: 9XXMy4uH6Tzpx3VXMOycI+CeMV+rRIH2axuRHNEaBUIOJPh9A2DmeLHRPy7u9Sj+Ukk20JI5eFiyo6/HSttPxQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB6PR0701MB2789
Archived-At: <https://mailarchive.ietf.org/arch/msg/teas/oOjK6WBHQPcPa9t32KHm0Biq-Vs>
Subject: Re: [Teas] Fw: I-D Action: draft-ietf-teas-yang-rsvp-16.txt
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: Thu, 05 Aug 2021 11:25:11 -0000

> From: Teas <teas-bounces@ietf.org> on behalf of Tarek Saad <tsaad=40juniper.net@dmarc.ietf.org>
> Sent: 19 February 2021 20:11
> To: teas@ietf.org; teas-chairs@ietf.org
> Subject: Re: [Teas] I-D Action: draft-ietf-teas-yang-rsvp-16.txt
>
> Hi WG,
>
> This revision includes editorial updates as well as the addition of "Appendix A" for example instance data tree using this model.
> With these changes, the authors believe I-D is ready to progress to WGLC.

Mmm I think not.

I cannot make sense of Graceful Restart and while this is in part my
ignorance, in part it is not.

RFC3473 says that Restart and Recovery times are in milliseconds and
that zero and FFFFFFFF have a special meaning.

YANG local/neighbor restart/recovery all have units seconds default 120
although the YANG description of recovery says it is milliseconds.  zero 
gets a mention, FFFFFFFF does not.

I assume that the neighbor values may come from a Restart_Cap and so was
thinking that they should be config false but note that the RFC3473 says
that they may be configured.  This needs describing, probably needs
modelling, two separate pairs of objects perhaps.

Helper I do not understand.  The term is not present in the referenced
RFC5063 (nor in RFC3473 or RFC5495).  You need a better reference for it.
Lacking that I cannot check what units should be but by analogy with
restart/recovery I might expect milliseconds again and wonder about 
recommended defaults which the RFC have for some objects.

Where do the helper values come from?  What object of what message when?

Separately
       leaf destination-port {
         type inet:ip-address;
looks odd

I have yet to review much of this I-D having gotten bogged down on
Graceful Restart so expect to have more comments

Tom Petch

----- Original Message -----
From: "Tarek Saad" <tsaad@juniper.net>
To: <tom petch>; <teas@ietf.org>
Sent: Wednesday, February 24, 2021 3:47 PM

>
> On 2/19/21, 3:00 PM, "Teas on behalf of internet-drafts@ietf.org" <teas-bounces@ietf.org on behalf of internet-drafts@ietf.org> wrote:
>
>
>      A New Internet-Draft is available from the on-line Internet-Drafts directories.
>      This draft is a work item of the Traffic Engineering Architecture and Signaling WG of the IETF.
>
>              Title           : A YANG Data Model for Resource Reservation Protocol (RSVP)
>              Authors         : Vishnu Pavan Beeram
>                                Tarek Saad
>                                Rakesh Gandhi
>                                Xufeng Liu
>                                Igor Bryskin
>      Filename        : draft-ietf-teas-yang-rsvp-16.txt
>      Pages           : 57
>      Date            : 2021-02-19
>
>      Abstract:
>         This document defines a YANG data model for the configuration and
>         management of the RSVP protocol.  The YANG data model covers the
>         building blocks that may be augmented by other RSVP extension data
>         models such as RSVP Traffic-Engineering (RSVP-TE).  It is divided
>         into two modules that cover the basic and extended RSVP features.
>
>
>      The IETF datatracker status page for this draft is:
>      https://datatracker.ietf.org/doc/draft-ietf-teas-yang-rsvp/
>
>      There are also htmlized versions available at:
>      https://tools.ietf.org/html/draft-ietf-teas-yang-rsvp-16
>      https://datatracker.ietf.org/doc/html/draft-ietf-teas-yang-rsvp-16
>
>      A diff from the previous version is available at:
>      https://www.ietf.org/rfcdiff?url2=draft-ietf-teas-yang-rsvp-16
>
>
>      Please note that it may take a couple of minutes from the time of submission
>      until the htmlized version and diff are available at tools.ietf.org.
>
>      Internet-Drafts are also available by anonymous FTP at:
>      ftp://ftp.ietf.org/internet-drafts/
>
>
>      _______________________________________________
>      Teas mailing list
>      Teas@ietf.org
>      https://www.ietf.org/mailman/listinfo/teas
>
>
> Juniper Business Use Only
> _______________________________________________
> Teas mailing list
> Teas@ietf.org
> https://www.ietf.org/mailman/listinfo/teas
>