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-329 bgpcep <p>R(Remove - 1 bit): On PCRpt messages the R Flag indicates that the<br/> LSP has been removed from the PCC and the PCE SHOULD remove all<br/> state from its database. Upon receiving an LSP State Report with<br/> the R Flag set to 1 for an RSVP-signaled LSP, the PCE SHOULD<br/> remove all state for the path identified by the LSP Identifiers<br/> TLV from its database. When the all-zeros LSP-IDENTIFIERS TLV is<br/> used, the PCE SHOULD remove all state for the PLSP-ID from its<br/> database.</p> <p>Operating System: All<br/> Platform: All</p> BGPCEP-329

updateLsp () in class org.opendaylight.bgpcep.pcep.topology.provider.AbstractTopologySessionListener doesnt match "PCEP Extensions for Stateful PCE draft-ietf-pce-stateful-pce-11"

Bug Resolved Cannot Reproduce Unassigned Geng Xingyuan Wed, 9 Dec 2015 01:24:36 +0000 Sun, 3 Mar 2019 11:49:33 +0000 Wed, 20 Jan 2016 17:13:46 +0000 Bugzilla Migration Bugzilla Migration PCEP 0 1 <p>protected final synchronized void updateLsp(final MessageContext ctx, final L id, final String lspName,<br/> final ReportedLspBuilder rlb, final boolean solicited, final boolean remove) {</p> <p> final String name;<br/> if (lspName == null) {<br/> name = this.lsps.get(id);<br/> if (name == null) {<br/> LOG.error("PLSPID {} seen for the first time, not reporting the LSP", id);<br/> return;<br/> }<br/> } else </p> { name = lspName; } <p> LOG.debug("Saved LSP {} with name {}", id, name);<br/> this.lsps.put(id, name);</p> <p> final ReportedLsp previous = this.lspData.get(name);<br/> // if no previous report about the lsp exist, just proceed<br/> if (previous != null) {<br/> final List&lt;Path&gt; updatedPaths = makeBeforeBreak(rlb, previous, name, remove);<br/> // if all paths or the last path were deleted, delete whole tunnel<br/> if (updatedPaths == null || updatedPaths.isEmpty()) {<br/> LOG.debug("All paths were removed, removing LSP with {}.", id);<br/> removeLsp(ctx, id);<br/> return;<br/> }<br/> rlb.setPath(updatedPaths);<br/> }<br/> rlb.setKey(new ReportedLspKey(name));<br/> rlb.setName(name);</p> <p> // If this is an unsolicited update. We need to make sure we retain the metadata already present<br/> if (solicited) </p> { this.nodeState.setLspMetadata(name, rlb.getMetadata()); } <p> else </p> { rlb.setMetadata(this.nodeState.getLspMetadata(name)); } <p> //FIX!!! should check R flag in this place!!!!!!!!<br/> final ReportedLsp rl = rlb.build();<br/> ctx.trans.put(LogicalDatastoreType.OPERATIONAL, this.pccIdentifier.child(ReportedLsp.class, rlb.getKey()), rl);<br/> this.lspData.put(name, rl);</p> <p> }</p> <p>Input parameter "remove" represents "R" flag. The flag is later treated in "makeBeforeBreak(...)" as described in pce-stateful draft.</p> <p>Are you experiencing any problems with LSP removal? If so, could you please describe the use-case?</p> <p>Insufficient information provided in a bug report.</p> Development External issue ID 4744 External issue URL Issue Type ODL SR Target Milestone Rank 0|i02bm7: