-
Notifications
You must be signed in to change notification settings - Fork 2.5k
Closed
Labels
Description
Describe the bug
With #17611, we refactored the way feature flags work. I'm trying to onboard the neural search plugin to use the new feature flag constructs but it does not work today because we check against all registered feature flags in core before we set the value. Since plugins do not register their feature flags to core this breaks while checking
void set(String featureFlagName, Boolean value) {
for(Setting<Boolean> ff : this.featureFlags.keySet()) {
if (ff.getKey().equals(featureFlagName)) {
this.featureFlags.put(ff, value);
}
}
}
Related component
Plugins
To Reproduce
Trying to run tests in neural search
Expected behavior
Tests should pass
Additional Details
Plugins
Please list all plugins currently enabled.
Screenshots
If applicable, add screenshots to help explain your problem.
Host/Environment (please complete the following information):
- OS: [e.g. iOS]
- Version [e.g. 22]
Additional context
Add any other context about the problem here.
Reactions are currently unavailable