Code Review / controller.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 6e76bb4) | patch
Fix sporadic failures in ClusterAdminRpcServiceTest 85/35285/2
authorTom Pantelis <tpanteli@brocade.com>
Tue, 23 Feb 2016 18:10:21 +0000 (13:10 -0500)
committerGerrit Code Review <gerrit@opendaylight.org>
Tue, 23 Feb 2016 21:06:38 +0000 (21:06 +0000)
commit497d356874c8033e0d0f9fccfa7ed6695d6889fc
tree76273967814b8df20a532caf0aa7aaf84e880191tree | snapshot
parent6e76bb44514ea79524f46ff283ea0d0d4ad8c7f8commit | diff
Fix sporadic failures in ClusterAdminRpcServiceTest

The testAddShardReplica is failing sporadically indirectly due to
https://git.opendaylight.org/gerrit/#/c/35097/. The leader's payload
version was -1 so a transaction's BatchedModifications instance was
serialized to the legacy protobuf message type and thus wasn't delivered.

The underlying problem is that RaftActor.updateConfigParams
needs to pass the previous leader's payload version when re-constructing
the Follower instance on RaftPolicy change.

Change-Id: Id6b8da98c145f2e265dddbc1b27384d54e400370
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>
opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/RaftActor.java diff | blob | history
opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/behaviors/Follower.java diff | blob | history
OpenDaylight controller