diff options
author | Tor Brede Vekterli <vekterli@verizonmedia.com> | 2019-05-21 13:16:08 +0000 |
---|---|---|
committer | Tor Brede Vekterli <vekterli@verizonmedia.com> | 2019-05-21 13:23:15 +0000 |
commit | 20cd5dda7e3adda277b0d116075c89c3886dc26e (patch) | |
tree | 8650aa281eca845d3eee8d7f84f6bb939f366bff /storage/src/tests/distributor/CMakeLists.txt | |
parent | fb212d0abb8ed773106cbdadd68e9800b257e44a (diff) |
Avoid resurrecting replicas for nodes that are unavailable in pending state
We previously only checked for node availability in the _active_ state
without looking at the pending state. This opened up for a race condition
where a reply for a previously DB-pruned node could bring a replica back
in the DB iff received during a pending state window.
Consider Maintenance as unavailable for this case, not just Down.
Also move all `PutOperation` tests to GTest.
Diffstat (limited to 'storage/src/tests/distributor/CMakeLists.txt')
-rw-r--r-- | storage/src/tests/distributor/CMakeLists.txt | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/storage/src/tests/distributor/CMakeLists.txt b/storage/src/tests/distributor/CMakeLists.txt index fdccf9b1394..245d54e8e69 100644 --- a/storage/src/tests/distributor/CMakeLists.txt +++ b/storage/src/tests/distributor/CMakeLists.txt @@ -24,7 +24,6 @@ vespa_add_library(storage_testdistributor TEST ownership_transfer_safe_time_point_calculator_test.cpp pendingmessagetrackertest.cpp persistence_metrics_set_test.cpp - putoperationtest.cpp removebucketoperationtest.cpp removelocationtest.cpp removeoperationtest.cpp @@ -50,6 +49,7 @@ vespa_add_library(storage_gtestdistributor TEST bucketdatabasetest.cpp bucketdbupdatertest.cpp mapbucketdatabasetest.cpp + putoperationtest.cpp # Fixture etc. dupes with non-gtest runner : distributortestutil.cpp bucket_db_prune_elision_test.cpp |