[SERVER-62154] The tenant_migration.py script should ignore error related to not supported operations during tenant_migration Created: 17/Dec/21  Updated: 04/Jan/22  Resolved: 04/Jan/22

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

Type: Bug Priority: Major - P3
Reporter: Mathis Bessa Assignee: Mathis Bessa
Resolution: Won't Do Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Depends
Related
is related to SERVER-62216 When abortTenantIndexBuilds failed to... Closed
Operating System: ALL
Sprint: Server Serverless 2021-12-27, Server Serverless 2022-01-10
Participants:
Linked BF Score: 15

 Description   

tenant_migration.py has a condition ignore error such as "simulate a tenant migration error".

We should add a condition to this to also ignore if the error contains
"Index creation, except createIndex on empty collections, is not supported in tenant migration"

Since the failure of this BF is related to hooks after the test actually passed, this should resolve that issue.



 Comments   
Comment by Mathis Bessa [ 04/Jan/22 ]

Closing this Jira as no longer needed :

We are going to cancel this PR. Even if we would add a check for this error code, the hook actually threw an exception because the tenant migration thread was not running at the end of the test. This is because the tenant migration thread threw an exception that was not handled due to the createIndex failing to abort. The changes that will be done in SERVER-62216 will fix that error. This will result in the tenant migration thread not throwing and will not resulting in the hook throwing an exception for that test.

Comment by Esha Maharishi (Inactive) [ 04/Jan/22 ]

mathis.bessa sounds good to me to close this, since in SERVER-62216 we decided the donor should try to abort any in-progress index builds, then wait for all index builds to commit or abort.

Comment by Mathis Bessa [ 04/Jan/22 ]

esha.maharishi jesse I think we should cancel this Jira since this was about ignoring the failure. There is an actual bug about index building that we are going to fix with SERVER-62216.

I don't think this Jira is still needed, any thoughts about continuing with the changes here ?

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