scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-10-06 02:48:13 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 103

fix(team_member_update): update existing budget in-place instead of creating new one

Browse source 
When a member already has a budget_id, patch only the fields the caller
provided rather than always creating a fresh budget record.  The old
code ignored existing_budget_id entirely, so updating only allowed_models
silently dropped the stored max_budget / tpm_limit / rpm_limit values.
This commit is contained in:
Ishaan Jaffer 2026-04-01 16:19:45 -07:00
parent 1814fd64b0
commit 48ecd59aa5
1 changed files with 21 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
litellm/proxy/management_endpoints/common_utils.py
View file

@ -385,7 +385,27 @@ async def _upsert_budget_and_membership(
)
return
# create a new budget
if existing_budget_id is not None:
# Update the existing budget in-place to preserve fields not being changed.
# Only write fields that the caller explicitly provided (non-None).
update_data: Dict[str, Any] = {
"updated_by": user_api_key_dict.user_id or "",
}
if max_budget is not None:
update_data["max_budget"] = max_budget
if tpm_limit is not None:
update_data["tpm_limit"] = tpm_limit
if rpm_limit is not None:
update_data["rpm_limit"] = rpm_limit
if allowed_models is not None:
update_data["allowed_models"] = allowed_models
await tx.litellm_budgettable.update(
where={"budget_id": existing_budget_id},
data=update_data,
)
return
# No existing budget — create a new one and link it to the membership.
create_data: Dict[str, Any] = {
"created_by": user_api_key_dict.user_id or "",
"updated_by": user_api_key_dict.user_id or "",
@ -403,7 +423,6 @@ async def _upsert_budget_and_membership(
data=create_data,
include={"team_membership": True},
)
# upsert the team membership with the new/updated budget
await tx.litellm_teammembership.upsert(
where={
"user_id_team_id": {