Re: [netmod] comments on draft-ietf-netmod-yang-instance-file-format-04

Balázs Lengyel <balazs.lengyel@ericsson.com> Wed, 06 November 2019 21:57 UTC

Return-Path: <balazs.lengyel@ericsson.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 37AD2120145 for <netmod@ietfa.amsl.com>; Wed, 6 Nov 2019 13:57:55 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.002
X-Spam-Level:
X-Spam-Status: No, score=-2.002 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, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-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 TwKhWwdgeuUb for <netmod@ietfa.amsl.com>; Wed, 6 Nov 2019 13:57:52 -0800 (PST)
Received: from EUR02-VE1-obe.outbound.protection.outlook.com (mail-eopbgr20057.outbound.protection.outlook.com [40.107.2.57]) (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 E3A6512002F for <netmod@ietf.org>; Wed, 6 Nov 2019 13:57:51 -0800 (PST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=NePPzTsoFkJUbW79nn4D34emo4qOEiZjiRVEUsVjU3nR5fHpABc6z0b5QMVqVbAUI9E4ucQkkB3k24oHZxJ+x7+EifRnCUm1xWoV2vBv8LJfEGtoIQHdSv+L4v4JCpgSd6x30lkh+BXbwn2dbrTqdnktYSB/sL9/CZ5Qp5BTOafLsg+KMFHAEyu4Q2efJxWYa45C0acEPjYjxzkCfRdVq9gM2SDkA1kfgfYbRC/MI5TzuvPFZoMTru1fC1Ohhccg4qOyM5bf8SCFfhJFoL9aY8PbaALdNSHGFY2l5WQN5/GLLjN8ywbCsdf8Py9skjBN7n7o2zjNrsUc//ZQHvKlWA==
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=Peno5ZjuvJ4H3bRxOnm144wmGGISWH9LkdodxCywry8=; b=Cp388qB2pVFkpKFC3Ph8nOpy0Zx8Nf+nYYtlpX6Z61jK88rEbPKtC30T//zNda8B7W15BvMHDbTMlGZcqDhrDak//vbFbxbG93RFs8yCLjf8esN/4rqtnXqi/UXCDKmjR7bZ6xpK2fOflW6ZdM1Pd/iqICug7z0eqBIL4zlv5cP7770kzWyXzLdrotAIYm9yV89tdDCNtpbIa4g8wpNpabss0cUgTZtnYqbX2m96S2m9Bhfk77YbJ7JDxQWh0GChNXY+wJhLdtcYCbYvfirBQ7CHESyXE1Ww4RmVQXc/ixOIAJxe2MKT6YCA6ISC7PuUF8cVsZS19ALokQdE+WenMQ==
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=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Peno5ZjuvJ4H3bRxOnm144wmGGISWH9LkdodxCywry8=; b=rT2PoHe9JcxbCnYWlL79UZtnoxFkvcs4sLly1F4dnHtY5vJ1pMbWPS2pMNdgkCeAYtu8Rx3t5gz1fB3PBM2IWIkXTm4Ti940EhiAVycOQnzNIiXNZnDNBH15yst7uhZXIRbeMy1naikdCeQL2MX0zaGjHgGAVFxZaXOaQXkKfOM=
Received: from AM7PR07MB6214.eurprd07.prod.outlook.com (10.186.170.77) by AM7PR07MB6310.eurprd07.prod.outlook.com (10.186.168.150) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2430.16; Wed, 6 Nov 2019 21:57:49 +0000
Received: from AM7PR07MB6214.eurprd07.prod.outlook.com ([fe80::3c4a:6fb:4b5a:8a]) by AM7PR07MB6214.eurprd07.prod.outlook.com ([fe80::3c4a:6fb:4b5a:8a%7]) with mapi id 15.20.2430.020; Wed, 6 Nov 2019 21:57:49 +0000
From: =?iso-8859-1?Q?Bal=E1zs_Lengyel?= <balazs.lengyel@ericsson.com>
To: Martin Bjorklund <mbj@tail-f.com>, "netmod@ietf.org" <netmod@ietf.org>
Thread-Topic: [netmod] comments on draft-ietf-netmod-yang-instance-file-format-04
Thread-Index: AQHVf2MbQga78mSle0Opwa3noseNw6d+f8qg
Date: Wed, 6 Nov 2019 21:57:49 +0000
Message-ID: <AM7PR07MB621477C933EE522C6C2EA3B2F0790@AM7PR07MB6214.eurprd07.prod.outlook.com>
References: <20191010.140525.904627955349075516.mbj@tail-f.com>
In-Reply-To: <20191010.140525.904627955349075516.mbj@tail-f.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach: yes
X-MS-TNEF-Correlator:
authentication-results: spf=none (sender IP is ) smtp.mailfrom=balazs.lengyel@ericsson.com;
x-originating-ip: [80.98.34.204]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: dc189876-c82e-4c70-1a2f-08d763045d70
x-ms-traffictypediagnostic: AM7PR07MB6310:
x-microsoft-antispam-prvs: <AM7PR07MB6310A758A6DE73D9A4A6F324F0790@AM7PR07MB6310.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:9508;
x-forefront-prvs: 02135EB356
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(4636009)(136003)(39860400002)(396003)(346002)(376002)(366004)(13464003)(199004)(189003)(486006)(33656002)(99936001)(229853002)(64756008)(66446008)(66556008)(66476007)(66616009)(66946007)(55016002)(2501003)(478600001)(6306002)(9686003)(6436002)(305945005)(14444005)(6246003)(66574012)(256004)(966005)(14454004)(71200400001)(71190400001)(45776006)(7736002)(76116006)(74316002)(476003)(8676002)(25786009)(3846002)(6116002)(186003)(110136005)(446003)(11346002)(102836004)(316002)(2906002)(52536014)(5660300002)(8936002)(76176011)(81156014)(99286004)(7696005)(81166006)(86362001)(6506007)(26005)(53546011)(66066001)(21314003); DIR:OUT; SFP:1101; SCL:1; SRVR:AM7PR07MB6310; H:AM7PR07MB6214.eurprd07.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1;
received-spf: None (protection.outlook.com: ericsson.com does not designate permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: dNycjeO2DD4EvH0YVWeuTr2Z2u8pbL76zTJFjtcOf6eMcm75qAXA5V3gjNEVtO5NnaxWlT04hKSKjOoouiyIbOjUuC7B6xcZUW3vZFhVJ6mE7sqG/vgYwGKkv3rmZ7c9CkYny9vwVeW52EKJiiBgkei4NKC3od98hwWAS5GwK6Zd+9F8/9xp7yIIeta/QddQbwWasMqBsqZJqMwT0sZqLWdcbx+fhE5AeZTl+JjlAHzsc91GdvfP6NeNQNUgOLlTMRvcQ5z3Cc5pd+O9S4xptpUZprvY3/szsAYc5miWDEev7YyE2lo6Fl04zmpIGhro/FhWlSHl6HOQ2jHkIjdWAGAM+jyzKKY7Z79oYSjuTBofAOcNIV9s7+cc3Y6iihVcpr3DzEp1RLSXdpTODjb+m6aRH9QQ5sFCZJyCExyTQNvXOKNzNIOYf/2ZKpqgafX4
x-ms-exchange-transport-forked: True
Content-Type: multipart/signed; protocol="application/x-pkcs7-signature"; micalg=SHA1; boundary="----=_NextPart_000_091B_01D594F5.9B8AA230"
MIME-Version: 1.0
X-OriginatorOrg: ericsson.com
X-MS-Exchange-CrossTenant-Network-Message-Id: dc189876-c82e-4c70-1a2f-08d763045d70
X-MS-Exchange-CrossTenant-originalarrivaltime: 06 Nov 2019 21:57:49.3656 (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: mRi4mnuwL5Rzoun1jYHsbjo13IGzgIjRJE31P0ynq4K0Hykb6lirHODICWjtJXXXFksq+n7I3INBQOD5z++KSGe/hjtauBahwul3RNCJINc=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM7PR07MB6310
Archived-At: <https://mailarchive.ietf.org/arch/msg/netmod/8REZYxu6HELtx_j5cTkQ6dQE5RM>
Subject: Re: [netmod] comments on draft-ietf-netmod-yang-instance-file-format-04
X-BeenThere: netmod@ietf.org
X-Mailman-Version: 2.1.29
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: Wed, 06 Nov 2019 21:57:55 -0000

Hello Martin,
I will update the draft following most of your comments. See details below.
Regards Balazs

-----Original Message-----
From: netmod <netmod-bounces@ietf.org>; On Behalf Of Martin Bjorklund
Sent: 2019. október 10., csütörtök 14:05
To: netmod@ietf.org
Subject: [netmod] comments on draft-ietf-netmod-yang-instance-file-format-04

Hi,

I have some mostly cosmetic comments on this draft.
  o  "YANG" should be spelled "YANG".  Not Yang etc.
BALAZS: OK
  o  "NETCONF" should be spelled "NETCONF".
BALAZS: OK
  o  leaf-list module
    The type of this leaf-list is a string with:
      pattern '.+@\d{4}-\d{2}-\d{2}\.yang';;
    I think the revision needs to be optional, and the suffix ".yang"
    dropped, since it doesn't add any value:
      pattern '.+(@\d{4}-\d{2}-\d{2})?';
   (same for inline-spec).
BALAZS: I disagree, IMHO we need the revision date. We want to know the 
exact version the data was produced against. If the version would be 
unknown it might become very hard to understand whether the instance 
data is correct or not.

  o  schema-uri
    The description says:
          A reference to another YANG instance data file.
          This instance data file will use the same set of target
          YANG modules, revisions, supported features and deviations
          as the referenced YANG instance data file.

   I don't understand what this means.  Does it mean that the schema
   for this document is the same as the schema defined in the
   schema-uri file, or that the schema-uri file defines the schema in
   its content-data?

   I *think* it is the former.  In either case, the name of the leaf
   can perhaps be changed to reflect the semantics, rather than the
   syntax (i.e., don't call it xxx-uri just b/c its type is an uri).
   Perhaps 'same-schema-as-file'.
BALAZS:  OK, I changed the description hope it is easier to understand now.
            description
              "A reference to another YANG instance data file.
               This instance data file will use the same
               content schema as the referenced file.";


  o  Data node naming.
    The current structure of the model is:
        +--rw (content-schema-spec)?
        |  +--:(simplified-inline)
        |     +--rw module*                 string
        |  +--:(inline)
        |  |  +--rw inline-spec*            string
        |  |  +--rw inline-content-schema   <anydata>
        |  +--:(uri)
        |     +--rw schema-uri?           inet:uri
        ...
        +--rw content-data?         <anydata>


    To make the instance document more understandable, I suggest the
    following structure, which adds a wrapping container for the
    schema, and renames the inline and uri nodes:

        +--rw content-schema
           +--rw (content-schema-spec)?
           |  +--:(simplified-inline)
           |     +--rw module*                 string
           |  +--:(inline)
           |  |  +--rw inline-module*          string
           |  |  +--rw inline-schema           <anydata>
           |  +--:(uri)
           |     +--rw same-schema-as-file?    inet:uri
        ...
        +--rw content-data?         <anydata>
BALAZS: OK, accepted

  o  Format the YANG module
    I suggest you run the YANG module through:
      pyang -f yang --keep-comments --yang-line-length 69
BALAZS: OK (I will do it, but I don't agree with a number of its formatting
changes.

  o  3.2
    The element "<netconf-state>" needs a namespace declaration.
BALAZS: OK

/martin

_______________________________________________
netmod mailing list
netmod@ietf.org
https://www.ietf.org/mailman/listinfo/netmod