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-223 bgpcep <p>The 'LSP Identifiers TLV' must be included in the LSP object in PCRpt messages for RSVP-signaled LSPs (draft-ietf-pce-stateful-pce-07). However, in case the path is setup using Segment Routing technique (PST=1) the 'LSP Identifiers TLV' might be not present. In this case LspId is not set in the PathBuilder and the updateLsp DS operation throws the following exception:</p> <p>2015-05-08 11:11:03,783 | WARN | oupCloseable-3-1 | DefaultChannelPipeline | 104 - io.netty.common - 4.0.26.Final | An exceptionCaught() event was fired, and it reached at the tail of the pipeline. It usually means the last handler in the pipeline did not handle the exception.<br/> java.lang.IllegalArgumentException: All keys must be specified for class org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.rev131024.pcep.client.attributes.path.computation.client.reported.lsp.PathKey. Missing key is getLspId. Supplied key is PathKey []</p> <p>The reported LSP is not written in the DS.</p> <p>Operating System: All<br/> Platform: All</p> BGPCEP-223

If no LSP Identifiers TLV is present in PCRpt message updateLsp in DS throws exception

Bug Resolved Done Milos Fabian Francesco Fondelli Wed, 13 May 2015 14:22:34 +0000 Sun, 3 Mar 2019 11:50:26 +0000 Tue, 16 Jun 2015 10:35:11 +0000 Bugzilla Migration Bugzilla Migration PCEP Wed, 17 Jun 2015 00:00:00 +0000 0 4 <p>please consider patch <a href="https://git.opendaylight.org/gerrit/#/c/20259/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/20259/</a></p> <p>ciao<br/> fra</p> <p>&gt; in case the path is setup using Segment Routing technique (PST=1)<br/> &gt; the 'LSP Identifiers TLV' might be not present</p> <p>My expectation is that as the first step towards the solution, a new design for this file (in BGPCEP repository) needs to be agreed upon:<br/> /pcep/topology-api/src/main/yang/network-topology-pcep.yang<br/> as currently lines 101 - 109 state this:<br/> list path {<br/> leaf lsp-id </p> { type rsvp:lsp-id; mandatory true; } <p> key lsp-id;</p> <p> uses pcep:path-definition;<br/> }</p> <p>The change in Java code then would depend on how the "type rsvp:lsp-id;" condition gets relaxed to accommodate for SR paths.</p> <p>Hi Vratko,</p> <p>yes you are right.</p> <p>What about replacing it with:</p> <p> list path {<br/> leaf plsp-id </p> { type plsp-id; mandatory true; } <p> key plsp-id;</p> <p> uses pcep:path-definition;<br/> }</p> <p>The plsp-id has PCC scope and is valid in both cases (RSVP-TE and SR). The rsvp:lsp-id is broken anyway because is unique within the scope of the tunnel-id (i.e. 2 RSVP-TE tunnels on the same PCC with the same lsp-id would collide in the reported-lsp list).</p> <p>ciao<br/> fra</p> <p>proposed patch: </p> <p><a href="https://git.opendaylight.org/gerrit/#/c/21672/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/21672/</a></p> <p>Needs to be fixed before Lithium goes out.</p> <p>stable/lithium: <a href="https://git.opendaylight.org/gerrit/#/c/22570/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/22570/</a></p> <p>master: <a href="https://git.opendaylight.org/gerrit/22675" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/22675</a></p> Development External issue ID 3204 External issue URL Issue Type ODL SR Target Milestone Rank 0|i02ayn: