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/CONTROLLER-641 controller <p>device reporting yang modules but without revision<br/> In this case ODL is not even able to read local Yang files from /cache/schema folder.</p> <p>Operating System: Linux<br/> Platform: PC</p> CONTROLLER-641

Read Yang files from Local if device reported capabilities don't have the revision

Bug Resolved Done Maros Marsalek Ankit agarwal Fri, 18 Jul 2014 13:41:26 +0000 Wed, 17 Sep 2014 13:23:29 +0000 Wed, 17 Sep 2014 13:23:29 +0000 Helium netconf 0 8 <p>Below mentioned gerrit changes can be used to fix these issues.</p> <p><a href="https://git.opendaylight.org/gerrit/#/c/9112/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/9112/</a></p> <p><a href="https://git.opendaylight.org/gerrit/#/c/9135/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/9135/</a></p> <p>Hi Ankit,<br/> would you rework your controller patch<br/> <a href="https://git.opendaylight.org/gerrit/#/c/9135/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/9135/</a> <br/> to not require this change in YANGTools?</p> <p>Would you abandon your YANGTools patch <br/> <a href="https://git.opendaylight.org/gerrit/#/c/9112/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/9112/</a><br/> since it starts allowing Null values in places, where they were not allowed<br/> and has potential to introduce NPE based regression at various places (parser, codecs...).</p> <p>Hi Tony,</p> <p>I have submitted a new patch on </p> <p><a href="https://git.opendaylight.org/gerrit/#/c/9135/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/9135/</a> <br/> and</p> <p><a href="https://git.opendaylight.org/gerrit/#/c/9112/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/9112/</a><br/> But both the changes are required to fix the Bug.</p> <p>In 9112 now only YangModelDependencyInfo class has changes which is a caller for the parsing method.</p> <p>Regarding this bug, there is plenty of evidence in RFC 6020 that the first revision of a YANG module does not need to have a "revision" statement, and hence not needed in the &lt;hello&gt; message, but section 5.6.4.1 says:</p> <p> "A server MUST advertise all revisions of all modules it implements."</p> <p>Which seems contradictory. Discussing with the NETMOD WG, it is agreed that the sentence should have been written:</p> <p> "A server MUST advertise all known revisions of all modules it implements."</p> <p>Thus providing the necessary leeway.</p> <p>FWIW, the only time it is envisioned a YANG module not wanting to have an initial revision date is when the module itself is already version-specific (e.g., <a href="http://xml.juniper.net/netconf/junos/14.2R2.4" class="external-link" target="_blank" rel="nofollow noopener">http://xml.juniper.net/netconf/junos/14.2R2.4</a>), which clearly will have one and only one revision. In all other cases, it seems the module is expected to be versioned over time and therefore SHOULD have a revision statement. For this reason, the NETMOD WG may also add a clarifying statement into section 7.1.9.</p> <p>Thanks,<br/> Kent</p> Duplicate CONTROLLER-793 Development External issue ID 1393 External issue URL ODL SR Target Milestone Rank 0|i02lnj: