[SERVER-46060] Add test that FCV 4.4 config server expects commitChunkMigration to contain a ChunkVersion Created: 10/Feb/20  Updated: 29/Oct/23  Resolved: 05/Mar/20

Status: Closed
Project: Core Server
Component/s: Sharding
Affects Version/s: None
Fix Version/s: 4.4.0-rc0, 4.7.0

Type: Task Priority: Major - P3
Reporter: Esha Maharishi (Inactive) Assignee: Alexander Taskov (Inactive)
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Backports
Backwards Compatibility: Fully Compatible
Backport Requested:
v4.4
Sprint: Sharding 2020-03-09
Participants:

 Description   

This ticket is to add test coverage for this assertion in ShardingCatalogManager::checkChunkMatchesRequest.

It could be done through a unit test or integration test. We have unit these two related unit test files:

https://github.com/mongodb/mongo/blob/master/src/mongo/db/s/config/sharding_catalog_manager_split_chunk_test.cpp

https://github.com/mongodb/mongo/blob/master/src/mongo/db/s/config/sharding_catalog_manager_merge_chunks_test.cpp

that call methods on ShardingCatalogManager, so it might make sense to create an analogous sharding_catalog_manager_move_chunks_test.cpp.

Alternatively, it could be tested in an integration test by directly sending commitChunkMigration to the config server without a ChunkVersion when the config server is in FCV 4.4. SeeĀ this class that represents a commiitChunkMigration request.



 Comments   
Comment by Max Hirschhorn [ 01/Apr/20 ]

The changes from 3a4b19a were backported to the 4.4 branch together along with the changes from 3cbe84a from SERVER-46061.


Author:

{'name': 'Alex Taskov', 'username': 'alextaskov', 'email': 'alex.taskov@mongodb.com'}

Message: SERVER-46061 Add test that a lagged commitChunkMigration that contains a ChunkVersion is rejected if the chunk's version has been bumped

(cherry picked from commit 3cbe84a6a68a87942a9e040c44a4cc41f46f4e86)
Branch: v4.4
https://github.com/mongodb/mongo/commit/238c485dbbbdb370312ace5c32bcaf5370ad0647

Comment by Githook User [ 05/Mar/20 ]

Author:

{'username': 'alextaskov', 'name': 'Alex Taskov', 'email': 'alex.taskov@mongodb.com'}

Message: SERVER-46060 Add test that FCV 4.4 config server expects commitChunkMigration to contain a ChunkVersion
Branch: master
https://github.com/mongodb/mongo/commit/3a4b19a0d5dd1fb10dc6a8d8f85566b48b946702

Generated at Thu Feb 08 05:10:23 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.