*/
package org.opendaylight.controller.cluster.raft;
+import static akka.pattern.Patterns.ask;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertNotNull;
+
import akka.actor.ActorRef;
import akka.actor.InvalidActorNameException;
import akka.actor.PoisonPill;
import akka.actor.Terminated;
import akka.dispatch.Dispatchers;
+import akka.dispatch.Mailboxes;
import akka.testkit.JavaTestKit;
import akka.testkit.TestActorRef;
-import com.google.common.base.Predicate;
-import com.google.common.base.Supplier;
+import akka.util.Timeout;
+import com.google.common.base.Stopwatch;
import com.google.common.collect.ImmutableMap;
import com.google.common.util.concurrent.Uninterruptibles;
+import java.io.OutputStream;
import java.util.ArrayList;
import java.util.Collections;
import java.util.List;
import java.util.Map;
+import java.util.Optional;
import java.util.concurrent.ConcurrentHashMap;
import java.util.concurrent.TimeUnit;
+import java.util.function.Consumer;
+import java.util.function.Predicate;
+import org.apache.commons.lang3.SerializationUtils;
import org.junit.After;
+import org.opendaylight.controller.cluster.raft.MockRaftActor.MockSnapshotState;
import org.opendaylight.controller.cluster.raft.MockRaftActorContext.MockPayload;
-import org.opendaylight.controller.cluster.raft.base.messages.ApplyJournalEntries;
import org.opendaylight.controller.cluster.raft.base.messages.ApplyState;
import org.opendaylight.controller.cluster.raft.base.messages.CaptureSnapshotReply;
import org.opendaylight.controller.cluster.raft.base.messages.SendHeartBeat;
import org.opendaylight.controller.cluster.raft.behaviors.RaftActorBehavior;
+import org.opendaylight.controller.cluster.raft.client.messages.GetOnDemandRaftState;
+import org.opendaylight.controller.cluster.raft.client.messages.OnDemandRaftState;
+import org.opendaylight.controller.cluster.raft.persisted.ApplyJournalEntries;
+import org.opendaylight.controller.cluster.raft.persisted.ServerConfigurationPayload;
+import org.opendaylight.controller.cluster.raft.persisted.Snapshot;
import org.opendaylight.controller.cluster.raft.protobuff.client.messages.Payload;
import org.opendaylight.controller.cluster.raft.utils.InMemoryJournal;
import org.opendaylight.controller.cluster.raft.utils.InMemorySnapshotStore;
import org.opendaylight.controller.cluster.raft.utils.MessageCollectorActor;
+import org.opendaylight.yangtools.concepts.Identifier;
+import org.opendaylight.yangtools.util.AbstractStringIdentifier;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
+import scala.concurrent.Await;
import scala.concurrent.duration.FiniteDuration;
/**
*/
public abstract class AbstractRaftActorIntegrationTest extends AbstractActorTest {
+ private static final class MockIdentifier extends AbstractStringIdentifier<MockIdentifier> {
+ private static final long serialVersionUID = 1L;
+
+ protected MockIdentifier(String string) {
+ super(string);
+ }
+ }
+
public static class SetPeerAddress {
private final String peerId;
private final String peerAddress;
public static class TestRaftActor extends MockRaftActor {
private final TestActorRef<MessageCollectorActor> collectorActor;
- private final Map<Class<?>, Boolean> dropMessages = new ConcurrentHashMap<>();
+ private final Map<Class<?>, Predicate<?>> dropMessages = new ConcurrentHashMap<>();
private TestRaftActor(Builder builder) {
super(builder);
this.collectorActor = builder.collectorActor;
}
- void startDropMessages(Class<?> msgClass) {
- dropMessages.put(msgClass, Boolean.TRUE);
+ public void startDropMessages(Class<?> msgClass) {
+ dropMessages.put(msgClass, msg -> true);
+ }
+
+ <T> void startDropMessages(Class<T> msgClass, Predicate<T> filter) {
+ dropMessages.put(msgClass, filter);
}
- void stopDropMessages(Class<?> msgClass) {
+ public void stopDropMessages(Class<?> msgClass) {
dropMessages.remove(msgClass);
}
void setMockTotalMemory(final long mockTotalMemory) {
- if(mockTotalMemory > 0) {
- getRaftActorContext().setTotalMemoryRetriever(new Supplier<Long>() {
- @Override
- public Long get() {
- return mockTotalMemory;
- }
-
- });
- } else {
- getRaftActorContext().setTotalMemoryRetriever(null);
- }
+ getRaftActorContext().setTotalMemoryRetriever(mockTotalMemory > 0 ? () -> mockTotalMemory : null);
}
+ @SuppressWarnings({ "rawtypes", "unchecked", "checkstyle:IllegalCatch" })
@Override
public void handleCommand(Object message) {
- if(message instanceof MockPayload) {
- MockPayload payload = (MockPayload)message;
- super.persistData(collectorActor, payload.toString(), payload);
+ if (message instanceof MockPayload) {
+ MockPayload payload = (MockPayload) message;
+ super.persistData(collectorActor, new MockIdentifier(payload.toString()), payload, false);
return;
}
- if(message instanceof SetPeerAddress) {
+ if (message instanceof ServerConfigurationPayload) {
+ super.persistData(collectorActor, new MockIdentifier("serverConfig"), (Payload) message, false);
+ return;
+ }
+
+ if (message instanceof SetPeerAddress) {
setPeerAddress(((SetPeerAddress) message).getPeerId().toString(),
((SetPeerAddress) message).getPeerAddress());
return;
}
try {
- if(!dropMessages.containsKey(message.getClass())) {
+ Predicate drop = dropMessages.get(message.getClass());
+ if (drop == null || !drop.test(message)) {
super.handleCommand(message);
}
} finally {
- if(!(message instanceof SendHeartBeat)) {
+ if (!(message instanceof SendHeartBeat)) {
try {
collectorActor.tell(message, ActorRef.noSender());
} catch (Exception e) {
}
@Override
- public void createSnapshot(ActorRef actorRef) {
- try {
- actorRef.tell(new CaptureSnapshotReply(RaftActorTest.fromObject(getState()).toByteArray()), actorRef);
- } catch (Exception e) {
- e.printStackTrace();
+ @SuppressWarnings("checkstyle:IllegalCatch")
+ public void createSnapshot(ActorRef actorRef, Optional<OutputStream> installSnapshotStream) {
+ MockSnapshotState snapshotState = new MockSnapshotState(new ArrayList<>(getState()));
+ if (installSnapshotStream.isPresent()) {
+ SerializationUtils.serialize(snapshotState, installSnapshotStream.get());
}
+
+ actorRef.tell(new CaptureSnapshotReply(snapshotState, installSnapshotStream), actorRef);
}
public ActorRef collectorActor() {
public static class Builder extends AbstractBuilder<Builder, TestRaftActor> {
private TestActorRef<MessageCollectorActor> collectorActor;
- public Builder collectorActor(TestActorRef<MessageCollectorActor> collectorActor) {
- this.collectorActor = collectorActor;
+ public Builder collectorActor(TestActorRef<MessageCollectorActor> newCollectorActor) {
+ this.collectorActor = newCollectorActor;
return this;
}
}
}
+ protected static final int SNAPSHOT_CHUNK_SIZE = 100;
+
protected final Logger testLog = LoggerFactory.getLogger(getClass());
protected final TestActorFactory factory = new TestActorFactory(getSystem());
protected DefaultConfigParamsImpl newLeaderConfigParams() {
DefaultConfigParamsImpl configParams = new DefaultConfigParamsImpl();
configParams.setHeartBeatInterval(new FiniteDuration(100, TimeUnit.MILLISECONDS));
- configParams.setElectionTimeoutFactor(1);
+ configParams.setElectionTimeoutFactor(4);
configParams.setSnapshotBatchCount(snapshotBatchCount);
configParams.setSnapshotDataThresholdPercentage(70);
configParams.setIsolatedLeaderCheckInterval(new FiniteDuration(1, TimeUnit.DAYS));
+ configParams.setSnapshotChunkSize(SNAPSHOT_CHUNK_SIZE);
return configParams;
}
RaftActorTestKit.waitUntilLeader(actorRef);
}
- protected TestActorRef<TestRaftActor> newTestRaftActor(String id, Map<String, String> peerAddresses,
+ protected TestActorRef<TestRaftActor> newTestRaftActor(String id, Map<String, String> newPeerAddresses,
ConfigParams configParams) {
- return newTestRaftActor(id, TestRaftActor.newBuilder().peerAddresses(peerAddresses != null ? peerAddresses :
- Collections.<String, String>emptyMap()).config(configParams));
+ return newTestRaftActor(id, TestRaftActor.newBuilder().peerAddresses(newPeerAddresses != null
+ ? newPeerAddresses : Collections.<String, String>emptyMap()).config(configParams));
}
protected TestActorRef<TestRaftActor> newTestRaftActor(String id, TestRaftActor.Builder builder) {
factory.generateActorId(id + "-collector"))).id(id);
InvalidActorNameException lastEx = null;
- for(int i = 0; i < 10; i++) {
+ for (int i = 0; i < 10; i++) {
try {
- return factory.createTestActor(builder.props().withDispatcher(Dispatchers.DefaultDispatcherId()), id);
+ return factory.createTestActor(builder.props().withDispatcher(Dispatchers.DefaultDispatcherId())
+ .withMailbox(Mailboxes.DefaultMailboxId()), id);
} catch (InvalidActorNameException e) {
lastEx = e;
Uninterruptibles.sleepUninterruptibly(100, TimeUnit.MILLISECONDS);
throw lastEx;
}
- protected void killActor(TestActorRef<TestRaftActor> leaderActor) {
+ protected void killActor(TestActorRef<TestRaftActor> actor) {
JavaTestKit testkit = new JavaTestKit(getSystem());
- testkit.watch(leaderActor);
+ testkit.watch(actor);
- leaderActor.tell(PoisonPill.getInstance(), null);
+ actor.tell(PoisonPill.getInstance(), null);
testkit.expectMsgClass(JavaTestKit.duration("5 seconds"), Terminated.class);
- testkit.unwatch(leaderActor);
+ testkit.unwatch(actor);
}
protected void verifyApplyJournalEntries(ActorRef actor, final long expIndex) {
- MessageCollectorActor.expectFirstMatching(actor, ApplyJournalEntries.class, new Predicate<ApplyJournalEntries>() {
- @Override
- public boolean apply(ApplyJournalEntries msg) {
- return msg.getToIndex() == expIndex;
- }
- });
+ MessageCollectorActor.expectFirstMatching(actor, ApplyJournalEntries.class,
+ msg -> msg.getToIndex() == expIndex);
}
- @SuppressWarnings("unchecked")
protected void verifySnapshot(String prefix, Snapshot snapshot, long lastAppliedTerm,
long lastAppliedIndex, long lastTerm, long lastIndex)
throws Exception {
assertEquals(prefix + " Snapshot getLastTerm", lastTerm, snapshot.getLastTerm());
assertEquals(prefix + " Snapshot getLastIndex", lastIndex, snapshot.getLastIndex());
- List<Object> actualState = (List<Object>)MockRaftActor.toObject(snapshot.getState());
+ List<Object> actualState = ((MockSnapshotState)snapshot.getState()).getState();
assertEquals(String.format("%s Snapshot getState size. Expected %s: . Actual: %s", prefix, expSnapshotState,
actualState), expSnapshotState.size(), actualState.size());
- for(int i = 0; i < expSnapshotState.size(); i++) {
+ for (int i = 0; i < expSnapshotState.size(); i++) {
assertEquals(prefix + " Snapshot state " + i, expSnapshotState.get(i), actualState.get(i));
}
}
protected void verifyPersistedJournal(String persistenceId, List<? extends ReplicatedLogEntry> expJournal) {
List<ReplicatedLogEntry> journal = InMemoryJournal.get(persistenceId, ReplicatedLogEntry.class);
assertEquals("Journal ReplicatedLogEntry count", expJournal.size(), journal.size());
- for(int i = 0; i < expJournal.size(); i++) {
+ for (int i = 0; i < expJournal.size(); i++) {
ReplicatedLogEntry expected = expJournal.get(i);
ReplicatedLogEntry actual = journal.get(i);
verifyReplicatedLogEntry(expected, actual.getTerm(), actual.getIndex(), actual.getData());
}
}
- protected MockPayload sendPayloadData(ActorRef leaderActor, String data) {
- return sendPayloadData(leaderActor, data, 0);
+ protected MockPayload sendPayloadData(ActorRef actor, String data) {
+ return sendPayloadData(actor, data, 0);
}
- protected MockPayload sendPayloadData(ActorRef leaderActor, String data, int size) {
+ protected MockPayload sendPayloadData(ActorRef actor, String data, int size) {
MockPayload payload;
- if(size > 0) {
+ if (size > 0) {
payload = new MockPayload(data, size);
} else {
payload = new MockPayload(data);
}
- leaderActor.tell(payload, ActorRef.noSender());
+ actor.tell(payload, ActorRef.noSender());
return payload;
}
protected void verifyApplyState(ApplyState applyState, ActorRef expClientActor,
- String expId, long expTerm, long expIndex, MockPayload payload) {
+ String expId, long expTerm, long expIndex, Payload payload) {
assertEquals("ApplyState getClientActor", expClientActor, applyState.getClientActor());
- assertEquals("ApplyState getIdentifier", expId, applyState.getIdentifier());
+
+ final Identifier id = expId == null ? null : new MockIdentifier(expId);
+ assertEquals("ApplyState getIdentifier", id, applyState.getIdentifier());
ReplicatedLogEntry replicatedLogEntry = applyState.getReplicatedLogEntry();
verifyReplicatedLogEntry(replicatedLogEntry, expTerm, expIndex, payload);
}
assertEquals("ReplicatedLogEntry getData", payload, replicatedLogEntry.getData());
}
- protected String testActorPath(String id){
+ protected String testActorPath(String id) {
return factory.createTestActorPath(id);
}
assertEquals(name + " replicatedToAllIndex", replicatedToAllIndex,
actor.getCurrentBehavior().getReplicatedToAllIndex());
}
+
+ @SuppressWarnings("checkstyle:IllegalCatch")
+ static void verifyRaftState(ActorRef raftActor, Consumer<OnDemandRaftState> verifier) {
+ Timeout timeout = new Timeout(500, TimeUnit.MILLISECONDS);
+ AssertionError lastError = null;
+ Stopwatch sw = Stopwatch.createStarted();
+ while (sw.elapsed(TimeUnit.SECONDS) <= 5) {
+ try {
+ OnDemandRaftState raftState = (OnDemandRaftState)Await.result(ask(raftActor,
+ GetOnDemandRaftState.INSTANCE, timeout), timeout.duration());
+ verifier.accept(raftState);
+ return;
+ } catch (AssertionError e) {
+ lastError = e;
+ Uninterruptibles.sleepUninterruptibly(50, TimeUnit.MILLISECONDS);
+ } catch (Exception e) {
+ lastError = new AssertionError("OnDemandRaftState failed", e);
+ Uninterruptibles.sleepUninterruptibly(50, TimeUnit.MILLISECONDS);
+ }
+ }
+
+ throw lastError;
+ }
}
Code Review / controller.git / blobdiff