commit 52361e7e9fb66c3c1f3c0a830406067f28093967 Author: Nasser Grainawi Date: Thu Oct 15 15:30:05 2020 -0600 Update git submodules * Update plugins/replication from branch 'stable-3.2' to ba99e9bce51e2ee8e68b0de98860ee4c29179601 - Merge branch 'stable-3.1' into stable-3.2 * stable-3.1: ReplicationIT: Remove unnecessary storage inspection ReplicationIT: Fix invalid replicationDelay setting Split replication plugins tests in two groups Change-Id: Ib43f744124114097e650e5991489bc87ef087dbd - Merge branch 'stable-3.0' into stable-3.1 * stable-3.0: ReplicationIT: Remove unnecessary storage inspection ReplicationIT: Fix invalid replicationDelay setting Split replication plugins tests in two groups Change-Id: I9dfa3abd4907d74415bff6b77fc9ae49b9f6735f - Merge branch 'stable-2.16' into stable-3.0 * stable-2.16: ReplicationIT: Remove unnecessary storage inspection ReplicationIT: Fix invalid replicationDelay setting Split replication plugins tests in two groups Change-Id: I2d27b715a2bfc9832ee559556d1c8acfe671d893 - ReplicationIT: Remove unnecessary storage inspection Integration tests shouldn't need to rely on inspecting the underlying ReplicationTasksStorage layer(s). All of these tests already verify the expected end result. This leaves 4 tests that currently completely rely on inspecting the task storage to verify the expected result. Those tests need further improvement to decouple from the storage layer. Change-Id: I029d63ce7d07414d9bf5d9290d556378beedcabf - ReplicationIT: Fix invalid replicationDelay setting Setting config values for a remote in replication.config vs the remote's own config file results in the replication.config values being ignored. Fix this by setting the values in each remote's config file. This test had delays added to avoid any flakiness, but the delays weren't working because of this issue. While the test generally passes, the delay makes it safer from races. Change-Id: Idcdf5f07b3fc91724068ec6216527665c4a48bb3 - Split replication plugins tests in two groups Run unit-tests and integration tests in parallel by splitting them into two separate tasks. This also allows to potentially identify which group of tests is flaky, because Bazel would flag one or the other in case of instability. Change-Id: I21f969a17e3653dfc5ab93d71cc6955024fc2d8f diff --git a/plugins/replication b/plugins/replication index 10ba451..ba99e9b 160000 --- a/plugins/replication +++ b/plugins/replication @@ -1 +1 @@ -Subproject commit 10ba4516825e6c97e9bdc6efab2a82581f3edc03 +Subproject commit ba99e9bce51e2ee8e68b0de98860ee4c29179601