Always save to local copy in IdIntsListener 47/78347/2
authorTom Pantelis <tompantelis@gmail.com>
Sun, 2 Dec 2018 20:49:16 +0000 (15:49 -0500)
committerTom Pantelis <tompantelis@gmail.com>
Tue, 4 Dec 2018 17:06:33 +0000 (17:06 +0000)
commit78de7c0dbbb387a46c12f3e28efa15030efa6e62
treeb48db5d9b6d1f97797a7e48b7cbc27f13c1f6d43
parent4ecb8ecaf04594b3312a44d801423f515ea445b3
Always save to local copy in IdIntsListener

The Listener Stability CSIT fails intermittently due to the
IdIntsListener reporting that its local view of the data
tree as captured from DTCNs does not match the final read
of the data after the writes are done and the leader change
is complete. On DTCN, the IdIntsListener only stores the new
data if the data before in the DTCN matches its stored copy of
the data from the previous DTCN. When the test fails, this
check fails early on and after that it will never match. A
prior patch added a diff of the data nodes that failed the check
which shows that the id-int entry for "prefix-1" was missing
in the stored copy. This is added at the start of the test
(each node adds its own entry). It's unclear to me if it's
actually valid to ensure that the stored copy matches the
current DTCN's data before so I removed the check - let's see
if there's any further tests failures.

Change-Id: I7b861fc85b42586832072fdc19de103733e4fbf5
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
opendaylight/md-sal/samples/clustering-test-app/provider/src/main/java/org/opendaylight/controller/clustering/it/provider/impl/IdIntsListener.java