[netmod] few more YANG-11 comments

Andy Bierman <andy@yumaworks.com> Sat, 12 March 2016 20:34 UTC

Return-Path: <andy@yumaworks.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 379A612D57E for <netmod@ietfa.amsl.com>; Sat, 12 Mar 2016 12:34:37 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.9
X-Spam-Level:
X-Spam-Status: No, score=-1.9 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, HTML_MESSAGE=0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=yumaworks-com.20150623.gappssmtp.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 YosjZuLVNMwA for <netmod@ietfa.amsl.com>; Sat, 12 Mar 2016 12:34:35 -0800 (PST)
Received: from mail-lb0-x229.google.com (mail-lb0-x229.google.com [IPv6:2a00:1450:4010:c04::229]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 0073712D835 for <netmod@ietf.org>; Sat, 12 Mar 2016 12:34:34 -0800 (PST)
Received: by mail-lb0-x229.google.com with SMTP id bc4so195248996lbc.2 for <netmod@ietf.org>; Sat, 12 Mar 2016 12:34:34 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=yumaworks-com.20150623.gappssmtp.com; s=20150623; h=mime-version:date:message-id:subject:from:to; bh=lmYB09G5lnh+ON+dJ7hOoub5PhtSXjhVVjRFZel3vIE=; b=II3JaEf2h9j7Ww5hGpwoDAUCPNwWZJ16EXAxgZWhBdDf19zBGFbEwqlooO+o7jlzuj 1FIdR3KFp1AlDCrqupiwvJir1MQRdZJvdKyFbKKXzkHkoI7DCYCHIbXaWKgBE3WmgFvS nbsAUO/N/hrFMV9ol3Xoq5sXtOuXGhemtW/rOmQMqR8y7PeOMPsXHYSKX2j+Sgb2t2l9 TIghhMTkT//c1euuPPS0D3bqWWUr8pFPLTMq8Kzn1VpsT5hU2UbxIgZByxzMxHZ3ZrmK cDUcpoNA7/IkUApmJIsnSkQyLq0aXvjyJVq8b3Q9zwOZQWWrvbKAN80OQLKlk99sigTJ oU5g==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:date:message-id:subject:from:to; bh=lmYB09G5lnh+ON+dJ7hOoub5PhtSXjhVVjRFZel3vIE=; b=FeUCkieexPPwTKUZUQDlfwgal1nN6qN3dKpXlrlE3BeRxamYlnRYB73Xd86W2K5qU9 1JR2FCLwxnzei9uMJ0AE9SmcKt6degYhglUSGLCc6GhD2asQ7VFiRl9mDyqah4tejC/L OCV2BBmKHNO7gX1ODomAr6/fdTnEbNa8/wCCpkNqkfqDtNq4hV07RAFu/+8gn2CZTi3h Fw4DEqNLkQkmnES60ci0WAqyVdAo22bVvNH6kJPmPGzFbGK5HpV1n3VNk7LqvK5xrvbR LFstCtFjadYJz0+QH5f6MfeZka3y+ipmHWNL0vGK59phOgD/g1EkYD6EgXcrnbBpQBDE 9dPA==
X-Gm-Message-State: AD7BkJLiKdtiIP1X2DArfdFnN9jYnENh/vCQ8kKtgw1psDi1bZjcgsQlFRRgdh0FBHuRBggOeE90VkxoPPEhkQ==
MIME-Version: 1.0
X-Received: by 10.112.161.131 with SMTP id xs3mr5349530lbb.65.1457814872981; Sat, 12 Mar 2016 12:34:32 -0800 (PST)
Received: by 10.112.132.65 with HTTP; Sat, 12 Mar 2016 12:34:32 -0800 (PST)
Date: Sat, 12 Mar 2016 12:34:32 -0800
Message-ID: <CABCOCHSSPj0UnWzCJiaJmB3LFRqA4WrNtf3APtyS7No23Cm8KA@mail.gmail.com>
From: Andy Bierman <andy@yumaworks.com>
To: "netmod@ietf.org" <netmod@ietf.org>
Content-Type: multipart/alternative; boundary="001a11c31f52a719c1052ddff95b"
Archived-At: <http://mailarchive.ietf.org/arch/msg/netmod/bQ9sAwNABuPsO0_aRqldPaXavPA>
Subject: [netmod] few more YANG-11 comments
X-BeenThere: netmod@ietf.org
X-Mailman-Version: 2.1.17
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: Sat, 12 Mar 2016 20:34:37 -0000

1.1

It does not mention:
  - multiple import-stmt for the same module now allowed
 - description-stmt and reference-stmt now allowed in
    import-stmt and include-stmt

(There are no examples of any of the above changes)


5.1:

   o  A module MUST include all its submodules.


This is not 100% correct.
A module MUST include one revision of all its submodules.
This includes a submodule with all obsolete definitions I guess.


7.1.6

It does not say what happens if a nested submodule includes-by-revision
a different revision that the parent module.

  Multiple revisions of the same submodule MUST NOT be included.


It is not clear that this applies to all include-stmt for the same module


  module A {
    include A1 {
       revision-date 2016-01-01;
    }
    include A2;
    include A3;
  }

  submodule A1 { ... }

  submodule A2 {
     include A1;   // which revision is included if no date? conflict w/
module?
  }

 submodule A3 {
     include A1 {
         revision-date 2015-01-01;   // this is an error, right?
     }
  }

9.12

It doesn't say what happens when a union containing a leafref
or instance-identifier becomes invalid or changes member types.
Is this just a server implementation detail? Is should at least be clear
a valid union leaf/leaf-list applies to a valid datastore
(e.g. running, not candidate).



Andy