X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?p=controller.git;a=blobdiff_plain;f=opendaylight%2Fmd-sal%2Fsal-akka-raft%2Fsrc%2Ftest%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Fraft%2FMockRaftActor.java;h=9c58e6b7ca10400614a7eec6a3a2585e97522153;hp=c1a87e8c7b283e1f83b9114471dba911426127cb;hb=b4bf55727093657662d8c16a50fa85f87978a586;hpb=135129e0cca66040fd512fab740d59b2ab1f8382 diff --git a/opendaylight/md-sal/sal-akka-raft/src/test/java/org/opendaylight/controller/cluster/raft/MockRaftActor.java b/opendaylight/md-sal/sal-akka-raft/src/test/java/org/opendaylight/controller/cluster/raft/MockRaftActor.java index c1a87e8c7b..9c58e6b7ca 100644 --- a/opendaylight/md-sal/sal-akka-raft/src/test/java/org/opendaylight/controller/cluster/raft/MockRaftActor.java +++ b/opendaylight/md-sal/sal-akka-raft/src/test/java/org/opendaylight/controller/cluster/raft/MockRaftActor.java @@ -10,24 +10,27 @@ package org.opendaylight.controller.cluster.raft; import static org.junit.Assert.assertEquals; import static org.mockito.Mockito.mock; + import akka.actor.ActorRef; import akka.actor.Props; import com.google.common.base.Function; import com.google.common.base.Optional; +import com.google.common.io.ByteSource; import com.google.common.util.concurrent.Uninterruptibles; -import java.io.ByteArrayInputStream; import java.io.IOException; -import java.io.ObjectInputStream; +import java.io.OutputStream; import java.util.ArrayList; import java.util.Collections; import java.util.List; import java.util.Map; import java.util.concurrent.CountDownLatch; import java.util.concurrent.TimeUnit; -import javax.annotation.Nonnull; +import org.apache.commons.lang3.SerializationUtils; import org.opendaylight.controller.cluster.DataPersistenceProvider; import org.opendaylight.controller.cluster.raft.behaviors.RaftActorBehavior; +import org.opendaylight.controller.cluster.raft.persisted.Snapshot; import org.opendaylight.controller.cluster.raft.protobuff.client.messages.Payload; +import org.opendaylight.yangtools.concepts.Identifier; public class MockRaftActor extends RaftActor implements RaftActorRecoveryCohort, RaftActorSnapshotCohort { public static final short PAYLOAD_VERSION = 5; @@ -41,18 +44,21 @@ public class MockRaftActor extends RaftActor implements RaftActorRecoveryCohort, protected final CountDownLatch initializeBehaviorComplete = new CountDownLatch(1); private RaftActorRecoverySupport raftActorRecoverySupport; private RaftActorSnapshotMessageSupport snapshotMessageSupport; - private final byte[] restoreFromSnapshot; + private final Snapshot restoreFromSnapshot; final CountDownLatch snapshotCommitted = new CountDownLatch(1); private final Function pauseLeaderFunction; - protected MockRaftActor(AbstractBuilder builder) { - super(builder.id, builder.peerAddresses, Optional.fromNullable(builder.config), PAYLOAD_VERSION); + protected MockRaftActor(final AbstractBuilder builder) { + super(builder.id, builder.peerAddresses != null ? builder.peerAddresses : + Collections.emptyMap(), Optional.fromNullable(builder.config), PAYLOAD_VERSION); state = new ArrayList<>(); this.actorDelegate = mock(RaftActor.class); this.recoveryCohortDelegate = mock(RaftActorRecoveryCohort.class); - this.snapshotCohortDelegate = mock(RaftActorSnapshotCohort.class); - if(builder.dataPersistenceProvider == null){ + this.snapshotCohortDelegate = builder.snapshotCohort != null ? builder.snapshotCohort : + mock(RaftActorSnapshotCohort.class); + + if (builder.dataPersistenceProvider == null) { setPersistence(builder.persistent.isPresent() ? builder.persistent.get() : true); } else { setPersistence(builder.dataPersistenceProvider); @@ -64,7 +70,7 @@ public class MockRaftActor extends RaftActor implements RaftActorRecoveryCohort, pauseLeaderFunction = builder.pauseLeaderFunction; } - public void setRaftActorRecoverySupport(RaftActorRecoverySupport support) { + public void setRaftActorRecoverySupport(final RaftActorRecoverySupport support) { raftActorRecoverySupport = support; } @@ -79,6 +85,11 @@ public class MockRaftActor extends RaftActor implements RaftActorRecoveryCohort, (snapshotMessageSupport = super.newRaftActorSnapshotMessageSupport()); } + @Override + public RaftActorContext getRaftActorContext() { + return super.getRaftActorContext(); + } + public RaftActorSnapshotMessageSupport getSnapshotMessageSupport() { return snapshotMessageSupport; } @@ -87,7 +98,7 @@ public class MockRaftActor extends RaftActor implements RaftActorRecoveryCohort, try { assertEquals("Recovery complete", true, recoveryComplete.await(5, TimeUnit.SECONDS)); } catch (InterruptedException e) { - e.printStackTrace(); + throw new RuntimeException(e); } } @@ -95,14 +106,14 @@ public class MockRaftActor extends RaftActor implements RaftActorRecoveryCohort, try { assertEquals("Behavior initialized", true, initializeBehaviorComplete.await(5, TimeUnit.SECONDS)); } catch (InterruptedException e) { - e.printStackTrace(); + throw new RuntimeException(e); } } - public void waitUntilLeader(){ - for(int i = 0;i < 10; i++){ - if(isLeader()){ + public void waitUntilLeader() { + for (int i = 0; i < 10; i++) { + if (isLeader()) { break; } Uninterruptibles.sleepUninterruptibly(100, TimeUnit.MILLISECONDS); @@ -113,8 +124,8 @@ public class MockRaftActor extends RaftActor implements RaftActorRecoveryCohort, return state; } - - @Override protected void applyState(ActorRef clientActor, String identifier, Object data) { + @Override + protected void applyState(final ActorRef clientActor, final Identifier identifier, final Object data) { actorDelegate.applyState(clientActor, identifier, data); LOG.info("{}: applyState called: {}", persistenceId(), data); @@ -122,7 +133,6 @@ public class MockRaftActor extends RaftActor implements RaftActorRecoveryCohort, } @Override - @Nonnull protected RaftActorRecoveryCohort getRaftActorRecoveryCohort() { return this; } @@ -133,11 +143,11 @@ public class MockRaftActor extends RaftActor implements RaftActorRecoveryCohort, } @Override - public void startLogRecoveryBatch(int maxBatchSize) { + public void startLogRecoveryBatch(final int maxBatchSize) { } @Override - public void appendRecoveredLogEntry(Payload data) { + public void appendRecoveredLogEntry(final Payload data) { state.add(data); } @@ -158,33 +168,38 @@ public class MockRaftActor extends RaftActor implements RaftActorRecoveryCohort, } @Override - public void applyRecoverySnapshot(byte[] bytes) { - recoveryCohortDelegate.applyRecoverySnapshot(bytes); - applySnapshotBytes(bytes); + public void applyRecoverySnapshot(final Snapshot.State newState) { + recoveryCohortDelegate.applyRecoverySnapshot(newState); + applySnapshotState(newState); } - private void applySnapshotBytes(byte[] bytes) { - try { - Object data = toObject(bytes); - if (data instanceof List) { - state.addAll((List) data); - } - } catch (Exception e) { - e.printStackTrace(); + private void applySnapshotState(final Snapshot.State newState) { + if (newState instanceof MockSnapshotState) { + state.clear(); + state.addAll(((MockSnapshotState)newState).getState()); } } @Override - public void createSnapshot(ActorRef actorRef) { + public void createSnapshot(final ActorRef actorRef, final java.util.Optional installSnapshotStream) { LOG.info("{}: createSnapshot called", persistenceId()); - snapshotCohortDelegate.createSnapshot(actorRef); + snapshotCohortDelegate.createSnapshot(actorRef, installSnapshotStream); } @Override - public void applySnapshot(byte [] snapshot) { + public void applySnapshot(final Snapshot.State newState) { LOG.info("{}: applySnapshot called", persistenceId()); - applySnapshotBytes(snapshot); - snapshotCohortDelegate.applySnapshot(snapshot); + applySnapshotState(newState); + snapshotCohortDelegate.applySnapshot(newState); + } + + @Override + public Snapshot.State deserializeSnapshot(final ByteSource snapshotBytes) { + try { + return (Snapshot.State) SerializationUtils.deserialize(snapshotBytes.read()); + } catch (IOException e) { + throw new RuntimeException("Error deserializing state", e); + } } @Override @@ -201,69 +216,57 @@ public class MockRaftActor extends RaftActor implements RaftActorRecoveryCohort, return this.getId(); } - protected void newBehavior(RaftActorBehavior newBehavior) { + protected void newBehavior(final RaftActorBehavior newBehavior) { self().tell(newBehavior, ActorRef.noSender()); } @Override - public void handleCommand(final Object message) { - if(message instanceof RaftActorBehavior) { + protected void handleCommand(final Object message) { + if (message instanceof RaftActorBehavior) { super.changeCurrentBehavior((RaftActorBehavior)message); } else { super.handleCommand(message); - if(RaftActorSnapshotMessageSupport.COMMIT_SNAPSHOT.equals(message)) { + if (RaftActorSnapshotMessageSupport.COMMIT_SNAPSHOT.equals(message)) { snapshotCommitted.countDown(); } } } @Override - protected void pauseLeader(Runnable operation) { - if(pauseLeaderFunction != null) { + protected void pauseLeader(final Runnable operation) { + if (pauseLeaderFunction != null) { pauseLeaderFunction.apply(operation); } else { super.pauseLeader(operation); } } - public static Object toObject(byte[] bs) throws ClassNotFoundException, IOException { - Object obj = null; - ByteArrayInputStream bis = null; - ObjectInputStream ois = null; - try { - bis = new ByteArrayInputStream(bs); - ois = new ObjectInputStream(bis); - obj = ois.readObject(); - } finally { - if (bis != null) { - bis.close(); - } - if (ois != null) { - ois.close(); - } + public static List fromState(final Snapshot.State from) { + if (from instanceof MockSnapshotState) { + return ((MockSnapshotState)from).getState(); } - return obj; + + throw new IllegalStateException("Unexpected snapshot State: " + from); } - public ReplicatedLog getReplicatedLog(){ + public ReplicatedLog getReplicatedLog() { return this.getRaftActorContext().getReplicatedLog(); } @Override - public byte[] getRestoreFromSnapshot() { + public Snapshot getRestoreFromSnapshot() { return restoreFromSnapshot; } - public static Props props(final String id, final Map peerAddresses, - ConfigParams config){ + public static Props props(final String id, final Map peerAddresses, final ConfigParams config) { return builder().id(id).peerAddresses(peerAddresses).config(config).props(); } public static Props props(final String id, final Map peerAddresses, - ConfigParams config, DataPersistenceProvider dataPersistenceProvider){ - return builder().id(id).peerAddresses(peerAddresses).config(config). - dataPersistenceProvider(dataPersistenceProvider).props(); + final ConfigParams config, final DataPersistenceProvider dataPersistenceProvider) { + return builder().id(id).peerAddresses(peerAddresses).config(config) + .dataPersistenceProvider(dataPersistenceProvider).props(); } public static Builder builder() { @@ -277,12 +280,13 @@ public class MockRaftActor extends RaftActor implements RaftActorRecoveryCohort, private DataPersistenceProvider dataPersistenceProvider; private ActorRef roleChangeNotifier; private RaftActorSnapshotMessageSupport snapshotMessageSupport; - private byte[] restoreFromSnapshot; + private Snapshot restoreFromSnapshot; private Optional persistent = Optional.absent(); private final Class actorClass; private Function pauseLeaderFunction; + private RaftActorSnapshotCohort snapshotCohort; - protected AbstractBuilder(Class actorClass) { + protected AbstractBuilder(final Class actorClass) { this.actorClass = actorClass; } @@ -291,48 +295,53 @@ public class MockRaftActor extends RaftActor implements RaftActorRecoveryCohort, return (T) this; } - public T id(String id) { - this.id = id; + public T id(final String newId) { + this.id = newId; return self(); } - public T peerAddresses(Map peerAddresses) { - this.peerAddresses = peerAddresses; + public T peerAddresses(final Map newPeerAddresses) { + this.peerAddresses = newPeerAddresses; return self(); } - public T config(ConfigParams config) { - this.config = config; + public T config(final ConfigParams newConfig) { + this.config = newConfig; return self(); } - public T dataPersistenceProvider(DataPersistenceProvider dataPersistenceProvider) { - this.dataPersistenceProvider = dataPersistenceProvider; + public T dataPersistenceProvider(final DataPersistenceProvider newDataPersistenceProvider) { + this.dataPersistenceProvider = newDataPersistenceProvider; return self(); } - public T roleChangeNotifier(ActorRef roleChangeNotifier) { - this.roleChangeNotifier = roleChangeNotifier; + public T roleChangeNotifier(final ActorRef newRoleChangeNotifier) { + this.roleChangeNotifier = newRoleChangeNotifier; return self(); } - public T snapshotMessageSupport(RaftActorSnapshotMessageSupport snapshotMessageSupport) { - this.snapshotMessageSupport = snapshotMessageSupport; + public T snapshotMessageSupport(final RaftActorSnapshotMessageSupport newSnapshotMessageSupport) { + this.snapshotMessageSupport = newSnapshotMessageSupport; return self(); } - public T restoreFromSnapshot(byte[] restoreFromSnapshot) { - this.restoreFromSnapshot = restoreFromSnapshot; + public T restoreFromSnapshot(final Snapshot newRestoreFromSnapshot) { + this.restoreFromSnapshot = newRestoreFromSnapshot; return self(); } - public T persistent(Optional persistent) { - this.persistent = persistent; + public T persistent(final Optional newPersistent) { + this.persistent = newPersistent; return self(); } - public T pauseLeaderFunction(Function pauseLeaderFunction) { - this.pauseLeaderFunction = pauseLeaderFunction; + public T pauseLeaderFunction(final Function newPauseLeaderFunction) { + this.pauseLeaderFunction = newPauseLeaderFunction; + return self(); + } + + public T snapshotCohort(final RaftActorSnapshotCohort newSnapshotCohort) { + this.snapshotCohort = newSnapshotCohort; return self(); } @@ -342,8 +351,57 @@ public class MockRaftActor extends RaftActor implements RaftActorRecoveryCohort, } public static class Builder extends AbstractBuilder { - private Builder() { + Builder() { super(MockRaftActor.class); } } + + public static class MockSnapshotState implements Snapshot.State { + private static final long serialVersionUID = 1L; + + private final List state; + + public MockSnapshotState(final List state) { + this.state = state; + } + + public List getState() { + return state; + } + + @Override + public int hashCode() { + final int prime = 31; + int result = 1; + result = prime * result + (state == null ? 0 : state.hashCode()); + return result; + } + + @Override + public boolean equals(final Object obj) { + if (this == obj) { + return true; + } + if (obj == null) { + return false; + } + if (getClass() != obj.getClass()) { + return false; + } + MockSnapshotState other = (MockSnapshotState) obj; + if (state == null) { + if (other.state != null) { + return false; + } + } else if (!state.equals(other.state)) { + return false; + } + return true; + } + + @Override + public String toString() { + return "MockSnapshotState [state=" + state + "]"; + } + } }