From 1d742cdba3ee1ff554cb90e985fe5d640cae0e42 Mon Sep 17 00:00:00 2001 From: Janna Golden Date: Wed, 22 Oct 2025 13:45:43 -0400 Subject: [PATCH] SERVER-112617 Set failpoint before starting chunk migration in safe_secondary_reads_causal_consistency.js (#42884) GitOrigin-RevId: 6fe6cef1a906e7f0db9d9a64cdd16f378d75b26e --- .github/CODEOWNERS | 1 + jstests/sharding/OWNERS.yml | 3 +++ jstests/sharding/safe_secondary_reads_causal_consistency.js | 2 +- 3 files changed, 5 insertions(+), 1 deletion(-) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index 59cfcd288f8..2cebc261906 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -1556,6 +1556,7 @@ WORKSPACE.bazel @10gen/devprod-build @svc-auto-approve-bot /jstests/sharding/**/safe_secondary_reads_single_migration_waitForDelete.js @10gen/server-catalog-and-routing @svc-auto-approve-bot /jstests/sharding/**/startup_with_all_configs_down.js @10gen/server-catalog-and-routing-routing-and-topology @svc-auto-approve-bot /jstests/sharding/**/addshard*.js @10gen/server-catalog-and-routing-routing-and-topology @svc-auto-approve-bot +/jstests/sharding/**/safe_secondary_reads_causal_consistency.js @10gen/server-cluster-scalability @svc-auto-approve-bot # The following patterns are parsed from ./jstests/sharding/analyze_shard_key/OWNERS.yml /jstests/sharding/analyze_shard_key/**/* @10gen/server-cluster-scalability @svc-auto-approve-bot diff --git a/jstests/sharding/OWNERS.yml b/jstests/sharding/OWNERS.yml index e42060995bc..ea81fdfe729 100644 --- a/jstests/sharding/OWNERS.yml +++ b/jstests/sharding/OWNERS.yml @@ -130,3 +130,6 @@ filters: - "addshard*.js": approvers: - 10gen/server-catalog-and-routing-routing-and-topology + - "safe_secondary_reads_causal_consistency.js": + approvers: + - 10gen/server-cluster-scalability diff --git a/jstests/sharding/safe_secondary_reads_causal_consistency.js b/jstests/sharding/safe_secondary_reads_causal_consistency.js index e6cba38cb02..76a80cf45de 100644 --- a/jstests/sharding/safe_secondary_reads_causal_consistency.js +++ b/jstests/sharding/safe_secondary_reads_causal_consistency.js @@ -100,6 +100,7 @@ setupShardedCollection(st, dbName2, ns2, st.shard1.shardName); // shard1. Then it pauses the migration right before it enters the commit phase. jsTest.log("Starting migration."); +pauseMoveChunkAtStep(st.shard0, moveChunkStepNames.chunkDataCommitted); const staticMongod = MongoRunner.runMongod({}); const joinMoveChunk = moveChunkParallel( @@ -111,7 +112,6 @@ const joinMoveChunk = moveChunkParallel( st.shard1.shardName, true /** expectSuccess */, ); -pauseMoveChunkAtStep(st.shard0, moveChunkStepNames.chunkDataCommitted); waitForMoveChunkStep(st.shard0, moveChunkStepNames.chunkDataCommitted); // Sends a versioned read through mongos1, the second router, to cause a refresh on the secondary of