diff options
-rw-r--r-- | jstests/replsets/tenant_migration_concurrent_migrations.js | 23 |
1 files changed, 16 insertions, 7 deletions
diff --git a/jstests/replsets/tenant_migration_concurrent_migrations.js b/jstests/replsets/tenant_migration_concurrent_migrations.js index 5589676c5bc..eeda516b7d1 100644 --- a/jstests/replsets/tenant_migration_concurrent_migrations.js +++ b/jstests/replsets/tenant_migration_concurrent_migrations.js @@ -64,8 +64,8 @@ const kTenantIdPrefix = "testTenantId"; assert.commandWorked(tenantMigrationTest1.waitForMigrationToComplete(migrationOpts1)); // Verify that both migrations succeeded. - assert(stateRes0.state, TenantMigrationTest.State.kCommitted); - assert(stateRes1.state, TenantMigrationTest.State.kCommitted); + assert.eq(stateRes0.state, TenantMigrationTest.State.kCommitted); + assert.eq(stateRes1.state, TenantMigrationTest.State.kCommitted); const connPoolStatsAfter0 = assert.commandWorked(donorPrimary.adminCommand({connPoolStats: 1})); // Donor targeted two different replica sets. @@ -79,6 +79,13 @@ const kTenantIdPrefix = "testTenantId"; // After migrations are complete, RSMs are garbage collected. const connPoolStatsAfter1 = assert.commandWorked(donorPrimary.adminCommand({connPoolStats: 1})); assert.eq(Object.keys(connPoolStatsAfter1.replicaSets).length, 0); + + // TODO: SERVER-53276 the RSM from recipient rst1 to donor rst0 is not deleted. + // assert.eq( + // Object.keys(assert.commandWorked(rst1.getPrimary().adminCommand({connPoolStats: + // 1})).replicaSets) + // .length, + // 0); })(); // Test concurrent incoming migrations from different donors. @@ -110,8 +117,8 @@ const kTenantIdPrefix = "testTenantId"; assert.commandWorked(tenantMigrationTest1.waitForMigrationToComplete(migrationOpts1)); // Verify that both migrations succeeded. - assert(stateRes0.state, TenantMigrationTest.State.kCommitted); - assert(stateRes1.state, TenantMigrationTest.State.kCommitted); + assert.eq(stateRes0.state, TenantMigrationTest.State.kCommitted); + assert.eq(stateRes1.state, TenantMigrationTest.State.kCommitted); // Cleanup. assert.commandWorked(tenantMigrationTest0.forgetMigration(migrationOpts0.migrationIdString)); @@ -121,9 +128,11 @@ const kTenantIdPrefix = "testTenantId"; assert.commandWorked(rst0.getPrimary().adminCommand({connPoolStats: 1})); assert.eq(Object.keys(connPoolStatsAfter0.replicaSets).length, 0); - const connPoolStatsAfter1 = - assert.commandWorked(rst1.getPrimary().adminCommand({connPoolStats: 1})); - assert.eq(Object.keys(connPoolStatsAfter1.replicaSets).length, 0); + // TODO: SERVER-53276 the RSM from recipient rst1 to donor rst0 is not deleted in the test + // above. + // const connPoolStatsAfter1 = + // assert.commandWorked(rst1.getPrimary().adminCommand({connPoolStats: 1})); + // assert.eq(Object.keys(connPoolStatsAfter1.replicaSets).length, 0); })(); // Test concurrent outgoing migrations to same recipient. Verify that tenant |