Re: [CCAMP] draft-ietf-ccamp-client-signal-yang references to bandwidth profiles, make it a leafref

Daniele Ceccarelli <daniele.ceccarelli@ericsson.com> Tue, 16 June 2020 07:23 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 700E83A0A50 for <ccamp@ietfa.amsl.com>; Tue, 16 Jun 2020 00:23:59 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.101
X-Spam-Level:
X-Spam-Status: No, score=-2.101 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, HTML_MESSAGE=0.001, 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 NyR1GN1yq2qU for <ccamp@ietfa.amsl.com>; Tue, 16 Jun 2020 00:23:57 -0700 (PDT)
Received: from EUR02-AM5-obe.outbound.protection.outlook.com (mail-eopbgr00080.outbound.protection.outlook.com [40.107.0.80]) (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 74F633A0A1B for <ccamp@ietf.org>; Tue, 16 Jun 2020 00:23:57 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=NkOC5D5SoZJr06B+hqIAdGxkyxiYzWjpAaNSy4nC+Rc3/424D8yC1fkXED4AVlMkbdGA0EXNADwFKjAi4ED+6Cbn4QubSBX9aNI4S/zsAJToyV2qoO+Wmcd8y3rdlB4yPTt7TbbJa5oNCYY8fggvDSwbiAEGm86vOK5A4+rM/NadQ50uNllwp+VQDKDT5hIVzWA2389iKNgJW2g+R/gQIVgLF9l2WcIVkz1aF4JGGez1+Vip4MV3ruFd/LZniHxKiFTSmMBPxZu8qE7XnBksTLS5eMsJboXTzatJIYLzNoq0poIOWQUUsfeNPr//D4o7Bnrx/nGhjuc9x+CYlNm27Q==
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=sOAmlyvnCEad4mj6rkfU+zBZ/UO3Pbxo15iWGcfp+MQ=; b=cAfHVaUCtjXtRXcC9OlsbM7Sdmb9Yedu67KYbdbQNpzEcGKLFr2CQejofl4iji2490IixoOJlYHgVipJJRECYJS7YXoZk1vdbcZLL20V7YHP13Gd9Z9W/2fh5EJScuCCVHsimSuwQD7/rTnVA50DdPtp71rLHJPmIAmAVH07kIwjFVgCmyO20x14noUNj4/GE4SQSzEhR9nL5Y+znEHmQeQnoz28i0WAyRcIOKuF2fsuTkaoQ1ppr5My8wjFz/92i2T57wzkAuP/mPz0DPmTw97iYFioxb9cFVnGaxFJyeQzCg/h+7EuaZf17A42LzYgSGd7Wi+NRMDX7yqWd9KQJA==
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=sOAmlyvnCEad4mj6rkfU+zBZ/UO3Pbxo15iWGcfp+MQ=; b=ICE+NGl5KAgGblOg/r0Ylu3rH3Fw+2XdSOEwaWoIa4/9RXM3z14pTlGrFUGFplpknF2+3HWlAmAFYi9BWCw47VO8fRqtPwbfrO6LZwf8P0rMFCnIbibN7a+crh9VwZSuMssP0UL+EeR1pJwW3yzqIBCmvBUbBrkSS/GkvWPXNDU=
Received: from HE1PR07MB4156.eurprd07.prod.outlook.com (2603:10a6:7:9b::16) by HE1PR07MB4156.eurprd07.prod.outlook.com (2603:10a6:7:9b::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3109.7; Tue, 16 Jun 2020 07:23:54 +0000
Received: from HE1PR07MB4156.eurprd07.prod.outlook.com ([fe80::7d12:7db0:50be:6400]) by HE1PR07MB4156.eurprd07.prod.outlook.com ([fe80::7d12:7db0:50be:6400%7]) with mapi id 15.20.3109.010; Tue, 16 Jun 2020 07:23:54 +0000
From: Daniele Ceccarelli <daniele.ceccarelli@ericsson.com>
To: Aihua Guo <aihuaguo.ietf@gmail.com>, "Roque Gagliano (rogaglia)" <rogaglia=40cisco.com@dmarc.ietf.org>
CC: "ccamp@ietf.org" <ccamp@ietf.org>
Thread-Topic: [CCAMP] draft-ietf-ccamp-client-signal-yang references to bandwidth profiles, make it a leafref
Thread-Index: AQHWQzap22hcDI5rAEOzSRbUxA96BKjaNFUAgACjOvA=
Date: Tue, 16 Jun 2020 07:23:53 +0000
Message-ID: <HE1PR07MB4156BD8E67291457AFCD00C6F09D0@HE1PR07MB4156.eurprd07.prod.outlook.com>
References: <78319D17-A63B-4349-82C9-C564D25B21BB@cisco.com> <CAFS+G6Tojv3sii7OdyepE+MPk-QmVqUacV-OSVEUD=tNf0yGCQ@mail.gmail.com>
In-Reply-To: <CAFS+G6Tojv3sii7OdyepE+MPk-QmVqUacV-OSVEUD=tNf0yGCQ@mail.gmail.com>
Accept-Language: it-IT, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: gmail.com; dkim=none (message not signed) header.d=none;gmail.com; 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: 8ea64ad7-5ffb-4b33-ea87-08d811c639b5
x-ms-traffictypediagnostic: HE1PR07MB4156:
x-microsoft-antispam-prvs: <HE1PR07MB41568EC5D73C42B425387ABDF09D0@HE1PR07MB4156.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-forefront-prvs: 04362AC73B
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: u/SlAt/q7z8TsaBtPQWbgcCCphotpqHEMymxrHdQcaOb/oBi6af7ddVCN7cQvV4ZRHhG66WZDiw41ioDuyCuO4Svdj5/R8dvRuqD7mi6Ugr4/ve8kwXPwvQkP5FgU64wxb9G/dUd9h2xuKgW5G2vFGm4UPYb/tnS1vz33qtp0EfkTrmgW7cgai1W8xQ5vR0kJvJVtNjS23KHRWq8TURh8ZquayfiHhzhDk2WuXaq/SFQ8vRyqi7155GMj0fDlO3DLgKk5uJxvIq+f7h39nOT6rQNlb47pssehpruGIKr1VYwW4zJ6jiTQAmZx/DeaMGwt28yg791sxVYUo7umsyfpcFoOAgIvnlHfbBW0aOVXYJ0UgHnddeBSujBscAR7HPJOc672VzroPvhZCFNYKvpRA==
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; SFTY:; SFS:(4636009)(39860400002)(376002)(396003)(366004)(136003)(346002)(5660300002)(83380400001)(2906002)(26005)(71200400001)(186003)(4326008)(52536014)(86362001)(9686003)(966005)(8676002)(33656002)(6506007)(7696005)(66446008)(55016002)(66556008)(64756008)(478600001)(66476007)(66946007)(110136005)(166002)(53546011)(316002)(76116006)(8936002)(44832011); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata: iO1Qsg52xOCccYK8Nx7UwM0HMYtf+3QvZBiQw/Uh1xQp5TypxbWyFWkTHwI4p7GyK6tn3JV2VdH3F0C6NWyvCyHU71vXKuWfzNrt4ClOHkHshMnQvDQ/dH5D7yK3WUwh2m5faZg26i3O+5SBKVxkikF52vZE4Py0c+LyP6ng9N81WLv08bMd1bqbQ+wii1somEaClzhy5OxGmXD0UQcvlBsTl3Fj4jGGvbbA9QTYZYOqqVMzJh2Afo38ROa+idJ07fYFOFFxlpt3vV1ybKm1BnWFgIDV5Ar9/ZaPRM+3hpcpQrU3YGxJZl4d+s8qVbPc2nKanqKJvc3gzWHI9LPy3EF17pp5a/2nSMWfZfmLfc2Cf/HXyyFGoyCE7P+X3lIILIF3N2yh32eyZCVinJsLaRuzunb9RcsK27Y38WwQGsvw9NQUdP9wWaqR0vRAnSeM9tAvgIxvJm/6tqc+gm47L9cxQ96/7/T2sKBhk2nYrc4=
x-ms-exchange-transport-forked: True
Content-Type: multipart/alternative; boundary="_000_HE1PR07MB4156BD8E67291457AFCD00C6F09D0HE1PR07MB4156eurp_"
MIME-Version: 1.0
X-OriginatorOrg: ericsson.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 8ea64ad7-5ffb-4b33-ea87-08d811c639b5
X-MS-Exchange-CrossTenant-originalarrivaltime: 16 Jun 2020 07:23:53.9846 (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: I17lw/sw2wPPkSeTKd7XosrqRJbdkTzaUBYedM42DQ4zet2jNf9pWsG3PByhTXmygnx5TGIvSJFdkr6/WL2EDuohjvMkUcm4nCmIovI3cfA=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR07MB4156
Archived-At: <https://mailarchive.ietf.org/arch/msg/ccamp/r1VglY3nfxA6PUBCgj1_dZJYXxg>
Subject: Re: [CCAMP] draft-ietf-ccamp-client-signal-yang references to bandwidth profiles, make it a leafref
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: Tue, 16 Jun 2020 07:23:59 -0000

I agree as well, makes a lot of sense.
Thanks Roque for spotting it.

Daniele

From: CCAMP <ccamp-bounces@ietf.org> On Behalf Of Aihua Guo
Sent: den 15 juni 2020 23:39
To: Roque Gagliano (rogaglia) <rogaglia=40cisco.com@dmarc.ietf.org>
Cc: ccamp@ietf.org
Subject: Re: [CCAMP] draft-ietf-ccamp-client-signal-yang references to bandwidth profiles, make it a leafref

Hi Roque,

I agree with you. It should be a leafref to allow for YANG auto validation. Thanks for the catch.

Aihua

On Mon, Jun 15, 2020 at 1:02 PM Roque Gagliano (rogaglia) <rogaglia=40cisco.com@dmarc.ietf.org<mailto:40cisco.com@dmarc.ietf.org>> wrote:
Hi,

I implemented the draft and I think the bandwidth-profile should be a leaf-ref and not a string (for both directions symmetrical and asymmetrical):

          |  +--rw (direction)?
          |  |  +--:(symmetrical)
          |  |  |  +--rw ingress-egress-bandwidth-profile
          |  |  |     +--rw (style)?
          |  |  |        +--:(named)
          |  |  |        |  +--rw bandwidth-profile-name?
          |  |  |        |          string                   <-- We need a leafref to the definition of the bandwidth-profile
          |  |  |        +--:(value)

In particular, I believe it should be:
type leafref {

                   path "/ethtsvc:etht-svc/ethtsvc:globals/ethtsvc:named-bandwidth-profiles/ethtsvc:bandwidth-profile-name";
               }

Why is this important? As an implementor, I need to understand the relationship between these nodes in order to select the right order of RESTCONF commands.

If the leafref is not present, I would assume that the nodes are not related and I could try to delete a “named-bandwidth-profile” that is actually used in a service. Also, when I am trying to delete both a “named-bandwidth profile” and all its associated services, I would need to delete the service first and the profile later (when there Is not instance that reference it).

For all of this to happen, as both nodes belong to different trees, I need to reference them to make the association explicit for a compiler.

Regards,
Roque


_______________________________________________
CCAMP mailing list
CCAMP@ietf.org<mailto:CCAMP@ietf.org>
https://www.ietf.org/mailman/listinfo/ccamp