Re: [OPSAWG] Yangdoctors early review of draft-ietf-opsawg-vpn-common-02

"Joe Clarke (jclarke)" <jclarke@cisco.com> Thu, 14 January 2021 14:15 UTC

Return-Path: <jclarke@cisco.com>
X-Original-To: opsawg@ietfa.amsl.com
Delivered-To: opsawg@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 92F5B3A150B; Thu, 14 Jan 2021 06:15:44 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -9.62
X-Spam-Level:
X-Spam-Status: No, score=-9.62 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, 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=kWNNRCcG; dkim=pass (1024-bit key) header.d=cisco.onmicrosoft.com header.b=gOZZrv4l
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 CKL1fmAY3TLZ; Thu, 14 Jan 2021 06:15:42 -0800 (PST)
Received: from alln-iport-4.cisco.com (alln-iport-4.cisco.com [173.37.142.91]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 124663A142E; Thu, 14 Jan 2021 06:15:42 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=3818; q=dns/txt; s=iport; t=1610633742; x=1611843342; h=from:to:cc:subject:date:message-id:references: content-transfer-encoding:mime-version; bh=EVnb2oTVzkLWPMBzqMQSRFrDJ6qi02fu+fl6DB1PGb8=; b=kWNNRCcG7p6JAavrArD2WGtblszT+0HUXMVV/BvDjrawVvRtqUQZFBBs 1MW5skh/uh8+LX+YY0HsP2iS/l20Hi8i5Ixnxp2+MwvmHljKvRweCbu9f J7pDDp/EmpumPVVbUa3Gyn0Rwdv0k12u89juQzL1xF49w3qRHHbl82dJ+ Q=;
X-IPAS-Result: =?us-ascii?q?A0COAgCGTwBgmIMNJK1igQmBT4FTUYFYLy+IBwOOBAOZE?= =?us-ascii?q?oEugSUDVAsBAQENAQEtAgQBAYRKAoFtAiU1CA4CAwEBAQMCAwEBAQEFAQEBA?= =?us-ascii?q?gEGBBQBAQEBAQEBAYY2DIV0AQVAAQE3AQ8CAQgOCi4yJQIEAQ0NE4MLglYDL?= =?us-ascii?q?gGkaAKKJXSBNIMFAQEGhS0YghEJgTiCdYo+JhuBQT+BVIIhNT6EPgKDSoIKI?= =?us-ascii?q?oFZaRtPFywyUU8fGQcPYI9NiwGcRIEICoJ3kCyLXaJmlBicd4RSAgQCBAUCD?= =?us-ascii?q?gEBBoFYAzOBWXAVgyRQFwINjjIJCYNOilh0NwIGAQkBAQMJfI0sAQE?=
IronPort-PHdr: =?us-ascii?q?9a23=3Ak2nDjR1UCk33zPlbsmDT+zVfbzU7u7jyIg8e44?= =?us-ascii?q?YmjLQLaKm44pD+JxWGuadmgUTPUJmd4OAXw+bVsqW1X2sG7N7BtX0Za5VDWl?= =?us-ascii?q?cDjtlehA0vBsOJSCiZZP7nZiA3BoJOAVli+XzoOkVOHsviIVbA8TW+6DcIEU?= =?us-ascii?q?D5Mgx4bu3+Bo/ViZGx0Oa/s53eaglFnnyze7R3eR63tg7W8MIRhNhv?=
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-AV: E=Sophos;i="5.79,347,1602547200"; d="scan'208";a="628218042"
Received: from alln-core-1.cisco.com ([173.36.13.131]) by alln-iport-4.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 14 Jan 2021 14:15:40 +0000
Received: from XCH-ALN-004.cisco.com (xch-aln-004.cisco.com [173.36.7.14]) by alln-core-1.cisco.com (8.15.2/8.15.2) with ESMTPS id 10EEFe65011469 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Thu, 14 Jan 2021 14:15:40 GMT
Received: from xhs-aln-002.cisco.com (173.37.135.119) by XCH-ALN-004.cisco.com (173.36.7.14) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Thu, 14 Jan 2021 08:15:40 -0600
Received: from xhs-aln-001.cisco.com (173.37.135.118) by xhs-aln-002.cisco.com (173.37.135.119) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Thu, 14 Jan 2021 08:15:40 -0600
Received: from NAM10-DM6-obe.outbound.protection.outlook.com (173.37.151.57) by xhs-aln-001.cisco.com (173.37.135.118) with Microsoft SMTP Server (TLS) id 15.0.1497.2 via Frontend Transport; Thu, 14 Jan 2021 08:15:39 -0600
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=eqW2Jqguz5/Kh2sTBEnWwx0Qk0SiLf6D83IFaXDFzEr8HQ66zUELQsx2kq/qOcIQFlP3KlChEeunKFWf5qEciopAzgm005XJWomp9HN3P/Evruix8D64sa6JurT1B8DeB0fgEgQe4S7RF/oBjIWN6YR5dCpVo7dUfahhdY1gvOtdot+dpDTy/XNbP3THez27sjII399t4R8tC694RwObyRFy5Uu4/Ng+pNYuHYJUsdTx2ZaHnMoxy4YJqduoAsPShm2mgO62/DKecF/7nbgP9FO8lFN9uz3tfJrTYTiHrDjVGF+QhQZ3BklrS2qcOISiEDJQQP8RlrNQgp2EXPLeAQ==
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=Enb42BrOK4qGZUUoWiSlD/PCVFpqTOIiMfYhVWzxyJk=; b=fPKljUN7c/Dxi3KnZQpXRLhvWLJPdNy+vdRWwJqyVCZikvfYvJJ774oiNJ1+2Zh3ozAEdHotlMk24XBtmqdnjQ4JMx0p8Gwi21LrslFcHIA6KfpP+RjARJCbzyqHg1ZNsa8I25R14cjGsCRwt5+2AIo1uQRv2r5xVO8rDTy0uJG+GMK9doAjsawkbXgBoyh22D8FQZvWkgeaMrIvrIlRTwMlEQIBVjl2J5wO/hGUeUedVwKyuaCpR6qaffLyljn5whzbLbX+MeoTLKRHeCDA2gvS7stT+40JukyFb0qZ58j8vj0deywEtOlRUmo0AJKioWX2644HK9xr7MDx55HjKw==
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=Enb42BrOK4qGZUUoWiSlD/PCVFpqTOIiMfYhVWzxyJk=; b=gOZZrv4lDG0NxrsLACteqtHLS2q857YgHxyyTFB7q8f0Z8HaiOXy3oVTmvKGqLJ4j7A34kjuotOUdCOyzJJCbOl+Hp5nMfZvQwdIr4wjH2JOdUtY0ViOinu2aELnnu0itf4bErppeXuxGR9FJWm/7x1baRnnOQ0g6/u0S7i40ck=
Received: from BN6PR11MB1667.namprd11.prod.outlook.com (2603:10b6:405:e::12) by BN8PR11MB3731.namprd11.prod.outlook.com (2603:10b6:408:8b::27) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3742.9; Thu, 14 Jan 2021 14:15:38 +0000
Received: from BN6PR11MB1667.namprd11.prod.outlook.com ([fe80::fd07:517e:fe73:789d]) by BN6PR11MB1667.namprd11.prod.outlook.com ([fe80::fd07:517e:fe73:789d%12]) with mapi id 15.20.3763.011; Thu, 14 Jan 2021 14:15:38 +0000
From: "Joe Clarke (jclarke)" <jclarke@cisco.com>
To: =?iso-8859-2?Q?Radek_Krej=E8=ED?= <rkrejci@cesnet.cz>, "yang-doctors@ietf.org" <yang-doctors@ietf.org>
CC: "draft-ietf-opsawg-vpn-common.all@ietf.org" <draft-ietf-opsawg-vpn-common.all@ietf.org>, "opsawg@ietf.org" <opsawg@ietf.org>
Thread-Topic: Yangdoctors early review of draft-ietf-opsawg-vpn-common-02
Thread-Index: AQHW1TdZeGp0tUxxGU6mT/UQhX/vXw==
Date: Thu, 14 Jan 2021 14:15:38 +0000
Message-ID: <BN6PR11MB16672BF1C625F7F846EA410EB8A80@BN6PR11MB1667.namprd11.prod.outlook.com>
References: <160829366323.14887.4241710682101990781@ietfa.amsl.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: cesnet.cz; dkim=none (message not signed) header.d=none;cesnet.cz; dmarc=none action=none header.from=cisco.com;
x-originating-ip: [2001:420:2481:6:4d53:6c2a:3fdb:8986]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 6f10fbb8-c289-4758-bb57-08d8b896de6e
x-ms-traffictypediagnostic: BN8PR11MB3731:
x-microsoft-antispam-prvs: <BN8PR11MB3731CE9274EAF5F262AE6A5EB8A80@BN8PR11MB3731.namprd11.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:9508;
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: tl/JZZjESTDXYLZnSPjU/CAz6r/Hdoj+0PCPqFVTF3GSh7PDR4XUsMZsBZ/BalBCxIE3rd16mYvu4XAOVzG3FaHh3B8d5HTsTUNiq0ErXfHmHluHqca0mEhu76N/FRdy0cUdhyFKwN9Q3kurZ33NbZTw+ClPU7XIZCjowjyBpvhllEN4uK/+b/IxtMTLAerokLWuhej7hsfugWJGAfZLLz5bskh8qNc9kulPF44CX89IfyHIImHiCNePop14JhDbLYR5SzBQftqllpCm90SGZ2p7b4xxNN9sCdHaQPLoMAG3vGYagyr1B+HD59Yq4zA/pEl7In2gf+aqgJ75TaGa8RptfCpN8BQwt3gXqCjtTW/1SWG68Q9uuOySb35ILHiUu/LqoyMNHHEakJmGJ1BRbA==
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:BN6PR11MB1667.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(136003)(396003)(366004)(376002)(346002)(39860400002)(66476007)(4326008)(8936002)(64756008)(110136005)(186003)(478600001)(316002)(86362001)(9686003)(66556008)(66946007)(6506007)(2906002)(71200400001)(83380400001)(5660300002)(53546011)(76116006)(7696005)(33656002)(8676002)(52536014)(55016002)(66446008)(91956017)(54906003); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata: =?iso-8859-2?Q?TYsPX/4dtztCI91gFa8BgH/p+FqozVy+lcbF+yhZVlpSQRF4l+W69a2HS9?= =?iso-8859-2?Q?6fQ7KcIEzlSO8HtTu67MpVXxTVWTqOjSt1ywLZAuWyrd6nENzmnry4Kwzu?= =?iso-8859-2?Q?foYXH5MN3m3bZQrro4NyLNYk0ckH5stfMk1EpHT1y/j0Iu+7s5FtIbG1Hf?= =?iso-8859-2?Q?GnY5xAdzV1Aae9GlQ7X0QBBs1INFF1rwAcDLiwpBa+e0GrAYH7AkDnyCx0?= =?iso-8859-2?Q?RmyH/jC/wRP5VNJ7pD7T1aHMf+1ost81upVjpfQIhKv1IyqyvuFb/RD6HE?= =?iso-8859-2?Q?wyCmgofr/vtv/sloHXthdMiQTsEN4cW3ySq5oKj9D6s1ljvbynVkTxfHCu?= =?iso-8859-2?Q?pTACmPU9WqOiAU0BWVmcGKtpS+q68HulM2OJJIk6qjjHUusg5/X/oPn6UP?= =?iso-8859-2?Q?poEeEjecXRsnIZaSkXjATAblYtSr9cS1GECdmjXaNSrkpPnAwlQE1yX9cg?= =?iso-8859-2?Q?LQnNg1OqSieF+GbKxbQlqrLOCDBfW+u8+k/9Cp9AEptqdbDTrJrUiflqjU?= =?iso-8859-2?Q?lKNjKaqzrosgXn59/1/qIfHcR34c5GmFF+qqSEcK/iEBIKZf58GVjrfkU9?= =?iso-8859-2?Q?rnkvNdOv7o9GubBxLXgh2Cm7ztSyGM+KQo5hU1YFArYed8xQuNS1RAnOnr?= =?iso-8859-2?Q?Fbwpz8AZ1pbFiYRx5sF10ZILXxXHYm6ifeb492jCNTZKxxm8QtLseIsvHH?= =?iso-8859-2?Q?8LX8FbY5Dh0XK+F6DSh52pZTLlMV8/vxrADg0If+HwK8EfB/gbPWcoI3x3?= =?iso-8859-2?Q?V0YdayJ2iA3tbLyEe3XHtpImEUWJBttg1ZGh8wjYTEjj11wC51srQESjNa?= =?iso-8859-2?Q?Jvlgj8oPMgGHPZ24SvI/r92H4CUm9JPwMg4T5t7kxZ5jAq9srodTAKDqFO?= =?iso-8859-2?Q?8cdgglHb5h+qxho4RbSbILnGoadcJPOB979PvMxR5BOnLiXOUViQRBX7zL?= =?iso-8859-2?Q?rT/WQSfhbxiv3Y8kehclHwEFJSopjqbswSWLR4KoiXtinMSwqR4DTiPSeA?= =?iso-8859-2?Q?D54Sf2JgUzfBbed7fBf5UmM6HYODx0IcPWaECSZsgPMDp5DC3bbgLjcLow?= =?iso-8859-2?Q?u/ROteARGLltHUrapavUr829eHzuzujTkCOlLGS7hcoW?=
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="iso-8859-2"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: BN6PR11MB1667.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 6f10fbb8-c289-4758-bb57-08d8b896de6e
X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Jan 2021 14:15:38.8229 (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: wd2xz33AcLrk49m68dTGyCN6K/9waYjCymXyhRaLMRptBv3lGOoRH9pQDnBuMnKgPgtFn5NmU3zLvEnfsuvFfw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN8PR11MB3731
X-OriginatorOrg: cisco.com
X-Outbound-SMTP-Client: 173.36.7.14, xch-aln-004.cisco.com
X-Outbound-Node: alln-core-1.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/opsawg/kFfefugtCqCRdDyenfaYuIBZ0wk>
Subject: Re: [OPSAWG] Yangdoctors early review of draft-ietf-opsawg-vpn-common-02
X-BeenThere: opsawg@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: OPSA Working Group Mail List <opsawg.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/opsawg>, <mailto:opsawg-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/opsawg/>
List-Post: <mailto:opsawg@ietf.org>
List-Help: <mailto:opsawg-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/opsawg>, <mailto:opsawg-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 14 Jan 2021 14:15:45 -0000

I thought I had sent this out, Radek, but I guess it got missed with the
winter break.

Thank you so much for your review!  This will help to shore up an
important piece of this overall network modeling work.

Joe

On 12/18/20 07:14, Radek Krejčí via Datatracker wrote:
> Reviewer: Radek Krejčí
> Review result: Ready with Issues
>
> *** draft ***
>
> Seems well written with a clear statement about its purpose and describing the
> groupings of the module.
>
> *** module ***
>
> * layout
> Use pyang's `--yang-canonical` option to unify the order of the statements and
> make the order canonical - e.g. starting the groupings with description
> improves readability of the module. Using pyang to print the module would also
> remove probably forgotten comment `//L2xMs`.
>
> * typo
> - Module's description: s/Section 4.c/Section 4/
>
> * features
> Almost none of the features is actually used in the module, which might be
> fine, but there are identities referring to the same areas as the features, so
> I believe that these identities should have if-feature statement referring to
> the appropriate feature defined in the module.
>
> * enumeration typedefs
> Since the enumeration cannot be extended, are you really sure, that the
> enumeration types you define are really complete forever? I would say that
> address families, negotiation modes and  control modes might need extensions in
> the modules that will use those types. Defining it as identityref with
> specified identities instead of enums would solve the problem.
>
> * `service-status` and `status-timestamp` groupings
> Both groupings seem to have config false meaning.  But only the
> service-status/status/oper-status container is defined as config false. The
> uses statement doesn't have its own config statement, so if you want to place
> the mentioned groupings into config true data, an extra grouping or refine will
> be required. The common sense of the groupings is config false, so define them
> this way. If there will be some exception to make them config true, the uses
> can refine it in such an exceptional case.
>
> * grouping rt-rd/rd-choice
> There are 2 cases (pool-assigned and full-autoassigned) which seem to have a
> node with the same meaning, but since there would be a name collision, they
> have different names `rd-assign` and `rd-assigneed`. I don't think that this is
> a good solution. If they have the same meaning, they should have also the same
> name. I see 2 possible solutions: - move the leafs in the cases into a
> container - it adds another level in data, but allows a common name
> `rd-assigned` - move the `rd-assigned` leaf outside the choice. If it makes
> sense, it can be constrained by must/when to the (non)-presence of the nodes
> from the choice. The main point here is that the rd-assigned status is then
> always at the same place.
>
> * grouping group/
> The grouping seems odd to me, like it misses something or it might be just by
> too generic name. When looking into the `placement-constraint` grouping, it
> seems that it refers to the groupings, but since they are separated, it is not
> possible to refer to the group-id explicitly. Is the `group` grouping intended
> to be instantiated standalone, without `placement-constraint`? If not, join
> them together and refer the group-id from target-flavor via leafref.
>
>
>
>