https://jira.opendaylight.org This file is an XML representation of an issue en-us 8.20.10 820010 22-06-2022 https://jira.opendaylight.org/browse/NETCONF-122 netconf <p>In Beryllium CSIT, there is a test case which sends data <span class="error">&#91;0&#93;</span>.<br/> This data does not contain keepalive-executor container. The container would have "type" field, which is explicitly set as not mandatory in <span class="error">&#91;1&#93;</span>, but it would also have a "name" field, which is inherited as mandatory from <span class="error">&#91;2&#93;</span>.</p> <p>ODL resolves this situation as user error, as "name" is mandatory but not present in data.<br/> This is possibly a netconf Bug, odl-sal-netconf-connector-cfg.yang should make sure keepalive-executor is not mandatory.</p> <p>Looking at <span class="error">&#91;3&#93;</span>, this can be fixed by making keepalive-executor a presence container, in which case "type" may be kept mandatory.</p> <p>Here is a piece of karaf.log:</p> <p>Caused by: java.lang.IllegalArgumentException: Node (urn:opendaylight:params:xml:ns:yang:controller:config?revision=2013-04-05)configuration is missing mandatory descendant /(urn:opendaylight:params:xml:ns:yang:controller:md:sal:connector:netconf?revision=2015-08-03)keepalive-executor/name<br/> at com.google.common.base.Preconditions.checkArgument(Preconditions.java:145)<span class="error">&#91;39:com.google.guava:18.0.0&#93;</span><br/> at org.opendaylight.yangtools.yang.data.impl.schema.tree.MandatoryLeafEnforcer$Strict.enforceOnTreeNode(MandatoryLeafEnforcer.java:58)<span class="error">&#91;59:org.opendaylight.yangtools.yang-data-impl:0.8.0.SNAPSHOT&#93;</span></p> <p><span class="error">&#91;0&#93;</span> <a href="https://jenkins.opendaylight.org/releng/view/netconf/job/netconf-csit-1node-userfeatures-only-beryllium/lastSuccessfulBuild/robot/report/log.html#s1-s2-s1-t36-k2-k1-k2-k1-k3" class="external-link" target="_blank" rel="nofollow noopener">https://jenkins.opendaylight.org/releng/view/netconf/job/netconf-csit-1node-userfeatures-only-beryllium/lastSuccessfulBuild/robot/report/log.html#s1-s2-s1-t36-k2-k1-k2-k1-k3</a><br/> <span class="error">&#91;1&#93;</span> <a href="https://git.opendaylight.org/gerrit/gitweb?p=netconf.git;a=blob;f=opendaylight/netconf/sal-netconf-connector/src/main/yang/odl-sal-netconf-connector-cfg.yang;h=d903ccfa248f0a0409d2ae20c1a57b76fdf9ed49;hb=db7649383aa96be597a77f7a6da2b40be6fefc22#l171" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/gitweb?p=netconf.git;a=blob;f=opendaylight/netconf/sal-netconf-connector/src/main/yang/odl-sal-netconf-connector-cfg.yang;h=d903ccfa248f0a0409d2ae20c1a57b76fdf9ed49;hb=db7649383aa96be597a77f7a6da2b40be6fefc22#l171</a><br/> <span class="error">&#91;2&#93;</span> <a href="https://git.opendaylight.org/gerrit/gitweb?p=controller.git;a=blob;f=opendaylight/config/config-api/src/main/yang/config.yang;h=e46d327ece9aff4e89ecf7ab0088561acb17c5aa;hb=37cbf86dd56fb1d79a817750951283e1ae688cf7#l131" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/gitweb?p=controller.git;a=blob;f=opendaylight/config/config-api/src/main/yang/config.yang;h=e46d327ece9aff4e89ecf7ab0088561acb17c5aa;hb=37cbf86dd56fb1d79a817750951283e1ae688cf7#l131</a><br/> <span class="error">&#91;3&#93;</span> <a href="http://tools.ietf.org/html/rfc6020#section-7.6.5" class="external-link" target="_blank" rel="nofollow noopener">http://tools.ietf.org/html/rfc6020#section-7.6.5</a></p> <p>Operating System: All<br/> Platform: All</p> NETCONF-122

mandatory descendant keepalive-executor/name

Bug Resolved Done Tomas Cere Vratko Polak Mon, 11 Jan 2016 17:11:34 +0000 Fri, 15 Mar 2019 22:22:18 +0000 Tue, 19 Jan 2016 09:48:17 +0000 netconf 0 2 <p>Currently config subsystem does not support presence containers at all.<br/> The behavior in config is that once it sees that the type ref is not mandatory it expects that name is not present either, even if it still inherits mandatory true.<br/> This is not true for md-sal which correctly enforces the mandatory statements.</p> <p>Presence container would be the correct approach if cfg subsystem didn't have this limitation but since that is not the case in the meantime the solution will be to <br/> make both type and name non-mandatory.</p> <p><a href="https://git.opendaylight.org/gerrit/#/c/32766/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/32766/</a></p> <p><a href="https://git.opendaylight.org/gerrit/#/c/32434/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/32434/</a></p> Development External issue ID 4930 External issue URL Issue Type ODL SR Target Milestone Rank 0|i01wlb: