[SERVER-61131] Store backup cursor results and set state to "learned filenames" Created: 30/Oct/21  Updated: 29/Oct/23  Resolved: 06/Jan/22

Status: Closed
Project: Core Server
Component/s: None
Affects Version/s: None
Fix Version/s: 5.3.0

Type: Task Priority: Major - P3
Reporter: A. Jesse Jiryu Davis Assignee: Christopher Caplinger
Resolution: Fixed Votes: 0
Labels: shard-merge-milestone-1
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Depends
depends on SERVER-61130 Create backup cursor on donor primary Closed
is depended on by SERVER-62394 Follow up work for SERVER-61131 Closed
is depended on by SERVER-62882 Tenant Migration Recipient Service sh... Closed
is depended on by SERVER-61133 Copy donor files to temp directory Closed
Backwards Compatibility: Fully Compatible
Sprint: Server Serverless 2021-12-27, Server Serverless 2022-01-10
Participants:

 Description   

See design doc for details. Perhaps the donatedFiles.migrationId collection goes in the config database?



 Comments   
Comment by Githook User [ 27/Jan/22 ]

Author:

{'name': 'Christopher Caplinger', 'email': 'christopher.caplinger@mongodb.com', 'username': 'UnicodeSnowman'}

Message: SERVER-62394: Follow-up work for SERVER-61131
Branch: master
https://github.com/mongodb/mongo/commit/fb529499ff831684ee281bd13486b62b2bbcc3a3

Comment by Githook User [ 05/Jan/22 ]

Author:

{'name': 'Christopher Caplinger', 'email': 'christopher.caplinger@mongodb.com', 'username': 'UnicodeSnowman'}

Message: SERVER-61131: Store backup cursor results and set state to 'learned filenames'
Branch: master
https://github.com/mongodb/mongo/commit/3ad8827a2f8763f4ff2eac719e6b0420f0627b33

Comment by A. Jesse Jiryu Davis [ 16/Dec/21 ]

That sounds reasonable to me: manually/visually check that we're storing filenames in a collection, automatically test that we updated the state. Consider adapting your SERVER-61130 test if that's easy.

Comment by Christopher Caplinger [ 16/Dec/21 ]

jesse what do you think I should I do in terms of testing here? testing that we write the backup cursor results feels a bit contrived, since we'll presumably have some more end-to-end oriented tests in upcoming tickets that end up testing this behavior indirectly.

I suppose it could be worth testing that we set the state to "learned filenames" correctly

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