IETF Logo Mail Archive

Re: [yang-doctors] Review of draft-wu-l3sm-rfc8049bis?

Jan Lindblad <janl@tail-f.com> Thu, 05 October 2017 13:46 UTC

Return-Path: <janl@tail-f.com>
X-Original-To: yang-doctors@ietfa.amsl.com
Delivered-To: yang-doctors@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 6926812895E; Thu, 5 Oct 2017 06:46:02 -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, SPF_PASS=-0.001] 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 pWMxADDXgjK9; Thu, 5 Oct 2017 06:46:01 -0700 (PDT)
Received: from mail.tail-f.com (mail.tail-f.com [46.21.102.45]) by ietfa.amsl.com (Postfix) with ESMTP id CA91613457B; Thu, 5 Oct 2017 06:45:23 -0700 (PDT)
Received: from [10.147.40.220] (unknown [173.38.220.62]) by mail.tail-f.com (Postfix) with ESMTPSA id 9E38D1AE018C; Thu, 5 Oct 2017 15:44:52 +0200 (CEST)
Content-Type: text/plain; charset="us-ascii"
Mime-Version: 1.0 (Mac OS X Mail 10.3 \(3273\))
From: Jan Lindblad <janl@tail-f.com>
In-Reply-To: <196ce82f-2bc3-18c5-4232-2a874efb5062@labn.net>
Date: Thu, 05 Oct 2017 15:44:24 +0200
Cc: YANG Doctors <yang-doctors@ietf.org>, draft-wu-l3sm-rfc8049bis.all@ietf.org
Content-Transfer-Encoding: quoted-printable
Message-Id: <50B59D11-E865-4025-AAFA-7E2505B540B9@tail-f.com>
References: <262fdd36-5213-f716-016a-02442c427a0a@labn.net> <EA5D631C-818D-4F84-83C9-26B8A2D73A70@tail-f.com> <196ce82f-2bc3-18c5-4232-2a874efb5062@labn.net>
To: Lou Berger <lberger@labn.net>
X-Mailer: Apple Mail (2.3273)
Archived-At: <https://mailarchive.ietf.org/arch/msg/yang-doctors/oGgtG8wL0xECzkocBOMzqWvNCvA>
Subject: Re: [yang-doctors] Review of draft-wu-l3sm-rfc8049bis?
X-BeenThere: yang-doctors@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: Email list of the yang-doctors directorate <yang-doctors.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/yang-doctors>, <mailto:yang-doctors-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/yang-doctors/>
List-Post: <mailto:yang-doctors@ietf.org>
List-Help: <mailto:yang-doctors-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/yang-doctors>, <mailto:yang-doctors-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 05 Oct 2017 13:46:02 -0000

Lou,

> What about the type changes (int32->int64,

This is a backwards compatible change according to both 6020 and 7950.

> int->string)?

Even if a type change int->str might qualify as "backwards compatible" according to 6020 and 7950, the particular change at hand is not because of considerable changes in semantics.

/jan


> On 10/5/2017 9:27 AM, Jan Lindblad wrote:
>> Lou,
>> 
>> I have reviewed the updated model from a YANG perspective. The old model contained syntax errors(!), so there was no way to (correctly) implement it. Hence I believe requirements on backwards compatibility should be treated lightly.
>> 
>> /jan
>> 
>> 
>> 
>>> Hi,
>>> 
>>>    Has there been a review conducted or is there one planned for
>>> draft-wu-l3sm-rfc8049bis?  If not, I think one is needed.  I'm looking
>>> at it now as part of RtgDir review and I believe some non-backwards
>>> compatible changes in the defined yang module.
>>> 
>>> Thanks,
>>> Lou
>>> 
>>> PS for module diffs see:
>>> 
>>> https://tools.ietf.org/rfcdiff?difftype=--hwdiff&url2=draft-wu-l3sm-rfc8049bis-05.txt&url1=rfc8049
>>> 
>>> _______________________________________________
>>> yang-doctors mailing list
>>> yang-doctors@ietf.org
>>> https://www.ietf.org/mailman/listinfo/yang-doctors
>> 
>

v2.23.0 | Report a Bug | By Email