Re: [netmod] WHEN statement within mandatory objects doesn't ensure presence of the mandatory object

Michael Rehder <Michael.Rehder@Amdocs.com> Wed, 10 October 2018 18:17 UTC

Return-Path: <Michael.Rehder@amdocs.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 16AEB1252B7 for <netmod@ietfa.amsl.com>; Wed, 10 Oct 2018 11:17:38 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.79
X-Spam-Level:
X-Spam-Status: No, score=-1.79 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, RCVD_IN_DNSWL_NONE=-0.0001, T_DKIM_INVALID=0.01] autolearn=no autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=fail (1024-bit key) reason="fail (body has been altered)" header.d=amdocs.onmicrosoft.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 AL_9gZEyVrn1 for <netmod@ietfa.amsl.com>; Wed, 10 Oct 2018 11:17:36 -0700 (PDT)
Received: from mx3.amdocs.com (ramail2.amdocs.com [193.43.244.136]) (using TLSv1.2 with cipher RC4-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 125C7126DBF for <netmod@ietf.org>; Wed, 10 Oct 2018 11:17:34 -0700 (PDT)
Received: from unknown (HELO ILHFDAGDRFE2.corp.amdocs.com) ([10.224.0.130]) by ilmail02.corp.amdocs.com with ESMTP; 10 Oct 2018 21:17:29 +0300
Received: from ILRNAEXCHCAS02.corp.amdocs.com (10.232.216.232) by ILHFDAGDRFE2 (10.237.240.45) with Microsoft SMTP Server (TLS) id 14.3.399.0; Wed, 10 Oct 2018 21:17:29 +0300
Received: from ILRNAEXCHCAS01.corp.amdocs.com (10.232.216.231) by ILRNAEXCHCAS02.corp.amdocs.com (10.232.216.232) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.1.845.34; Wed, 10 Oct 2018 21:17:28 +0300
Received: from ILRNAEXCHEDGE02.corp.amdocs.com (10.233.34.168) by ILRNAEXCHCAS01.corp.amdocs.com (10.232.216.231) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.1.845.34 via Frontend Transport; Wed, 10 Oct 2018 21:17:28 +0300
Received: from EUR02-HE1-obe.outbound.protection.outlook.com (192.168.34.8) by msgedge.amdocs.com (192.168.34.212) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.1.845.34; Wed, 10 Oct 2018 21:17:28 +0300
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Amdocs.onmicrosoft.com; s=selector1-amdocs-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=4Wr5lemq1ulBiIEfXVH8pOlWwS7bS7aKE+N5f6PgaDk=; b=iXkLNHBW3xQiK4EQnmvDzKEVE2MpmYOjBSujq32RZDizoI/JTJNyVVydLJtK+lGsDwDB3ENN9Nd4e3pb7uOrLx3+xFn7uyWhN2Zn9HV2oCUKiIBfXQH/chjCij5AgibcXnhK0wCYJVVTyIPrNUwUWO7NIglFdt6Gjgfto2tvrC0=
Received: from AM0PR06MB4083.eurprd06.prod.outlook.com (52.133.58.152) by AM0PR06MB4641.eurprd06.prod.outlook.com (20.178.18.214) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1228.23; Wed, 10 Oct 2018 18:17:26 +0000
Received: from AM0PR06MB4083.eurprd06.prod.outlook.com ([fe80::389e:ca21:ccc7:d6b1]) by AM0PR06MB4083.eurprd06.prod.outlook.com ([fe80::389e:ca21:ccc7:d6b1%3]) with mapi id 15.20.1207.024; Wed, 10 Oct 2018 18:17:26 +0000
From: Michael Rehder <Michael.Rehder@Amdocs.com>
To: Robert Wilton <rwilton@cisco.com>, Ladislav Lhotka <lhotka@nic.cz>, "netmod@ietf.org" <netmod@ietf.org>
CC: "Walker, Jason (Jason_Walker2@comcast.com)" <Jason_Walker2@comcast.com>
Thread-Topic: [netmod] WHEN statement within mandatory objects doesn't ensure presence of the mandatory object
Thread-Index: AdRf+I5wQXpIeuYeT8uFcZ4kUETNSAArOyOAAAESuDAAATBXgAAAIxIg
Date: Wed, 10 Oct 2018 18:17:26 +0000
Message-ID: <AM0PR06MB4083B172F2424F1EEF08CFA0E7E00@AM0PR06MB4083.eurprd06.prod.outlook.com>
References: <AM0PR06MB4083426FA0F1D3F6515F2ECFE7E70@AM0PR06MB4083.eurprd06.prod.outlook.com> <87zhvlvpts.fsf@nic.cz> <AM0PR06MB40833D8AED0744BB597394E7E7E00@AM0PR06MB4083.eurprd06.prod.outlook.com> <d322e012-2767-a045-767a-ddf57649f36e@cisco.com>
In-Reply-To: <d322e012-2767-a045-767a-ddf57649f36e@cisco.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [185.139.140.77]
x-ms-publictraffictype: Email
x-microsoft-exchange-diagnostics: 1; AM0PR06MB4641; 6:apAsIsxXSObKFeNDZgGUcYhnlf2xK/LazDceieN6DNJhbYVG1U8fhezgvv8pdQhNczbsxNhOPCCSOM0xMaJJQYmXDsPAS9BC3/w0jb67Q3xTZtP43wuSraF2kr49Y8rH6JSsZFlUSPxM2lAvQjSH7Fz7rmOCchbFA09wqkgvZpWe4IJq0BuHIf+tQgvmT50EAO0KAPcr5qG39NwxE+TIxcxwCHkC7B+CElmMlLECBNXX2kp14hgTbVKmzlGWvdymoc82rK3D74NZCBlpBtU6+9g4zwBA/IKEPFBdoCFWrU57HcKXKD1AyJqAu8lrJsNEn/rnb0Kizjo6arJGUaP3amf+iA+GtaljSoW9AItCts1hJqVfUmSD6eRoadZFtNDW5WJLm6Eowk8TK93CjauY+T2Ah5atWaoYChOurHsJ3Dfo8+95XF9LWEyjSwjzrz0vQE7TFU+qbx7K/Xacu4orCg==; 5:Y24DppRdRuWHnv24dGODaBWCi75G6bcByX+UQ3xzO6dIywSSXizBcPc5jVorG0gvtOXVXP0Qsq15H5NGBhv1u0YAm1cfQxD32SeHJ09mRsHg9XE0GTBZ18WS25c5/aO7Mh6CWPbFj0MrV5jINekmXntudHzrTrWBL3Tkw3G+NSo=; 7:k3m4WgTcWxmQupBBgE7bO5Z38bc9cYMyKWVyGy8XDkdv6VUZ8rq6JKHNbM67YOAVahfBa6iFViJP2n9aEcgyx+8LTccKCKKERshFj0xH7DwdK8rUQXqCjmfTYSxUN0XWKkl59MT2BhYHcl79DCmK2w/OWYicp5wsJX6/skURJE6shMKmEDS7I3gMvR/ohbAQ3s6AKxzqc29X08lzPB9eZR6tzoG17Y9BL0xqrv/6CPTEyk5R75rKnp/MxGF3fAT2
x-ms-exchange-antispam-srfa-diagnostics: SOS;
x-ms-office365-filtering-correlation-id: 03fa783b-9535-4bb8-b233-08d62edca229
x-microsoft-antispam: BCL:0; PCL:0; RULEID:(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600074)(711020)(2017052603328)(7153060)(7193020); SRVR:AM0PR06MB4641;
x-ms-traffictypediagnostic: AM0PR06MB4641:
authentication-results: spf=none (sender IP is ) smtp.mailfrom=Michael.Rehder@Amdocs.com;
x-microsoft-antispam-prvs: <AM0PR06MB4641B8D617EA5C1352BFC45BE7E00@AM0PR06MB4641.eurprd06.prod.outlook.com>
x-exchange-antispam-report-test: UriScan:(95692535739014)(62221491112393)(131327999870524)(17755550239193)(166566539817055);
x-ms-exchange-senderadcheck: 1
x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(5005006)(8121501046)(3231355)(944501410)(52105095)(3002001)(10201501046)(93006095)(93001095)(149066)(150057)(6041310)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123558120)(20161123560045)(20161123564045)(20161123562045)(201708071742011)(7699051)(76991055); SRVR:AM0PR06MB4641; BCL:0; PCL:0; RULEID:; SRVR:AM0PR06MB4641;
x-forefront-prvs: 08213D42D3
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(396003)(136003)(366004)(39860400002)(376002)(346002)(51444003)(13464003)(199004)(189003)(486006)(68736007)(256004)(2501003)(106356001)(5250100002)(105586002)(74316002)(8936002)(97736004)(81166006)(81156014)(8676002)(4326008)(25786009)(6246003)(446003)(6116002)(3846002)(5660300001)(86362001)(2900100001)(71190400001)(71200400001)(476003)(11346002)(66066001)(26005)(966005)(2906002)(33656002)(72206003)(6436002)(93886005)(316002)(102836004)(229853002)(9686003)(186003)(53936002)(6506007)(114624004)(7736002)(99286004)(55016002)(53546011)(7696005)(305945005)(6306002)(110136005)(14454004)(478600001)(76176011); DIR:OUT; SFP:1101; SCL:1; SRVR:AM0PR06MB4641; H:AM0PR06MB4083.eurprd06.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1;
received-spf: None (protection.outlook.com: Amdocs.com does not designate permitted sender hosts)
x-microsoft-antispam-message-info: kr1KtrM6PzOyZBkh5JB+SEDzizBw/G/OrXXz7K8RuM3gATPk28JSyRxmRcykT5/24Qbztw/V964zUXRBRZWe+asA8BP59EH0gm8ubjr/7rHG3TL+K/HiHpzkO9aUWOjv+sf/LSMDNJGlVp8Vnc5PoWC64NYT4MlejYR2nycKFXFeJyvcNAMKLiZH/rFxDjooVCFgAdXjXdKAPrJBpE+wrkgyiXNTwA2D0R36z6vbIJhz1dViDhS8WfT0R01Vw0qiWA0uh8sypYSj37QjVt4HrdTy67o9MCWq0R9zp/1YFQ70KQIpeGVVs8vcrdmO1hW0+l7gnCSD6Y7Xrrzns+no0RYTfiYQZUXYb8cu3eQlfPs=
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 03fa783b-9535-4bb8-b233-08d62edca229
X-MS-Exchange-CrossTenant-originalarrivaltime: 10 Oct 2018 18:17:26.7359 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: c8eca3ca-1276-46d5-9d9d-a0f2a028920f
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR06MB4641
X-OriginatorOrg: amdocs.com
Content-Transfer-Encoding: base64
Archived-At: <https://mailarchive.ietf.org/arch/msg/netmod/O9rqTDhMAWVx78EtwtqfXY4b5BE>
Subject: Re: [netmod] WHEN statement within mandatory objects doesn't ensure presence of the mandatory object
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, 10 Oct 2018 18:17:38 -0000

If the list has a "when" clause the RNG file actually produces a "OneOrMore" which has a choice of <empty> or the list so it actually doesn't enforce the presence at least one row of the list (unless I'm mistaken in my reading).
              <oneOrMore>
                <choice>
                  <empty/>
                  <element name="IPAddresses">
                    <element name="Address">
                      <ref name="types__IPv4Address"/>
                    </element>
                    <empty/>
                  </element>
                </choice>
              </oneOrMore>

A leaf/container would be a simpler example but would result in the same lack of enforcement of the mandatory status of an element with a "when" clause.

This RNG seems consistent with the Schematron rules that "when" makes something optional.


I think a workaround would be choice with mandatory true and a when clause on the cases. This would ensure that at least one case is present since the mandatory clause implements a Schematron existence constraint.

Thanks
Mike
> -----Original Message-----
> From: Robert Wilton [mailto:rwilton@cisco.com]
> Sent: Wednesday, October 10, 2018 11:33 AM
> To: Michael Rehder <Michael.Rehder@Amdocs.com>; Ladislav Lhotka
> <lhotka@nic.cz>; netmod@ietf.org
> Cc: Walker, Jason (Jason_Walker2@comcast.com)
> <Jason_Walker2@comcast.com>
> Subject: Re: [netmod] WHEN statement within mandatory objects doesn't
> ensure presence of the mandatory object
> 
> Hi Mike,
> 
> I think that the YANG below already enforces what you want, or otherwise I
> don't follow your issue.
> 
> The YANG below is valid in two cases:
> 
> (1) AssignmentMechanism = DHCP, and IPAddresses is not present in the config
> (due to the when statement).
> (2) AssignmentMechanism = Static, IPAddresses exists and has at least one
> element (due to min-elements 1).
> 
> Thanks,
> Rob
> 
> 
> On 10/10/2018 16:23, Michael Rehder wrote:
> > Container "foo" would be mandatory if not for the "when" child element.
> > With the "when" child element, the logic becomes "inverted" and the
> constraint is a negative one of "disallowed under certain condition".
> >
> > The UC is for enforcement in REST API payloads.
> > For a practical example:
> >
> >           leaf AssignmentMechanism {
> >              type enumeration {
> >                enum "DHCP";
> >                enum "Static";
> >              }
> >              mandatory true;
> >              description "The address assignment mechanism.";
> >            }
> >            list IPAddresses {
> >              when "../AssignmentMechanism = 'Static'";
> >              key Address;
> >              min-elements 1;
> >
> >              leaf Address {
> >                type capit:IPv4Address;
> >                description "An ipv4 address.";
> >              }
> >             }
> >
> > There is no way in the IPAddresses list to enforce that there is at least one IP
> Address when the assignment method is "Static".
> > One could put a "must" on "AssignmentMechanism" to ensure at least one
> element of the IPAddresses list when "Static", but I don't see this as a good
> schema design, to have the controlling attribute check controlled attributes.
> >
> > I appreciate that this semantic can't be changed in YANG at this point.
> > Could the "when" statement have a modifying child element to state that the
> mandatory status of the element is to be enforced?
> > Like
> >      container foo {
> >        when "condition" {
> >            enforce-mandatory-status;
> >        }
> >
> > There is already back-end for existential checks for mandatory choice so this
> seems reasonably consistent to me.
> > I appreciate there are existing issues for "when" but I don't see why this
> would make things any worse.
> > In fact by promoting a better dependency "direction" between schema
> elements,  think it could simplify things (so I naively think :) ).
> >
> > Thanks
> > Mike
> >> -----Original Message-----
> >> From: Ladislav Lhotka [mailto:lhotka@nic.cz]
> >> Sent: Wednesday, October 10, 2018 10:28 AM
> >> To: Michael Rehder <Michael.Rehder@Amdocs.com>; netmod@ietf.org
> >> Subject: Re: [netmod] WHEN statement within mandatory objects doesn't
> >> ensure presence of the mandatory object
> >>
> >> Michael Rehder <Michael.Rehder@Amdocs.com> writes:
> >>
> >>> I have a question about “when” and mandatory objects.
> >>>
> >>> It seems to me that the implemented semantics of “when” are really
> >> “optional when”, in that the enclosing object can be absent even
> >> though it is mandatory and the “when” clause holds true.
> >>> The RFC could be clearer about this.
> >>>
> >>> Example
> >>>
> >>>     leaf color {
> >>>       enumeration  {
> >>>          enum “blue”;
> >>>          enum “black”;
> >>>       }
> >>>       mandatory true;
> >>>     }
> >>>     container foo {
> >>>        when ../color = ‘blue’;
> >>>        etc.
> >>>     }
> >>>
> >>> “foo” is optional due to the presence of the “when” statement even
> >>> though the object is mandatory (same is true for mandatory leaf,
> >>> min-elements=1 list etc.).
> >> Maybe you intended to have, e.g., a "mandatory true" leaf inside
> >> "container foo"?
> >>
> >>> This is considered valid XML for the above
> >>>      <color>blue</color>
> >> Yes, it is, under current YANG rules, no matter what "etc." stands
> >> for. Note that evaluation of the XPath expression in this case (with
> >> "foo" missing) requires the peculiar procedure of sec. 7.21.5 in RFC 7950.
> >>
> >>> In my view this makes conditionally variant schemas “loose” in their
> >>> enforcement (some scenarios can use choice but it doesn’t cover
> >>> everything).
> >>>
> >>> I think that mandatory should be respected for the enclosing objects
> >>> of a “when” statement.  That is, a mandatory object must be present
> >>> when its “when” clause holds true and a Schematron statement should
> >>> enforce that.
> >> In fact, this is one case where the DSDL mapping (RFC 6110) deviates
> >> from YANG 1.0. Nodes that mandatory aren't enclosed in the RELAX NG
> >> <optional> pattern, and are then required no matter what any "when"
> >> statements say (because RELAX NG validation comes before Schematron).
> >>
> >>> What is the rationale behind the current YANG rules behavior, that
> >>> the “when” Schematron mapping doesn’t check for presence of the
> >>> enclosing mandatory object?
> >> FWIW, I have been repeatedly protesting against this behaviour but
> >> without much luck. See for example
> >>
> >> https://www.ietf.org/mail-archive/web/netmod/current/msg14012.html
> >>
> >> As a result, "when" is the trickiest feature in YANG by far.
> >>
> >> Lada
> >>
> >>> thanks
> >>> Mike Rehder
> >> --
> >> Ladislav Lhotka
> >> Head, CZ.NIC Labs
> >> PGP Key ID: 0xB8F92B08A9F76C67
> > “Amdocs’ email platform is based on a third-party, worldwide, cloud-based
> system. Any emails sent to Amdocs will be processed and stored using such
> system and are accessible by third party providers of such system on a limited
> basis. Your sending of emails to Amdocs evidences your consent to the use of
> such system and such processing, storing and access”.
> > _______________________________________________
> > netmod mailing list
> > netmod@ietf.org
> > https://www.ietf.org/mailman/listinfo/netmod

“Amdocs’ email platform is based on a third-party, worldwide, cloud-based system. Any emails sent to Amdocs will be processed and stored using such system and are accessible by third party providers of such system on a limited basis. Your sending of emails to Amdocs evidences your consent to the use of such system and such processing, storing and access”.