*/
package org.opendaylight.bgpcep.pcep.topology.provider;
+import static com.google.common.base.Verify.verifyNotNull;
import static java.util.Objects.requireNonNull;
import com.google.common.collect.Iterables;
import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.rev220730.pcep.client.attributes.path.computation.client.reported.lsp.Path;
import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.rev220730.pcep.client.attributes.path.computation.client.reported.lsp.PathKey;
import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.Node;
+import org.opendaylight.yangtools.concepts.ObjectRegistration;
import org.opendaylight.yangtools.yang.binding.DataObject;
import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
import org.opendaylight.yangtools.yang.common.RpcResult;
@GuardedBy("this")
final Map<PlspId, String> lsps = new HashMap<>();
@GuardedBy("this")
- SessionStateImpl listenerState;
+ private ObjectRegistration<SessionStateImpl> listenerState;
// FIXME: clarify lifecycle rules of this map, most notably the interaction of multiple SrpIdNumbers
@GuardedBy("this")
session = psession;
nodeState = state;
- LOG.trace("Peer {} resolved to topology node {}", peerAddress, state.getNodeId());
+ final var nodeId = state.getNodeId();
+ LOG.trace("Peer {} resolved to topology node {}", peerAddress, nodeId);
// Our augmentation in the topology node
final PathComputationClientBuilder pccBuilder = new PathComputationClientBuilder()
synced.set(isSynchronized());
- final InstanceIdentifier<Node1> topologyAugment = state.getNodeId().augmentation(Node1.class);
+ final InstanceIdentifier<Node1> topologyAugment = nodeId.augmentation(Node1.class);
pccIdentifier = topologyAugment.child(PathComputationClient.class);
if (haveLspDbVersion) {
state.storeNode(topologyAugment,
new Node1Builder().setPathComputationClient(pccBuilder.build()).build(), psession);
- // TODO: collapse assignment? needs to be verified through bytecode
- final var sessionState = new SessionStateImpl(this, psession);
- listenerState = sessionState;
- statsProvider.bind(state.getNodeId(), sessionState);
- LOG.info("Session with {} attached to topology node {}", peerAddress, state.getNodeId());
+ listenerState = statsProvider.bind(nodeId, new SessionStateImpl(this, psession));
+ LOG.info("Session with {} attached to topology node {}", peerAddress, nodeId);
}
}
}
LOG.error("Session {} cannot be closed.", psession, e);
}
session = null;
- listenerState = null;
syncOptimization = null;
clearRequests();
}
session.close(TerminationReason.UNKNOWN);
session = null;
}
- listenerState = null;
syncOptimization = null;
clearRequests();
}
@Holding({"this.serverSessionManager", "this"})
private void clearNodeState() {
if (nodeState != null) {
- statsProvider.unbind(nodeState.getNodeId());
LOG.debug("Clear Node state: {}", nodeState.getNodeId());
+ if (listenerState != null) {
+ listenerState.close();
+ listenerState = null;
+ }
nodeState = null;
}
}
final synchronized PCEPRequest removeRequest(final SrpIdNumber id) {
final PCEPRequest ret = requests.remove(id);
if (ret != null && listenerState != null) {
- listenerState.processRequestStats(ret.getElapsedMillis());
+ // FIXME: just update fields
+ listenerState.getInstance().processRequestStats(ret.getElapsedMillis());
}
LOG.trace("Removed request {} object {}", id, ret);
return ret;
final synchronized ListenableFuture<OperationResult> sendMessage(final Message message, final SrpIdNumber requestId,
final Metadata metadata) {
final var sendFuture = session.sendMessage(message);
- listenerState.updateStatefulSentMsg(message);
+ // FIXME: just update fields
+ listenerState().updateStatefulSentMsg(message);
// Note: the timeout is held back by us holding the 'this' monitor, which timeoutExpired re-acquires
final var timeout = serverSessionManager.newRpcTimeout(this::timeoutExpired, requestId);
return lspUpdateCapability.get();
}
-
@Override
public synchronized ListenableFuture<RpcResult<Void>> tearDownSession(final TearDownSessionInput input) {
close();
return RpcResultBuilder.<Void>success().buildFuture();
}
+ final synchronized @NonNull SessionStateImpl listenerState() {
+ return verifyNotNull(listenerState).getInstance();
+ }
+
static final class MessageContext {
private final Collection<PCEPRequest> requests = new ArrayList<>();
private final WriteTransaction trans;
if (!(message instanceof PcrptMessage)) {
return true;
}
- listenerState.updateLastReceivedRptMsg();
+ // FIXME: update just a field
+ listenerState().updateLastReceivedRptMsg();
final var rpt = ((PcrptMessage) message).getPcrptMessage();
for (final Reports report : rpt.nonnullReports()) {
if (!manageNextReport(report, ctx)) {
*/
package org.opendaylight.bgpcep.pcep.topology.provider;
+import edu.umd.cs.findbugs.annotations.CheckReturnValue;
import org.eclipse.jdt.annotation.NonNull;
import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.pcep.stats.rev171113.PcepSessionState;
import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.Node;
import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.NodeKey;
+import org.opendaylight.yangtools.concepts.ObjectRegistration;
import org.opendaylight.yangtools.yang.binding.KeyedInstanceIdentifier;
/**
* @param nodeId Identifier of the topology node where it will be stored session stats under DS
* @param sessionState containing all Stats Session information
*/
- void bind(@NonNull KeyedInstanceIdentifier<Node, NodeKey> nodeId, @NonNull PcepSessionState sessionState);
-
- /**
- * Unregister Node from Stats Registry handler.
- *
- * @param nodeId Identifier of the topology node to be removed from registry
- */
- void unbind(@NonNull KeyedInstanceIdentifier<Node, NodeKey> nodeId);
+ // FIXME: BGPCEP-1105: nodeId is a bit superfluous, lifecycle is driven by AbstractTopologySessionListener
+ @CheckReturnValue
+ <T extends PcepSessionState> @NonNull ObjectRegistration<T> bind(
+ @NonNull KeyedInstanceIdentifier<Node, NodeKey> nodeId, @NonNull T sessionState);
}
import com.google.common.util.concurrent.MoreExecutors;
import java.util.HashMap;
import java.util.Map;
-import java.util.Map.Entry;
import java.util.Set;
import java.util.TimerTask;
import java.util.concurrent.ConcurrentHashMap;
import java.util.concurrent.TimeUnit;
import org.checkerframework.checker.lock.qual.GuardedBy;
import org.checkerframework.checker.lock.qual.Holding;
+import org.eclipse.jdt.annotation.NonNull;
import org.eclipse.jdt.annotation.Nullable;
import org.opendaylight.mdsal.binding.api.DataBroker;
import org.opendaylight.mdsal.binding.api.Transaction;
import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.stats.rev181109.PcepTopologyNodeStatsAugBuilder;
import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.Node;
import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.NodeKey;
+import org.opendaylight.yangtools.concepts.AbstractObjectRegistration;
+import org.opendaylight.yangtools.concepts.NoOpObjectRegistration;
+import org.opendaylight.yangtools.concepts.ObjectRegistration;
import org.opendaylight.yangtools.yang.binding.KeyedInstanceIdentifier;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
// retry in face of failing transactions.
private final Set<KeyedInstanceIdentifier<Node, NodeKey>> statsPendingDelete = ConcurrentHashMap.newKeySet();
@GuardedBy("this")
- private final Map<KeyedInstanceIdentifier<Node, NodeKey>, PcepSessionState> statsMap = new HashMap<>();
+ private final Map<KeyedInstanceIdentifier<Node, NodeKey>, Reg<?>> statsMap = new HashMap<>();
// Note: null indicates we have been shut down
@GuardedBy("this")
private DataBroker dataBroker;
final WriteTransaction tx = chain.newWriteOnlyTransaction();
try {
- for (Entry<KeyedInstanceIdentifier<Node, NodeKey>, PcepSessionState> entry : statsMap.entrySet()) {
+ for (var entry : statsMap.entrySet()) {
if (!statsPendingDelete.contains(entry.getKey())) {
- tx.put(LogicalDatastoreType.OPERATIONAL,
+ final var reg = entry.getValue();
+ if (reg.notClosed()) {
+ tx.put(LogicalDatastoreType.OPERATIONAL,
entry.getKey().augmentation(PcepTopologyNodeStatsAug.class),
new PcepTopologyNodeStatsAugBuilder()
- .setPcepSessionState(new PcepSessionStateBuilder(entry.getValue()).build())
- .build());
+ .setPcepSessionState(new PcepSessionStateBuilder(reg.getInstance()).build())
+ .build());
+ }
}
}
} catch (Exception e) {
}
@Override
- public synchronized void bind(final KeyedInstanceIdentifier<Node, NodeKey> nodeId,
- final PcepSessionState sessionState) {
- if (dataBroker != null) {
- statsMap.put(nodeId, sessionState);
- } else {
+ public synchronized <T extends PcepSessionState> ObjectRegistration<T> bind(
+ final KeyedInstanceIdentifier<Node, NodeKey> nodeId, final T sessionState) {
+ if (dataBroker == null) {
LOG.debug("Ignoring bind of Pcep Node {}", nodeId);
+ return NoOpObjectRegistration.of(sessionState);
}
+
+ final var ret = new Reg<>(sessionState, nodeId);
+ // FIXME: a replace should never happen, and hence regs are just a Set (which can be concurrent and this method
+ // does not need synchronization
+ statsMap.put(nodeId, ret);
+ return ret;
}
- @Override
- public synchronized void unbind(final KeyedInstanceIdentifier<Node, NodeKey> nodeId) {
+ private synchronized void removeRegistration(final @NonNull Reg<?> reg) {
+ final var nodeId = reg.nodeId;
+
+ if (!statsMap.remove(nodeId, reg)) {
+ // Already replaced by a subsequent bind()
+ LOG.debug("Ignoring overridden unbind of Pcep Node {}", nodeId);
+ return;
+ }
+
final TransactionChain chain = accessChain();
if (chain == null) {
// Already closed, do not bother
return;
}
- final PcepSessionState node = statsMap.remove(nodeId);
- if (node == null) {
- LOG.debug("Ignoring duplicate unbind of Pcep Node {}", nodeId);
- return;
- }
-
statsPendingDelete.add(nodeId);
final WriteTransaction wTx = chain.newWriteOnlyTransaction();
wTx.delete(LogicalDatastoreType.OPERATIONAL, nodeId);
}
}, MoreExecutors.directExecutor());
}
+
+ private final class Reg<T extends PcepSessionState> extends AbstractObjectRegistration<T> {
+ private final @NonNull KeyedInstanceIdentifier<Node, NodeKey> nodeId;
+
+ Reg(final @NonNull T instance, final KeyedInstanceIdentifier<Node, NodeKey> nodeId) {
+ super(instance);
+ this.nodeId = requireNonNull(nodeId);
+ }
+
+ @Override
+ protected void removeRegistration() {
+ TopologyStatsProviderImpl.this.removeRegistration(this);
+ }
+ }
}
import static org.junit.Assert.assertTrue;
import static org.mockito.ArgumentMatchers.any;
import static org.mockito.Mockito.doAnswer;
-import static org.mockito.Mockito.doNothing;
import static org.mockito.Mockito.doReturn;
import static org.mockito.Mockito.mock;
import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.rev130715.IpPrefix;
import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.rev130715.Ipv4Prefix;
import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.graph.rev220720.graph.topology.GraphKey;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.pcep.stats.rev171113.PcepSessionState;
import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.pcep.types.rev181109.explicit.route.object.Ero;
import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.pcep.types.rev181109.explicit.route.object.EroBuilder;
import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.pcep.types.rev181109.explicit.route.object.ero.Subobject;
import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.TopologyKey;
import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.Node;
import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.NodeKey;
+import org.opendaylight.yangtools.concepts.NoOpObjectRegistration;
import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
import org.opendaylight.yangtools.yang.binding.KeyedInstanceIdentifier;
import org.opendaylight.yangtools.yang.binding.Notification;
doReturn(pipeline).when(pipeline).replace(any(ChannelHandler.class), any(String.class),
any(ChannelHandler.class));
doReturn(eventLoop).when(clientListener).eventLoop();
- doNothing().when(statsRegistry).bind(any(), any());
- doNothing().when(statsRegistry).unbind(any());
+ doAnswer(inv -> NoOpObjectRegistration.of(inv.getArgument(1, PcepSessionState.class)))
+ .when(statsRegistry).bind(any(), any());
doReturn(null).when(eventLoop).schedule(any(Runnable.class), any(long.class), any(TimeUnit.class));
doReturn(true).when(clientListener).isActive();
final InetSocketAddress ra = new InetSocketAddress(testAddress, 4189);
import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.rev220730.pcep.client.attributes.path.computation.client.ReportedLsp;
import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.topology.pcep.rev220730.pcep.client.attributes.path.computation.client.reported.lsp.Path;
import org.opendaylight.yangtools.yang.common.RpcResult;
+import org.opendaylight.yangtools.yang.common.Uint16;
import org.opendaylight.yangtools.yang.common.Uint32;
public class PCEPTopologySessionListenerTest extends AbstractPCEPSessionTest {
@Test
public void testPCEPTopologySessionListener() throws Exception {
listener.onSessionUp(session);
- final PcepSessionState listenerState = listener.listenerState;
- assertEquals(testAddress, listenerState.getPeerPref().getIpAddress());
- final LocalPref state = listener.listenerState.getLocalPref();
+ final PcepSessionState listenerState = listener.listenerState();
+ final LocalPref state = listenerState.getLocalPref();
assertNotNull(state);
assertEquals(DEAD_TIMER, state.getDeadtimer().shortValue());
assertEquals(KEEP_ALIVE, state.getKeepalive().shortValue());
- assertEquals(0, state.getSessionId().intValue());
+ assertEquals(Uint16.ZERO, state.getSessionId());
assertEquals(testAddress, state.getIpAddress());
final PeerPref peerState = listenerState.getPeerPref();
-
+ assertEquals(testAddress, peerState.getIpAddress());
assertEquals(DEAD_TIMER, peerState.getDeadtimer().shortValue());
assertEquals(KEEP_ALIVE, peerState.getKeepalive().shortValue());
- assertEquals(0, peerState.getSessionId().intValue());
+ assertEquals(Uint16.ZERO, peerState.getSessionId());
assertEquals(testAddress, peerState.getIpAddress());
// add-lsp
.build(), Optional.of(MsgBuilderUtil.createSrp(srpId)), MsgBuilderUtil.createPath(
req.getEro().getSubobject()));
listener.onMessage(session, pcRpt);
- checkEquals(() -> assertEquals(1, listener.listenerState.getDelegatedLspsCount().intValue()));
+ checkEquals(() -> assertEquals(Uint16.ONE, listener.listenerState().getDelegatedLspsCount()));
}
@Test
.build(), Optional.of(MsgBuilderUtil.createSrp(srpId)),
MsgBuilderUtil.createPath(req.getEro().getSubobject()));
listener.onMessage(session, pcRpt);
- checkEquals(() -> assertEquals(0, listener.listenerState.getDelegatedLspsCount().intValue()));
+ checkEquals(() -> assertEquals(Uint16.ZERO, listener.listenerState().getDelegatedLspsCount()));
}
@Override