From 12435c9d286615ecebee800c1dd608d3765b768a Mon Sep 17 00:00:00 2001
From: Robert Varga <rovarga@cisco.com>
Date: Wed, 22 Jan 2014 14:27:28 +0100
Subject: [PATCH] Make sure we record a reused node

This fixes a bug where we fail to record the base node identifier when
we reuse an already-existing node in MD-SAL.

Change-Id: I33f407810da6ab452a0c12933d3572bfa14a1f53
Signed-off-by: Robert Varga <rovarga@cisco.com>
---
 .../bgpcep/pcep/topology/provider/ServerSessionManager.java   | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/pcep/topology-provider/src/main/java/org/opendaylight/bgpcep/pcep/topology/provider/ServerSessionManager.java b/pcep/topology-provider/src/main/java/org/opendaylight/bgpcep/pcep/topology/provider/ServerSessionManager.java
index 95e5ec79cf..4448330f19 100644
--- a/pcep/topology-provider/src/main/java/org/opendaylight/bgpcep/pcep/topology/provider/ServerSessionManager.java
+++ b/pcep/topology-provider/src/main/java/org/opendaylight/bgpcep/pcep/topology/provider/ServerSessionManager.java
@@ -124,6 +124,9 @@ final class ServerSessionManager implements SessionListenerFactory<PCEPSessionLi
 			for (final Node n : topo.getNode()) {
 				LOG.debug("Matching topology node {} to id {}", n, pccId);
 				if (n.getNodeId().getValue().equals(pccId)) {
+					this.topologyNode =
+							InstanceIdentifier.builder(ServerSessionManager.this.topology).child(Node.class, n.getKey()).toInstance();
+					LOG.debug("Reusing topology node {} for id {} at {}", n, pccId, this.topologyNode);
 					return n;
 				}
 			}
@@ -139,6 +142,7 @@ final class ServerSessionManager implements SessionListenerFactory<PCEPSessionLi
 			final Node ret = new NodeBuilder().setKey(nk).setNodeId(id).build();
 
 			trans.putOperationalData(nti, ret);
+			LOG.debug("Created topology node {} for id {} at {}", ret, pccId, nti);
 			this.ownsTopology = true;
 			this.topologyNode = nti;
 			return ret;
-- 
2.36.6