[SERVER-67268] Investigate simplifying usages of featureFlagXX, requires_fcv_xx and FeatureFlagUtil.isEnabled in JStests Created: 14/Jun/22  Updated: 02/Mar/23  Resolved: 09/Feb/23

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

Type: Task Priority: Major - P3
Reporter: Huayu Ouyang Assignee: Backlog - Replication Team
Resolution: Duplicate Votes: 0
Labels: milestone-2, pm-2821-milestone-1
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Depends
depends on SERVER-71467 Dont run feature flag specific tests ... Closed
Duplicate
duplicates SERVER-69883 Investigate improving JS test tags fo... Open
Related
related to SERVER-71898 Multiversion unreleased feature flagg... Closed
is related to SERVER-70203 Investigate using IDL to exclude JS t... Closed
is related to SERVER-69883 Investigate improving JS test tags fo... Open
Assigned Teams:
Replication
Participants:

 Description   

The requires_fcv_5x and requires_fcv_60 can simply be removed

The requires_fcv_61 tag should be replaced with the appropriate feature flag tag if the test depends on a specific feature flag

Edit: We currently cannot actually stop using requires_fcv_yy since if we have a feature flag that is enabled, (either through running in an all feature flags variant, or through being enabled by default), and we have a test that tests this feature, the featureFlagXX tag would not prevent it from being run on sharding_multiversion, so we should be tagging it with requires_fcv_yy.
The current guidance for feature flag test tagging is here, but it is slightly complicated, so we should investigate simplifying this. Additionally, we should investigate moving away from using FeatureFlagUtil.isEnabled where possible, since it is wasteful to start up a whole cluster only to see that the feature flag is not enabled and then skip the whole test.


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