X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=opendaylight%2Fmd-sal%2Fsal-akka-raft%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Fraft%2FRaftActorContextImpl.java;h=8b28d5213fbbe45c51bb1a260f5fe324a9a9c40b;hb=15f11d24306fe2796411e8078a84fc82213bacf6;hp=f5362b56b467930d69177fbb099e281173757463;hpb=cabb96a79c2b155092e65fb25d271fff85c3f786;p=controller.git diff --git a/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/RaftActorContextImpl.java b/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/RaftActorContextImpl.java index f5362b56b4..8b28d5213f 100644 --- a/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/RaftActorContextImpl.java +++ b/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/RaftActorContextImpl.java @@ -14,6 +14,7 @@ import akka.actor.ActorSelection; import akka.actor.ActorSystem; import akka.actor.Props; import com.google.common.annotations.VisibleForTesting; +import com.google.common.base.Preconditions; import com.google.common.base.Supplier; import java.util.ArrayList; import java.util.Collection; @@ -24,6 +25,7 @@ import java.util.Map; import java.util.Set; import org.opendaylight.controller.cluster.DataPersistenceProvider; import org.opendaylight.controller.cluster.raft.ServerConfigurationPayload.ServerInfo; +import org.opendaylight.controller.cluster.raft.behaviors.RaftActorBehavior; import org.opendaylight.controller.cluster.raft.policy.RaftPolicy; import org.slf4j.Logger; @@ -64,6 +66,8 @@ public class RaftActorContextImpl implements RaftActorContext { private boolean votingMember = true; + private RaftActorBehavior currentBehavior; + public RaftActorContextImpl(ActorRef actor, ActorContext context, String id, ElectionTerm termInformation, long commitIndex, long lastApplied, Map peerAddresses, ConfigParams configParams, DataPersistenceProvider persistenceProvider, Logger logger) { @@ -192,9 +196,11 @@ public class RaftActorContextImpl implements RaftActorContext { @Override public void updatePeerIds(ServerConfigurationPayload serverConfig){ votingMember = true; + boolean foundSelf = false; Set currentPeers = new HashSet<>(this.getPeerIds()); for(ServerInfo server: serverConfig.getServerConfig()) { if(getId().equals(server.getId())) { + foundSelf = true; if(!server.isVoting()) { votingMember = false; } @@ -212,6 +218,11 @@ public class RaftActorContextImpl implements RaftActorContext { for(String peerIdToRemove: currentPeers) { this.removePeer(peerIdToRemove); } + + if(!foundSelf) { + votingMember = false; + } + setDynamicServerConfigurationInUse(); } @@ -226,7 +237,11 @@ public class RaftActorContextImpl implements RaftActorContext { @Override public void removePeer(String name) { - peerInfoMap.remove(name); + if(getId().equals(name)) { + votingMember = false; + } else { + peerInfoMap.remove(name); + } } @Override public ActorSelection getPeerActorSelection(String peerId) { @@ -312,4 +327,23 @@ public class RaftActorContextImpl implements RaftActorContext { public boolean isVotingMember() { return votingMember; } + + @Override + public RaftActorBehavior getCurrentBehavior() { + return currentBehavior; + } + + void setCurrentBehavior(final RaftActorBehavior behavior) { + this.currentBehavior = Preconditions.checkNotNull(behavior); + } + + void close() { + if (currentBehavior != null) { + try { + currentBehavior.close(); + } catch (Exception e) { + LOG.debug("{}: Error closing behavior {}", getId(), currentBehavior.state()); + } + } + } }