Conversation
WalkthroughThe MongoDB feature compatibility version was upgraded from 5.x to 6.x in the fixer script. Version comparison thresholds and log messages were updated accordingly to reflect the new target version while maintaining existing control flow logic. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
subrata71
left a comment
There was a problem hiding this comment.
Will this change have any impact on existing customer data for instances that were previously running MongoDB 5.x or any version prior to 6.x?
|
It'll just set the compatibility level to 6 (we're running 6 now) so that we can hop to 7. It does mean that you can't start up an older version of MongoDB using the data directory. But that would also not work once Appsmith starts anyway, so it doesn't matter. |
Description
Looks like when the embedded MongoDB version was last updated, the feature compatibility version parameter was not updated. This means that any instances that started on 1.66 or less using the embedded MongoDB didn't have their compatibility version updated. This'll make the jump to 7.x harder as we need to ensure this is at least 6 in those scenarios. Getting this in place now.
Fixes #
Issue Numberor
Fixes
Issue URLWarning
If no issue exists, please create an issue first, and check with the maintainers if the issue is valid.
Automation
/ok-to-test tags=""
🔍 Cypress test results
Caution
If you modify the content in this section, you are likely to disrupt the CI result for your PR.
Communication
Should the DevRel and Marketing teams inform users about this change?
Summary by CodeRabbit