Re: [yang-doctors] Yangdoctors early review of draft-ietf-idr-bgp-model-07

"Acee Lindem (acee)" <acee@cisco.com> Thu, 02 January 2020 21:18 UTC

Return-Path: <acee@cisco.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 B3E50120143; Thu, 2 Jan 2020 13:18:34 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.498
X-Spam-Level:
X-Spam-Status: No, score=-14.498 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_HI=-5, RCVD_IN_MSPIKE_H3=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=cisco.com header.b=MRmwAFHu; dkim=pass (1024-bit key) header.d=cisco.onmicrosoft.com header.b=HfgHPw+E
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 ddanxPHdF7OU; Thu, 2 Jan 2020 13:18:33 -0800 (PST)
Received: from rcdn-iport-9.cisco.com (rcdn-iport-9.cisco.com [173.37.86.80]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id DCDA512004D; Thu, 2 Jan 2020 13:18:32 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=4434; q=dns/txt; s=iport; t=1577999912; x=1579209512; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-id:content-transfer-encoding: mime-version; bh=RCIkOJE1ag6v3HuL28j8f+KS93SNOQyw+m/odAVm8zI=; b=MRmwAFHu6OPnSYNCgGGQ2JpB+O6HUXD3XXXA7x0Gy7COCqaaRmIK1iAh 06199eRAvqokoq3jOpHgdvKl9FsTookauq0x7J0EL9NyXoxJvSuorAOrp 15VNtpMDfp0Bhor4v/s5Ab2u3Lyy8d9WcUREwaQp7W+GNFHUwmwVn6vGg U=;
IronPort-PHdr: =?us-ascii?q?9a23=3A5rguARIX0lKZsSwu79mcpTVXNCE6p7X5OBIU4Z?= =?us-ascii?q?M7irVIN76u5InmIFeCuKd2lFGcW4Ld5roEkOfQv636EU04qZea+DFKa5lQT1?= =?us-ascii?q?kAgMQSkRYnBZuMAkD2BPXrdCc9Ws9FUQwt8g=3D=3D?=
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: =?us-ascii?q?A0C+BQDnXQ5e/5pdJa1lHAEBAQEBBwE?= =?us-ascii?q?BEQEEBAEBgXyBVFAFbFggBAsqhAiDRgOLAYI6mC+CUgNUCQEBAQwBARgLCgI?= =?us-ascii?q?BAYRAAheBUiQ4EwIDDQEBBAEBAQIBBQRthTcMhV8CAQMBARAREQwBASwLAQ8?= =?us-ascii?q?CAQgaAiYCAgIlCxUQAgQBDQUbB4MAAYJGAy4BAgygTwKBOIhhdYEygn4BAQW?= =?us-ascii?q?BSUGCfBiCDAMGgQ4ojBkaggCBEScMFIJMPoJkAQEDAYFJgyYyggoikD2eWwq?= =?us-ascii?q?CNYczikSEIhuaV45SiFKSBAIEAgQFAg4BAQWBaSKBWHAVOyoBgkFQGA2NEoN?= =?us-ascii?q?zhRSFP3SBKItXAiYEA4IUAQE?=
X-IronPort-AV: E=Sophos;i="5.69,388,1571702400"; d="scan'208";a="606664222"
Received: from rcdn-core-3.cisco.com ([173.37.93.154]) by rcdn-iport-9.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 02 Jan 2020 21:18:28 +0000
Received: from XCH-RCD-006.cisco.com (xch-rcd-006.cisco.com [173.37.102.16]) by rcdn-core-3.cisco.com (8.15.2/8.15.2) with ESMTPS id 002LISCb018735 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Thu, 2 Jan 2020 21:18:28 GMT
Received: from xhs-rtp-003.cisco.com (64.101.210.230) by XCH-RCD-006.cisco.com (173.37.102.16) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Thu, 2 Jan 2020 15:18:28 -0600
Received: from xhs-rtp-002.cisco.com (64.101.210.229) by xhs-rtp-003.cisco.com (64.101.210.230) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Thu, 2 Jan 2020 16:18:27 -0500
Received: from NAM12-BN8-obe.outbound.protection.outlook.com (64.101.32.56) by xhs-rtp-002.cisco.com (64.101.210.229) with Microsoft SMTP Server (TLS) id 15.0.1473.3 via Frontend Transport; Thu, 2 Jan 2020 16:18:27 -0500
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=WQ8NuxhTiD74fWEi8KaHkgjeCMul1WUFrFMB77x+60ssd55WV35MDGdmAv6RiHqZk+/MuOG8+3GipuacqTBoeCE+vZ+CvfHhn1o8w5NQqUBu+yq9Zvl2rhCDyJ8Kh/8gVDlJAUZ2wctslGPuOY0bTIcEdNVJdZaMr7YN+18X7A7XsM8vNvG0bBddMmh7zYr0To56XknoAflWThfTYo/SJrQvd6D9l2UWlRL2gCxfG3oi/C5/PdTUGNEiYmmlV6kQRlB97T+30qlG0wjt69LA+zAC+r08pbd4sv2vOvFtx+WFS5O0O7DgPIjw78ulPc3HeME8wi3zBIG5fS2Q3VPWow==
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=RCIkOJE1ag6v3HuL28j8f+KS93SNOQyw+m/odAVm8zI=; b=NhaPRsrZ8H20OBGXabG9yFSHWZaJ8iJlmzdTUK6h64u7QeTFg1J+wYyniCsUEXrCoAldm5tSQ9nhWFcCNYwxBpy3HpbYrqdkrD/x3AsFUPwuyyIBlO5HDSpWzFLpkBtbyLC+zyyjwinJLXycM22YpbMrqnEDMSanYWaUXe1bwx3Qe8hd/q7gvxNYlb+tvGf/oaU+Izui0eGW6ieGc5c0bIL8vsEQxIZvmGbvC2CHKU9M9cCkig3FZ1+FDDghDiPGWVxv32cnupbpfzCaHOiwilfWjiQCyQsgvOdx326DpL/XaDsdxIwqzUC0aqxwx1qPolRtNuxzuwwrx3AKqTxYmg==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=cisco.com; dmarc=pass action=none header.from=cisco.com; dkim=pass header.d=cisco.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cisco.onmicrosoft.com; s=selector2-cisco-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=RCIkOJE1ag6v3HuL28j8f+KS93SNOQyw+m/odAVm8zI=; b=HfgHPw+ExU+jLE2KME5q7h396g2aEwrPPxcP5U25udpxqzi5eXDtGrT5kCgDLFS01LMNIgNF2b7BxJtUofX3YmClZoQpKExa+RT4c5xsxLiY09zgsYGLBF4zYXnVouAS85koRi6Sl9Ww15dHDfjGz8yFbsfNAxxuEP1bdLVZq6s=
Received: from MN2PR11MB4221.namprd11.prod.outlook.com (52.135.38.14) by MN2PR11MB4349.namprd11.prod.outlook.com (10.255.90.17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2602.12; Thu, 2 Jan 2020 21:18:26 +0000
Received: from MN2PR11MB4221.namprd11.prod.outlook.com ([fe80::f065:2931:17b6:7b5d]) by MN2PR11MB4221.namprd11.prod.outlook.com ([fe80::f065:2931:17b6:7b5d%3]) with mapi id 15.20.2581.014; Thu, 2 Jan 2020 21:18:26 +0000
From: "Acee Lindem (acee)" <acee@cisco.com>
To: Andy Bierman <andy@yumaworks.com>, "yang-doctors@ietf.org" <yang-doctors@ietf.org>
CC: "idr@ietf.org" <idr@ietf.org>, "draft-ietf-idr-bgp-model.all@ietf.org" <draft-ietf-idr-bgp-model.all@ietf.org>
Thread-Topic: [yang-doctors] Yangdoctors early review of draft-ietf-idr-bgp-model-07
Thread-Index: AQHVv0o85gO0fPk7v0iiC+7A5IPHoqfXkewA
Date: Thu, 2 Jan 2020 21:18:26 +0000
Message-ID: <168831D6-8542-4762-9400-0D6D632619FC@cisco.com>
References: <157773531468.4577.5049465733732578910@ietfa.amsl.com>
In-Reply-To: <157773531468.4577.5049465733732578910@ietfa.amsl.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: spf=none (sender IP is ) smtp.mailfrom=acee@cisco.com;
x-originating-ip: [2001:420:c0c8:1002::1d5]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 3c0b98ad-b067-41c3-5cc4-08d78fc94e72
x-ms-traffictypediagnostic: MN2PR11MB4349:
x-microsoft-antispam-prvs: <MN2PR11MB4349869068B48E6A45C45151C2200@MN2PR11MB4349.namprd11.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-forefront-prvs: 0270ED2845
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(136003)(376002)(346002)(39860400002)(366004)(396003)(189003)(199004)(4326008)(6512007)(6486002)(33656002)(2906002)(71200400001)(2616005)(8936002)(6506007)(316002)(966005)(81156014)(66556008)(64756008)(66446008)(5660300002)(478600001)(86362001)(36756003)(91956017)(76116006)(66476007)(8676002)(66946007)(81166006)(110136005)(54906003)(186003); DIR:OUT; SFP:1101; SCL:1; SRVR:MN2PR11MB4349; H:MN2PR11MB4221.namprd11.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1;
received-spf: None (protection.outlook.com: cisco.com does not designate permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: iqk4BQ8xi5vloFzkrXFwH8ONnMDX3PyFJbw1u4suNsUIuViKCl262Kty79o4ASgh7B4/zaPlXkwE84q9IsrnZ7YlzbG1kozSfujKkVLi+OlPhX3UCC0s+BfHnXEntZnVk1R67al5csI8vXScSniIopYx2vey79vbUgVSopSPQxG8EWnN6Ft7fA7KrfZC5PU76TdOE1Snf0N6EoDtZZHEhR2a3m6s+JGTZoSKxdt4QVEYTE37yASc2O+UE1jTP64MvXNO4yyws2n7cP9CrWhxfOTldcZNV+0KhSP+jR9O1MfP03o9qkwQsKoQMEWcKC3OoNUfzPl9YZJim7SMrG9F4T05yfAYY+MeuecnuOQFmEoAZHrlydCLQvC5/dBhAOFiC4fWMcneTN53p49WL2TY60TrXKvHdvXOsE412xQbmllIGdzhGp2Jqizewx/z1IIR9MdYkZO3qtIXTwltJ3YO36OpIc9bbcMTwfGPpV2/dok=
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="utf-8"
Content-ID: <10EB9D4CDEAF7344864514E26EB02A2F@namprd11.prod.outlook.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 3c0b98ad-b067-41c3-5cc4-08d78fc94e72
X-MS-Exchange-CrossTenant-originalarrivaltime: 02 Jan 2020 21:18:26.1851 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5ae1af62-9505-4097-a69a-c1553ef7840e
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: M0Nh7onk2xNkQikJsfeNjyL+k9RYoJevr2e28rypYFKIHcQlie+9juaaQicClEvm
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR11MB4349
X-OriginatorOrg: cisco.com
X-Outbound-SMTP-Client: 173.37.102.16, xch-rcd-006.cisco.com
X-Outbound-Node: rcdn-core-3.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/yang-doctors/4wREhJtSh9n-1vQ-RBT0CQ4wviA>
Subject: Re: [yang-doctors] Yangdoctors early review of draft-ietf-idr-bgp-model-07
X-BeenThere: yang-doctors@ietf.org
X-Mailman-Version: 2.1.29
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, 02 Jan 2020 21:18:35 -0000

Hi Andy, et al, 

On 12/30/19, 2:49 PM, "yang-doctors on behalf of Andy Bierman via Datatracker" <yang-doctors-bounces@ietf.org on behalf of noreply@ietf.org> wrote:

    Reviewer: Andy Bierman
    Review result: On the Right Track
    
    
    Major issues:
    
      - The ietf-bgp module uses (legal) YANG 1.1 constructs that are
        not supported by available tools so it cannot be validated.
        The solution options are
        (A) rewrite the module so it does not place any definitions
            in the main module.  Instead place all definitions in
            submodules. Add YANG 1.0 include-stmts as needed so pyang
            can validate the module
        (B) wait until the opensource tools properly support this
            YANG 1.1 usage and resubmit the module at that time
    
      - The modules import ietf-routing-policy. Version used was
        2019-03-04.  This module has a fatal error caused because
        ietf-interface-common has apparently been replaced with
        ietf-if-extensions@2019-11-04
    
          leaf subinterface {
            type leafref {
              path "/if:interfaces/if:interface/if-cmn:encapsulation"
                 + "/if-l3-vlan:dot1q-vlan"
                 + "/if-l3-vlan:outer-tag/if-l3-vlan:vlan-id";
            }
    
        The path expression is wrong. if-cmn:encapsulation is now
        if-ext:encapsulation.  You need to check the XPath everywhere
        when you refactor YANG modules.

I have fixed this in the -08 version. 

https://datatracker.ietf.org/doc/draft-ietf-rtgwg-policy-model/

Now, if only we could get the IETF tools to include the ieee802-dot1q-types.yang model so that ietf-if-l3-vlan.yang parses....

Thanks,
Acee 
    
      - Full review of these modules is not possible at this time without
        proper opensource tools. Processing the YANG statements
        by hand is extremely difficult, given the large amount
        of groupings used which are spread across many modules and
        submodules.
    
    
    Minor Issues
    
      - The "clear" actions need to be specific about the exact set
        of objects that are affected. The value of each object once it
        is cleared needs to be specified in every object that is affected
        by the action.  Impact on operations should be explained in
        each action-stmt
    
      - Some TODO items still remain indicating the YANG definitions
        are temporary and will be replaced by more correct statements.
    
      - Additional reference-stmts would be useful to implementors
        who are familar with the routing RFCs that the YANG objects
        are based on
    
    Editorial Comments
    
      - The implementation complexity appears to be very high
        for both server and client developers.  The interactions between
        subtrees is non-trivial and probably requires more clarifications.
        This needs to be done by implementors, not document reviewers.
    
    
    _______________________________________________
    yang-doctors mailing list
    yang-doctors@ietf.org
    https://www.ietf.org/mailman/listinfo/yang-doctors