commit 37e91a00aa280cb9f2fd3386bad44148761050e0 Author: Nasser Grainawi Date: Thu Oct 15 15:56:12 2020 -0600 Update git submodules * Update plugins/replication from branch 'stable-3.3' to 5ddc976586c8da17fd93e7b3d3ee54122421ec21 - 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 bc47d23..5ddc976 160000 --- a/plugins/replication +++ b/plugins/replication @@ -1 +1 @@ -Subproject commit bc47d238733981d957a820f4c66f7128fca80663 +Subproject commit 5ddc976586c8da17fd93e7b3d3ee54122421ec21