Re: [CCAMP] Yangdoctors last call review of draft-ietf-ccamp-otn-tunnel-model-11

Daniele Ceccarelli <daniele.ceccarelli@ericsson.com> Mon, 19 October 2020 11:12 UTC

Return-Path: <daniele.ceccarelli@ericsson.com>
X-Original-To: ccamp@ietfa.amsl.com
Delivered-To: ccamp@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 04D853A0B18; Mon, 19 Oct 2020 04:12:49 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -0.202
X-Spam-Level:
X-Spam-Status: No, score=-0.202 tagged_above=-999 required=5 tests=[DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=ericsson.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 YjCJM7rpr-zl; Mon, 19 Oct 2020 04:12:47 -0700 (PDT)
Received: from EUR05-DB8-obe.outbound.protection.outlook.com (mail-db8eur05on2046.outbound.protection.outlook.com [40.107.20.46]) (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 4D2233A0B17; Mon, 19 Oct 2020 04:12:47 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=L5Dj/KmwgGd5Pn1NDrT8/51GzDTtT31y3cyzzuE8EKHDAtzwqOCPB/td8X2uKhNE5AQmzGRXe7+xxytzfxYh9l4GTPhPCwdu1PzZMcJURbACT1HLmBpoluf7LprcKRQn+Cj/9yn72Q+PPUFit6cYj2pOCwf+U3cpZFh2TNZek2AUW6C7PXyWBkvZDsoE9vFJvnzOKMpsEgz+uyecOIrnSkVf4xzW5y8b+M63oLFCRkGD0GKi8dW0wXfomDe7xk4OFeQAHim0czVNSvVhGU29Xa00sr71He8hfKXt51CSmKxs//+okFYKugRjgIUq0aJT5xKDWprZQ9qnTyXZqWf75Q==
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=XqRQK7784w1J5TTOCWeXkg8eHE0uTdh4uT7spuF4M7A=; b=eY5wWz6Hg7CO3dSul8Znkv15xk4f1o/ijMoFLMvKF5xg5yUz35FacuGH3tD8QLNHh2gvvE8iFq8wI2la7xzY8xN5mLpEQr9O2ZGQysxSCChv7QUdaWCXUcLenBbkgqWg2Ipz7Izo403zb08qORt77285DTNN7LFPGrx2gdZnhbcEV2cbiClYAmx5rI86lWWtukGMtpHdsBEqeu9MukH4vUQsEbpX6NODfPWS4bXdovooYuIr1r31rw61viYnYUElqdXPRx0dwLt/00hYVYu4DSQ/5uIXB0thfYw5eyG09vlNqoBKXX4ExiW7uG7hM9o0CS5pLDQJNe8N4a4okPdgKA==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=ericsson.com; dmarc=pass action=none header.from=ericsson.com; dkim=pass header.d=ericsson.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ericsson.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=XqRQK7784w1J5TTOCWeXkg8eHE0uTdh4uT7spuF4M7A=; b=CLOH4cqBL2BjYzOVQsnPHjDbZ8ojBecq5Kx9PfWiKfAt0EaIKnf8O+yWC+EtXKG+YcuzXslfDH6A2lPASYb5fzRXh+N3Gj30kPrFd6KslPSY0jJ1FRwd0Jo1WExEITyXgMz9bPCuvvjU5TLRFGeMK2AQ1XcB7IqzRjYZriHIutc=
Received: from HE1PR07MB4156.eurprd07.prod.outlook.com (2603:10a6:7:9b::16) by HE1PR0702MB3819.eurprd07.prod.outlook.com (2603:10a6:7:83::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3499.15; Mon, 19 Oct 2020 11:12:45 +0000
Received: from HE1PR07MB4156.eurprd07.prod.outlook.com ([fe80::8d9b:9ab1:1a37:6e93]) by HE1PR07MB4156.eurprd07.prod.outlook.com ([fe80::8d9b:9ab1:1a37:6e93%7]) with mapi id 15.20.3499.015; Mon, 19 Oct 2020 11:12:44 +0000
From: Daniele Ceccarelli <daniele.ceccarelli@ericsson.com>
To: Radek Krejci <rkrejci@cesnet.cz>, "yang-doctors@ietf.org" <yang-doctors@ietf.org>
CC: "last-call@ietf.org" <last-call@ietf.org>, "ccamp@ietf.org" <ccamp@ietf.org>, "draft-ietf-ccamp-otn-tunnel-model.all@ietf.org" <draft-ietf-ccamp-otn-tunnel-model.all@ietf.org>
Thread-Topic: Yangdoctors last call review of draft-ietf-ccamp-otn-tunnel-model-11
Thread-Index: AQHWo8DzndQFPN0EtUmnQ8WJYvNd36messtggAAV9ICAAADJkA==
Date: Mon, 19 Oct 2020 11:12:44 +0000
Message-ID: <HE1PR07MB41567316D8B91337D2B45D88F01E0@HE1PR07MB4156.eurprd07.prod.outlook.com>
References: <160285520741.1658.15600224799776978218@ietfa.amsl.com> <HE1PR07MB4156B1C2A43AF39C469F75B0F01E0@HE1PR07MB4156.eurprd07.prod.outlook.com> <be08d670-bfaf-ce17-ae8e-87e1da5d8504@cesnet.cz>
In-Reply-To: <be08d670-bfaf-ce17-ae8e-87e1da5d8504@cesnet.cz>
Accept-Language: it-IT, 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=ericsson.com;
x-originating-ip: [93.38.67.165]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 93a6d3c6-7d51-4b00-71ef-08d8741fe783
x-ms-traffictypediagnostic: HE1PR0702MB3819:
x-microsoft-antispam-prvs: <HE1PR0702MB3819F34EA6CFDA068D23DA9EF01E0@HE1PR0702MB3819.eurprd07.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: m1R8+IDZmnyFBuSkxOTHi2JigDUo+s7+oTciptfDM7A7HciGMryLnPqFSUv++97CteOf6U1K02wBrxK5iSpdikRxdhcxKgwiZwFbqT0ALnKgbzyIQkmGl+ijZJqLo1xrlMX7LukAX9E+M5wFOTHLFw3rmEGJWIYLQe5iFlpi4oNTeuTXhNY+XtpQNQDLI9nSVglCCsXXN2DMsn7iJ8FqiWnqQa585ZaEj9wqIv+kEdp547FUzYRfXiwH3v1g3FudOFOoUnB77JbRQmQuB1rvo2lTSZZtDNJz6Cikj89VA5h4S3vG8VnTkKM4cXyA+KV1XQdIE528BysVJWv4GTwnz4XT+IbKDYFlGI8WenketHb9lL/JrMX0vxWr0x/CaF8Awv8feXef8JtozPu8+ruPsw==
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:HE1PR07MB4156.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(346002)(376002)(396003)(39860400002)(366004)(136003)(53546011)(6506007)(8936002)(5660300002)(26005)(186003)(7696005)(478600001)(33656002)(55016002)(52536014)(8676002)(76116006)(2906002)(110136005)(44832011)(4326008)(66946007)(66446008)(66556008)(64756008)(66476007)(9686003)(71200400001)(316002)(86362001)(54906003)(83380400001); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata: 8HIftGKZZfLOycew6JJF4lL9Hor2ZgywW8zCDCdOQakVXvfgzYqf8GPOV9OSPVWGf19wh5TV2kJFG8UwAFH3u8lvYp6kpyIl+6lA0XpF3dGBIc/2UY8+I8fjIatJZCS/tp+YN57fCgUJokXaxOLtnn8MWFpYW8gqePqWEQHQuD5HX7OdK0as45yZkWNQ2GdOVPcd71uM20Yk3LEOKxKt3H3Snv9uzDpq+YEmlrqqOhPCKLW+ZDs70A7+R2MiFPigq0F2uYeNh+a4NA4sfExogoNoiDpsf/nJagXy6OGHwO6UM7Y2+2CVrksPULTJmePxty+91rnqY837WSHriDx5+QUba5DhfnJu016SQQtneyfuCfNuQ7Rh9XRns08+MbNi9qTaDnZVEy9yN6lk2wuzEgxPhk/IJM2o5fPKpTHYFetfkNf9M/0G/R6D1+VxxWIZNk/C9CLRnVgn4ZQdpaxXkZgvo75BSDQ0D+fm0IotQseIIPSnTpMZXxA0ZgENUEG6ZGIFhClRcxjX3Fx4oGwyHHe7ugYdNd+RWDIv+79jPZVo/W9hzwaWA53VkhOtkvj17s7J5Kwxgg8P3r6/h497JG7jhttywjCy2yf/uIQnyKrcURQcr6b8thiVIhzJ45xVxhKcCPG8DCmKK9QnezmYRw==
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: ericsson.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: HE1PR07MB4156.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 93a6d3c6-7d51-4b00-71ef-08d8741fe783
X-MS-Exchange-CrossTenant-originalarrivaltime: 19 Oct 2020 11:12:44.8255 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 92e84ceb-fbfd-47ab-be52-080c6b87953f
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: oWApaoQ+bhzY+Wo+qBJEI6WOUjvuHpzFleNsJB3XO1Q9wdL0o0hf4S+LR/hCU8XwXRrI5rNSmy8TRuKkOn+atqcrTHwAK/KcMK4BmkcdZf8=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR0702MB3819
Archived-At: <https://mailarchive.ietf.org/arch/msg/ccamp/Iyz_OzL5gbv8qRJvbEN4qrv7-UQ>
Subject: Re: [CCAMP] Yangdoctors last call review of draft-ietf-ccamp-otn-tunnel-model-11
X-BeenThere: ccamp@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Discussion list for the CCAMP working group <ccamp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ccamp>, <mailto:ccamp-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ccamp/>
List-Post: <mailto:ccamp@ietf.org>
List-Help: <mailto:ccamp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ccamp>, <mailto:ccamp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 19 Oct 2020 11:12:49 -0000

Hi Radek,

Indeed i see both of them in the mail archive of the working group.
There must be something wrong with my mailbox or I must have deleted by mistake the topology one.

Thanks once again,
Daniele  

-----Original Message-----
From: Radek Krejci <rkrejci@cesnet.cz> 
Sent: den 19 oktober 2020 13:08
To: Daniele Ceccarelli <daniele.ceccarelli@ericsson.com>om>; yang-doctors@ietf.org
Cc: last-call@ietf.org; ccamp@ietf.org; draft-ietf-ccamp-otn-tunnel-model.all@ietf.org
Subject: Re: Yangdoctors last call review of draft-ietf-ccamp-otn-tunnel-model-11

Hi Daniele,

I've reviewed both together, just the reviews were split and uploaded
via datatracker to each of the drafts. Both reviews were sent into the
mailing lists (I see them at least in yang-doctors@ietf.org).

Regards,
Radek

Dne 19. 10. 20 v 11:52 Daniele Ceccarelli napsal(a):
> Hi Radek,
>
> Thanks a lot for the review, much appreciated. 
> We requested a joint review of the OTN tunnel model and the OTN topology model (https://tools.ietf.org/html/draft-ietf-ccamp-otn-topo-yang-11 ).
> I'm just checking if you need a bit more time for the OTN topology model (which is perfectly fine) or if the request for the review of both drafts got lost.
>
> Thanks a lot,
> Daniele  
>
> -----Original Message-----
> From: Radek Krejčí via Datatracker <noreply@ietf.org> 
> Sent: den 16 oktober 2020 15:33
> To: yang-doctors@ietf.org
> Cc: last-call@ietf.org; ccamp@ietf.org; draft-ietf-ccamp-otn-tunnel-model.all@ietf.org
> Subject: Yangdoctors last call review of draft-ietf-ccamp-otn-tunnel-model-11
>
> Reviewer: Radek Krejčí
> Review result: Ready with Issues
>
> This is my yang doctor review of draft draft-ietf-ccamp-otn-tunnel-model-11
> with the ietf-otn-tunnel@2020-03-09 YANG module.
>
> Despite the size of the module, its structure is very simple repeatedly following a pattern of augmenting ietf-te by groupings defined in ietf-layer1-types module (except the single grouping defined in the module itself).
>
> Validation tools report a number of errors in the module. The problem is, that the module references (in augments) /te:te/te:tunnels/te:tunnel/te:p2p-primary-paths/ and /te:te/te:tunnels/te:tunnel/te:p2p-secondary-paths/ which are not present in current ietf-te@2020-07-12 module (the nodes were removed in draft-ietf-teas-yang-te-23, the reference in the Section 10 of the draft is to
> draft-ietf-teas-yang-te-25 but the module is not in line with this revision).
> So the module needs a crucial update. Also please consider if the grouping otn-tunnel-attributes is reusable in other modules. If the reusability is not the concern, I don't see any reason to define it instead of specifying the grouping content directly in the augment.
>
> Regarding the draft, as a reader, I would appreciate a more targeted description in section 3. Instead of just dumping the tree diagram in section 3.2, it would be useful to split it into several areas with some brief descriptions and examples.
>
> The list of paths is introduced in Section 6 as "the subtrees and data nodes and their sensitivity/vulnerability", but I don't see explained/described the mentioned sensitivity/vulnerability of those paths. Besides that, the paths include '..' as ellipsis, but '..' has its usual meaning in paths, please unify it with the format used in ietf-otn-topology and use '...'.
>
>
>