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/BGPCEP-311 bgpcep <p>Change 29551 <span class="error">&#91;0&#93;</span> which fixed <a href="https://jira.opendaylight.org/browse/BGPCEP-306" title="Module &quot;bgp&quot; name conflict" class="issue-link" data-issue-key="BGPCEP-306"><del>BGPCEP-306</del></a> has passed a system test once <span class="error">&#91;1&#93;</span>, but since then it is failing, first sign is this in karaf.log:</p> <p>2015-11-13 04:35:19,066 | WARN | config-pusher | JavassistUtils | 62 - org.opendaylight.mdsal.binding-generator-impl - 0.8.0.SNAPSHOT | Failed to customize org.opendaylight.yang.gen.v1.http.openconfig.net.yang.bgp.rev151009.bgp.global.base.Config$StreamWriter from prototype org.opendaylight.yangtools.binding.data.codec.gen.impl.DataObjectSerializerPrototype javassist.CannotCompileException: <span class="error">&#91;source error&#93;</span> no such class: org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.rev130715.AsNumber at javassist.CtBehavior.setBody(CtBehavior.java:446)</p> <p>Specific reason is not clear to me. Maybe the new BGP models end up using incorrect revision of ietf-inet-types, or perhaps MDSAL does not expose the newer revision correctly.</p> <p>Operating System: All<br/> Platform: All</p> BGPCEP-311

New openconfig modules frequently cause "server is unhealthy".

Bug Resolved Done Claudio David Gasparini Vratko Polak Fri, 13 Nov 2015 12:18:16 +0000 Sun, 3 Mar 2019 11:50:33 +0000 Fri, 20 Nov 2015 18:37:58 +0000 Bugzilla Migration Bugzilla Migration BGP 0 3 <p>Downstream consequences:</p> <p>Merges to integration/testing get -1 from verify job.<br/> Easy fix by Change 29652, with no external consequences.</p> <p>Merges to integration/distribution get -1 from verify job.<br/> The most prudent fix seem to be removing BGP from integration features (temporarily): <a href="https://git.opendaylight.org/gerrit/29470" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/29470</a><br/> This would give false positives to external projects which may be affecting this Bug (for example Vpnservice); so if this Bug takes too long to fix, some cross-project testing may be needed before BGP is included in integration again.<br/> Note that Bgpcep test would still install BGP features, as the feature repository stays available.</p> <p><a href="https://git.opendaylight.org/gerrit/#/c/29660/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/29660/</a></p> <p>&gt; Another alternative is to bump yang-types/ietf-types across the board &#8211; not sure how feasible that is.<br/> (source: Robert commenting in <a href="https://git.opendaylight.org/gerrit/#/c/29470/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/29470/</a>)</p> <p>I tried, not sure if it was worth of the time I spent on it.<br/> Good news: Locally when I build:<br/> Mdsal with <a href="https://git.opendaylight.org/gerrit/29692/1" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/29692/1</a><br/> Bgpcep with <a href="https://git.opendaylight.org/gerrit/29660/4" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/29660/4</a><br/> Distribution<br/> I get .tar.gz without those Javassist or Augmentation errors.<br/> Bad news:<br/> I had to use -DskipTests as some tests end up in failures.<br/> Restconf returns 401 Unauthorized.<br/> 2015-11-14 00:53:32,013 | ERROR | config-pusher | ConfigPusherImpl | 96 - org.opendaylight.controller.config-persister-impl - 0.4.0.SNAPSHOT | Failed to apply configuration snapshot: 31-bgp.xml(odl-bgpcep-bmp,odl-bgpcep-bmp)<br/> java.lang.IllegalStateException: Unable to push configuration due to missing yang models. Required yang models that are missing: <span class="error">&#91;urn:opendaylight:params:xml:ns:yang:controller:bgp:labeled:unicast?module=odl-bgp-labeled-unicast-cfg&amp;revision=2015-05-25&#93;</span></p> <p>I do not see an easy way to generate <b>.yang.network.topology.rev131021.</b> classes that would work with both older <b>.yang.ietf.inet.types.rev100924.</b> classes and the newer <b>.yang.ietf.inet.types.rev130715.</b> ones. Aside from just pasting contents of rev130715 yang file into the rev100924 one and keep using that (changes are compatible this way).</p> <p>I do not expect the long list of project to get their version of Change 29692.</p> <p>The yang model pasting solution may be the easiest one, followed by removing openconfig stuff from Beryllium Bgpcep.</p> <p>Do we plan some serious design effort on how to handle multiple revisions and revision bumping in Boron?</p> <p>&gt; The yang model pasting solution may be the easiest one,<br/> &gt; followed by removing openconfig stuff from Beryllium Bgpcep.</p> <p>Just to be clear, openconfig removal is a separate solution, not a next step of the first solution.</p> <p>By the way, we now have new job for system testing multiple patches at once. Change 29692 and Change 29660 fared like this:<br/> <a href="https://jenkins.opendaylight.org/sandbox/job/integration-multipatch-test-beryllium/7/" class="external-link" target="_blank" rel="nofollow noopener">https://jenkins.opendaylight.org/sandbox/job/integration-multipatch-test-beryllium/7/</a></p> <p>Bgpcep test job failed: <a href="https://jenkins.opendaylight.org/sandbox/job/bgpcep-csit-1node-userfeatures-all-beryllium/3/console" class="external-link" target="_blank" rel="nofollow noopener">https://jenkins.opendaylight.org/sandbox/job/bgpcep-csit-1node-userfeatures-all-beryllium/3/console</a></p> <p>2015-11-14 10:20:01,941 | ERROR | config-pusher | ConfigRegistryImpl | 88 - org.opendaylight.controller.config-manager - 0.4.0.SNAPSHOT | Configuration Transaction failed on 2PC, server is unhealthy<br/> java.lang.NoSuchMethodError: org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.TopologyId.&lt;init&gt;(Lorg/opendaylight/yang/gen/v1/urn/ietf/params/xml/ns/yang/ietf/inet/types/rev100924/Uri;)V<br/> at org.opendaylight.ovsdb.southbound.SouthboundConstants.&lt;clinit&gt;(SouthboundConstants.java:46)</p> <p>My testing suggests this patch by Claudio has no downsides.<br/> Bgpcep master: <a href="https://git.opendaylight.org/gerrit/29660/9" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/29660/9</a></p> Blocks MDSAL-78 Development External issue ID 4633 External issue URL Issue Type Rank 0|i02bi7: