commit 821a046102710589959426c0e75df3e5a38fb5ab Author: Nasser Grainawi Date: Thu Oct 15 15:56:17 2020 -0600 Update git submodules * Update plugins/replication from branch 'master' to 10b616284a846dade09a330896ec76e80df1c68c - Merge branch 'stable-3.3' into master * stable-3.3: ReplicationIT: Remove unnecessary storage inspection ReplicationIT: Fix invalid replicationDelay setting Split replication plugins tests in two groups Change-Id: I287200d0b7bacb6f39349a36a1080766881126fd - Merge branch 'stable-3.2' into stable-3.3 * stable-3.2: ReplicationIT: Remove unnecessary storage inspection ReplicationIT: Fix invalid replicationDelay setting Split replication plugins tests in two groups Change-Id: I24aae9834c518518539dae0e20106c74de1d714b - 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 63fb2b4..10b6162 160000 --- a/plugins/replication +++ b/plugins/replication @@ -1 +1 @@ -Subproject commit 63fb2b4ba85380d798acbdc076e8673353507569 +Subproject commit 10b616284a846dade09a330896ec76e80df1c68c