import org.slf4j.LoggerFactory;
import scala.concurrent.Future;
-class NetconfTopologyContext implements ClusterSingletonService {
+class NetconfTopologyContext implements ClusterSingletonService, AutoCloseable {
private static final Logger LOG = LoggerFactory.getLogger(NetconfTopologyContext.class);
private RemoteDeviceConnector remoteDeviceConnector;
private NetconfNodeManager netconfNodeManager;
private ActorRef masterActorRef;
- private boolean finalClose = false;
private final AtomicBoolean closed = new AtomicBoolean(false);
- private boolean isMaster;
+ private final AtomicBoolean stopped = new AtomicBoolean(false);
+ private volatile boolean isMaster;
NetconfTopologyContext(final NetconfTopologySetup netconfTopologyDeviceSetup,
final ServiceGroupIdentifier serviceGroupIdent,
netconfNodeManager = null;
}
- if (!finalClose) {
+ if (!closed.get()) {
final String masterAddress =
Cluster.get(netconfTopologyDeviceSetup.getActorSystem()).selfAddress().toString();
masterActorRef = netconfTopologyDeviceSetup.getActorSystem().actorOf(NetconfNodeActor.props(
@Override
public ListenableFuture<Void> closeServiceInstance() {
- if (!finalClose) {
+ if (!closed.get()) {
// in case that master changes role to slave, new NodeDeviceManager must be created and listener registered
netconfNodeManager = createNodeDeviceManager();
}
stopDeviceConnectorAndActor();
- return Futures.immediateCheckedFuture(null);
+ return Futures.immediateFuture(null);
}
@Override
return ndm;
}
- void closeFinal() throws Exception {
- finalClose = true;
+ @Override
+ public void close() throws Exception {
+ if (!closed.compareAndSet(false, true)) {
+ return;
+ }
if (netconfNodeManager != null) {
netconfNodeManager.close();
}
private void stopDeviceConnectorAndActor() {
- if (!closed.compareAndSet(false, true)) {
+ if (!stopped.compareAndSet(false, true)) {
return;
}
if (remoteDeviceConnector != null) {