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=d0deb7101a2f05529539d09107bc3c9ee52a9d49;hb=3676d1686706dbee6656e86a23c4bdb516d5267b;hp=4677ea946e65cf352779d027744e1c7e232bc572;hpb=ffe27dc93dc6f6a190287164f10444f4f6838d59;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 4677ea946e..d0deb7101a 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,7 +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.Supplier; +import com.google.common.base.Preconditions; import java.util.ArrayList; import java.util.Collection; import java.util.HashMap; @@ -22,8 +22,10 @@ import java.util.HashSet; import java.util.List; import java.util.Map; import java.util.Set; +import java.util.function.Supplier; 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(); } @@ -316,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()); + } + } + } }