[SERVER-68921] Handle multiversion scenario when returning 'InvalidNamespace' error in timeseries_collmod.js Created: 17/Aug/22  Updated: 29/Oct/23  Resolved: 25/Aug/22

Status: Closed
Project: Core Server
Component/s: None
Affects Version/s: None
Fix Version/s: 6.1.1, 6.2.0-rc0

Type: Bug Priority: Major - P3
Reporter: Fausto Leyva (Inactive) Assignee: Dianna Hohensee (Inactive)
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Backports
Depends
Backwards Compatibility: Fully Compatible
Operating System: ALL
Backport Requested:
v6.1
Steps To Reproduce:
  1. start up a mongod instance using a 6.0 binary
  2. run the mongo shell using a 6.1 binary
  3. run 'collMod' command with a 'system.buckets' NS and the command will not fail
Sprint: Execution Team 2022-09-05
Participants:
Linked BF Score: 19

 Description   

In SERVER-64791, we disallow calling the "collMod" command with a 'system.buckets.' collection namespace.

We also added a test case expecting an 'InvalidNamespace' error for the scenario above but if the server is on version 6.0, the command will work when it should have failed. 



 Comments   
Comment by Githook User [ 14/Oct/22 ]

Author:

{'name': 'Dianna Hohensee', 'email': 'dianna.hohensee@mongodb.com', 'username': 'DiannaHohensee'}

Message: SERVER-68921 Add requires_fcv_61 to timeseries_collmod.js for multiversion compatibility

(cherry picked from commit 42aef32217428c8d349ceb80ef62d077eafb835e)
Branch: v6.1
https://github.com/mongodb/mongo/commit/a146b706963b7e1fc08860bca25c41e016930ead

Comment by Liubov Molchanova [ 31/Aug/22 ]

Requesting backport as the issue reproduced on v6.1 in BFG-1446898

Comment by Githook User [ 25/Aug/22 ]

Author:

{'name': 'Dianna Hohensee', 'email': 'dianna.hohensee@mongodb.com', 'username': 'DiannaHohensee'}

Message: SERVER-68921 Add requires_fcv_61 to timeseries_collmod.js for multiversion compatibility
Branch: master
https://github.com/mongodb/mongo/commit/42aef32217428c8d349ceb80ef62d077eafb835e

Comment by Dianna Hohensee (Inactive) [ 18/Aug/22 ]

I'm not actually familiar with SERVER-66934 and it has little documentation: did it remove the internal {collMod: time-series-namespace} uses? I was blocked initially with the final SERVER-64791 solution by sharding internally converting the collMod namespace from timeseries to the underlying buckets.

Looks like my SERVER-64791 changes tripped a multi version w/ v6.0 test suite incompatibility. Yeah, I wouldn't backport the collMod craziness for a BF: too complicated whether we want what changes in v6.0. I'd recommend more simply disabling the test with 6.0.

Comment by Fausto Leyva (Inactive) [ 18/Aug/22 ]

dianna.hohensee@mongodb.com is it feasible to backport SERVER-64791 to 6.0? It looks like that ticket was enabled by SERVER-66934 which was not backported, so I don't think so, but just checking since the changes in SERVER-64791 look like they prevent bad behaviour. 

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