Re: [netmod] [OPSAWG] Minor change in ietf-access-control-list@2018-02-02.yang

Jan Lindblad <janl@tail-f.com> Fri, 09 February 2018 09:21 UTC

Return-Path: <janl@tail-f.com>
X-Original-To: netmod@ietfa.amsl.com
Delivered-To: netmod@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 6340B126D0C for <netmod@ietfa.amsl.com>; Fri, 9 Feb 2018 01:21:04 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.911
X-Spam-Level:
X-Spam-Status: No, score=-1.911 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01] autolearn=ham autolearn_force=no
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 7Vjc4h5iwqYE for <netmod@ietfa.amsl.com>; Fri, 9 Feb 2018 01:21:03 -0800 (PST)
Received: from mail.tail-f.com (mail.tail-f.com [46.21.102.45]) by ietfa.amsl.com (Postfix) with ESMTP id F374412422F for <netmod@ietf.org>; Fri, 9 Feb 2018 01:21:02 -0800 (PST)
Received: from [10.147.40.109] (unknown [173.38.220.33]) by mail.tail-f.com (Postfix) with ESMTPSA id 0DA4F1AE02C9; Fri, 9 Feb 2018 10:21:00 +0100 (CET)
Content-Type: text/plain; charset=us-ascii
Mime-Version: 1.0 (Mac OS X Mail 11.2 \(3445.5.20\))
From: Jan Lindblad <janl@tail-f.com>
In-Reply-To: <1f32428b-6751-8303-9fc7-ac2c2a92990a@cisco.com>
Date: Fri, 9 Feb 2018 10:20:59 +0100
Cc: netmod WG <netmod@ietf.org>
Content-Transfer-Encoding: quoted-printable
Message-Id: <DC46EA99-94E2-4512-82FA-74683083DFC9@tail-f.com>
References: <CAHiu4JMC6wDc16-Ronz7Ou0Ne9md7acAKJMpU0PN=Bub76tTPQ@mail.gmail.com> <1f32428b-6751-8303-9fc7-ac2c2a92990a@cisco.com>
To: Eliot Lear <lear@cisco.com>
X-Mailer: Apple Mail (2.3445.5.20)
Archived-At: <https://mailarchive.ietf.org/arch/msg/netmod/7AsTcUGaHxm1U85-tF8_h9zxA0U>
Subject: Re: [netmod] [OPSAWG] Minor change in ietf-access-control-list@2018-02-02.yang
X-BeenThere: netmod@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: NETMOD WG list <netmod.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/netmod>, <mailto:netmod-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/netmod/>
List-Post: <mailto:netmod@ietf.org>
List-Help: <mailto:netmod-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/netmod>, <mailto:netmod-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 09 Feb 2018 09:21:04 -0000

Eliot,

> In order to compile the published YANG model with OpenDaylight Yangtools I had to make the following change ( diff published file vs. changed file is below ):
> 
> 583c583
> <                 path "../../../../../../acl/name";
> ---
> >                 path "/access-lists/acl/name";
> 597c597
> <                   path "../../../../../../../acl/aces/ace/name";
> ---
> >                   path "/access-lists/acl/aces/ace/name";
> 
> 
> I am not sure (don't have enough YANG experience) to know if the error is with Yangtools or with the published YANG model but I thought I'd send this information to the list just in case.
> 
> Thank you for your attention.

Both the old and the new path look valid to me. Even if you can always replace a relative path with an absolute from a YANG validity perspective, changing from relative to absolute paths often *changes the semantics*, so that is not generally safe. In this case, however, they do amount to the same thing (since they both end up going all the way up to the top level container).

/jan