Skip to content

feat: add optional update_to model mapping when disabling a model - #1123

Open
aadirajput-dev26 wants to merge 1 commit into
Walkover-Web-Solution:testingfrom
aadirajput-dev26:feature/gtwy-523-specific-model-update-on-disabling
Open

feat: add optional update_to model mapping when disabling a model#1123
aadirajput-dev26 wants to merge 1 commit into
Walkover-Web-Solution:testingfrom
aadirajput-dev26:feature/gtwy-523-specific-model-update-on-disabling

Conversation

@aadirajput-dev26

Copy link
Copy Markdown
Contributor

Summary

Added support for specifying a fallback model while disabling an existing model through the model status admin API.

Changes
Added an optional update_to field to the model status API.
update_to is only accepted when status = 0 (model disable).
When disabling a model, admins can specify which model should be used as its replacement.
update_to is forbidden when status = 1 (model enable).
Added request validation to enforce the above behavior.

@windsurf-bot

windsurf-bot Bot commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

PR review rate limit exceeded

@Natwar589

Copy link
Copy Markdown
Collaborator

Your card was not found on the dbdash with the branch feature/gtwy-523-specific-model-update-on-disabling

usageInfo = await ConfigurationServices.findIdsByModelAndService(model_name, service, null);

const defaultModel = new_agent_service[service]?.model;
const defaultModel = update_to ? update_to : new_agent_service[service]?.model;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

model validation?

@harshhsahu
harshhsahu force-pushed the testing branch 2 times, most recently from 3f6ca9d to 593f1f4 Compare June 30, 2026 11:30
@aadirajput-dev26
aadirajput-dev26 force-pushed the feature/gtwy-523-specific-model-update-on-disabling branch from fc21e52 to 03df8b0 Compare July 4, 2026 05:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants