Details
-
Type:
Improvement
-
Status: Closed
-
Priority:
Major - P3
-
Resolution: Fixed
-
Affects Version/s: None
-
Fix Version/s: 4.3.1
-
Component/s: Querying, Testing Infrastructure
-
Labels:
-
Backwards Compatibility:Fully Compatible
Description
Lines like this one currently exclude profiling tests from the sharded passthroughs:
|
sharded_collections_jscore_passthrough.yml |
- jstests/core/profile*.js # profiling.
|
We added the 'requires_profiling' tag at some point, it looks like it was used for the mobile build variants. I suggest we update the suites to exclude based on the tag rather than based on the name.
We should then exclude this tag from other sharded suites, such as aggregation_mongos_passthrough. That should help us avoid causing failures and trouble by moving tests around and by naming the test something without 'profile', such as this other one.