[SERVER-84422] Remove `movePrimary` calls from tests not needing it Created: 27/Dec/23  Updated: 31/Jan/24  Resolved: 31/Jan/24

Status: Closed
Project: Core Server
Component/s: Sharding
Affects Version/s: None
Fix Version/s: 7.3.0-rc2

Type: Task Priority: Minor - P4
Reporter: Pierlauro Sciarelli Assignee: Pierlauro Sciarelli
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Related
is related to SERVER-81461 Change tests that uses movePrimary to... In Code Review
Backwards Compatibility: Fully Compatible
Sprint: CAR Team 2024-01-08
Participants:

 Description   

There are several tests that could avoid calling movePrimary. Purpose of this ticket is to readapt such tests in the following ways:

  • Tests unnecessarily invoking movePrimary:
    • Remove invocations
  • Tests invoking movePrimary right after enableSharding in order to place unsharded collections on a specific shard:
    • Invoke enableSharding with primaryShard argument


 Comments   
Comment by Githook User [ 17/Jan/24 ]

Author:

{'name': 'Marcos Grillo', 'email': 'marcos.grillo@mongodb.com', 'username': 'm4nti5'}

Message: SERVER-84422 Replace movePrimary calls with enableSharding from tests not needing it (#18100)

GitOrigin-RevId: 69c314a2ae3b1b6cf76e66003f02aca560b6e2d3
Branch: master
https://github.com/mongodb/mongo/commit/e2e519bdda7493d18d5579f77babbb489b2117a7

Comment by Githook User [ 17/Jan/24 ]

Author:

{'name': 'Marcos José Grillo Ramirez', 'email': 'marcos.grillo@mongodb.com', 'username': 'm4nti5'}

Message: Revert "SERVER-84422 Replace `movePrimary` calls with `enableSharding` from tests not needing it (#17927)"

This reverts commit c33b129c714aa222f426f7db65fe5c134c02379a.

GitOrigin-RevId: 5859b820964141244c93051570a9296e5e366d7b
Branch: master
https://github.com/mongodb/mongo/commit/c03e2c16b165a0c7a938d8693b245536667ec892

Comment by Githook User [ 17/Jan/24 ]

Author:

{'name': 'Marcos Grillo', 'email': 'marcos.grillo@mongodb.com', 'username': 'm4nti5'}

Message: SERVER-84422 Replace `movePrimary` calls with `enableSharding` from tests not needing it (#17927)

GitOrigin-RevId: c33b129c714aa222f426f7db65fe5c134c02379a
Branch: master
https://github.com/mongodb/mongo/commit/3abf20b06b3d53bdc127a2f6f14f6165f3e1119b

Comment by Githook User [ 27/Dec/23 ]

Author:

{'name': 'Pierlauro Sciarelli', 'email': 'pierlauro.sciarelli@mongodb.com', 'username': 'pierlauro'}

Message: SERVER-84422 Remove `movePrimary` calls from tests not needing it (#17774)

GitOrigin-RevId: ba30af34d315a11289855c8e09b7fd2155bb8be1
Branch: master
https://github.com/mongodb/mongo/commit/3a627b048e2488a0c2420ce1296324891de5902d

Generated at Thu Feb 08 06:54:59 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.