Loading
Remove ai_gateway_multi_default_models and duo_workflow_provider_stickiness feature flags
What does this MR do and why?
Removes two fully-rolled-out feature flags and all their references in application code and specs:
ai_gateway_multi_default_models
- Deleted the YAML definition (
config/feature_flags/gitlab_com_derisk/ai_gateway_multi_default_models.yml) - Removed
Gitlab::AiGateway.push_feature_flag(:ai_gateway_multi_default_models, ...)calls from:ee/app/services/ai/model_selection/fetch_model_definitions_service.rbee/lib/gitlab/llm/completions/chat.rbee/lib/api/code_suggestions.rbee/lib/api/helpers/duo_workflow_helpers.rb
- Simplified the cache key in
FetchModelDefinitionsServiceto useRESPONSE_CACHE_NAMEdirectly (removed the flag-state suffix per @alejandro's instruction)
duo_workflow_provider_stickiness
- Deleted the YAML definition (
ee/config/feature_flags/gitlab_com_derisk/duo_workflow_provider_stickiness.yml) - Made provider stickiness (reusing stored
model_metadata_jsonfrom a workflow) the unconditional default inee/lib/api/ai/duo_workflows/workflows.rb
All specs updated to reflect the now-default behavior.
References
Edited by Nick Leonard