Skip to content

docs: document default_team_params in config reference#25032

Merged
ryan-crabbe-berri merged 2 commits intolitellm_ryan-march-31from
litellm_docs-default-team-params
Apr 3, 2026
Merged

docs: document default_team_params in config reference#25032
ryan-crabbe-berri merged 2 commits intolitellm_ryan-march-31from
litellm_docs-default-team-params

Conversation

@ryan-crabbe-berri
Copy link
Copy Markdown
Collaborator

@ryan-crabbe-berri ryan-crabbe-berri commented Apr 2, 2026

Summary

  • Add default_team_params to the litellm_settings reference table in config_settings.md with all sub-fields documented
  • Update self_serve.md and msft_sso.md config examples to include team_member_permissions, tpm_limit, and rpm_limit
  • Fix misleading comment that implied default_team_params only applies to SSO auto-created teams — it applies to all /team/new calls

Test plan

  • Verify docs render correctly on local Docusaurus dev server

- Add default_team_params to litellm_settings reference table in
  config_settings.md with all sub-fields documented
- Update self_serve.md and msft_sso.md examples to include
  team_member_permissions, tpm_limit, and rpm_limit
- Fix misleading comment that implied default_team_params only applies
  to SSO auto-created teams — it applies to all /team/new calls
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 2, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
litellm Ready Ready Preview, Comment Apr 2, 2026 11:02pm

Request Review

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 2, 2026

Greptile Summary

This is a pure documentation PR that adds default_team_params to the litellm_settings config reference table and updates YAML examples in self_serve.md and msft_sso.md with the full set of supported sub-fields (tpm_limit, rpm_limit, team_member_permissions). It also corrects the previously misleading comment that implied the setting only applied to SSO auto-created teams — it applies to all /team/new calls, with the exception of models, which is correctly documented as SSO-only.

The documentation changes are accurate and consistent with the implementation:

  • max_budget, budget_duration, tpm_limit, rpm_limit, and team_member_permissions are applied via the explicit loop in team_endpoints.py (lines 862–868) for all /team/new calls
  • models is applied only through the _cast_and_deepcopy_litellm_default_team_params SSO path in ui_sso.py, which is correctly annotated in every updated example
  • The KeyManagementRoutes enum confirms that /team/daily/activity and /key/generate are valid route strings for team_member_permissions
  • The budget_duration example value "30d" is a valid format per the proxy _types.py docstring

The previous Greptile finding about models being incorrectly documented as applying to all /team/new calls has been addressed in this PR.

Confidence Score: 5/5

Documentation-only PR; all sub-field descriptions and applicability scopes have been verified against the implementation and are accurate.

No code changes are introduced. All documented sub-fields, their types, and their scope (SSO-only vs. all /team/new calls) are consistent with the source code in team_endpoints.py and ui_sso.py. The prior models-applicability concern raised in the previous Greptile review has been addressed explicitly.

No files require special attention.

Important Files Changed

Filename Overview
docs/my-website/docs/proxy/config_settings.md Adds a new default_team_params row to the litellm_settings reference table; sub-fields and their applicability scope are accurately described and match the source code in team_endpoints.py and ui_sso.py.
docs/my-website/docs/proxy/self_serve.md Updates two YAML code examples to include tpm_limit, rpm_limit, and team_member_permissions; also corrects the misleading comment that implied default_team_params was SSO-only. Minor inconsistency: the second comprehensive config block omits inline comments and the second permission route present in the first example.
docs/my-website/docs/tutorials/msft_sso.md Updates the Microsoft SSO config example to include team_member_permissions, aligning it with the other updated examples. Changes are accurate and non-breaking.

Reviews (2): Last reviewed commit: "docs: clarify that models sub-field only..." | Re-trigger Greptile

| enable_json_schema_validation | boolean | If true, enables json schema validation for all requests. |
| enable_key_alias_format_validation | boolean | If true, validates `key_alias` format on `/key/generate` and `/key/update`. Must be 2-255 chars, start/end with alphanumeric, only allow `a-zA-Z0-9_-/.@`. Default `false`. |
| disable_copilot_system_to_assistant | boolean | **DEPRECATED** - GitHub Copilot API supports system prompts. |
| default_team_params | object | Default parameters applied to every new team created via `/team/new` (including SSO auto-created teams). Only fills in fields not explicitly set in the request. Sub-fields: `max_budget` (float), `budget_duration` (string, e.g. `"30d"`), `models` (array of strings), `tpm_limit` (integer), `rpm_limit` (integer), `team_member_permissions` (array of strings, e.g. `["/team/daily/activity", "/key/generate"]`). |
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P1 models sub-field does not apply to all /team/new calls

The description states that default_team_params "Default parameters applied to every new team created via /team/new", and lists models as a sub-field. However, in litellm/proxy/management_endpoints/team_endpoints.py (lines 862–868), the loop that backfills defaults only iterates over max_budget, budget_duration, tpm_limit, rpm_limit, and team_member_permissionsmodels is deliberately excluded:

for field in (
    "max_budget",
    "budget_duration",
    "tpm_limit",
    "rpm_limit",
    "team_member_permissions",
):

models is applied for SSO-created teams because ui_sso.py copies all DefaultTeamSSOParams fields into the NewTeamRequest before calling new_team(). But a user making a direct POST /team/new request without specifying models will not have models populated from default_team_params.

The documentation should either clarify that the models field only takes effect for SSO auto-created teams, or "models" should be added to the loop in team_endpoints.py to make the behaviour consistent with the documented claim.

The same note should be added to self_serve.md (line ~364) and msft_sso.md (line ~134) wherever models appears under default_team_params.

@ryan-crabbe-berri ryan-crabbe-berri temporarily deployed to integration-postgres April 2, 2026 23:00 — with GitHub Actions Inactive
@ryan-crabbe-berri ryan-crabbe-berri temporarily deployed to integration-redis-postgres April 2, 2026 23:00 — with GitHub Actions Inactive
@ryan-crabbe-berri ryan-crabbe-berri temporarily deployed to integration-postgres April 2, 2026 23:00 — with GitHub Actions Inactive
@ryan-crabbe-berri ryan-crabbe-berri temporarily deployed to integration-postgres April 2, 2026 23:00 — with GitHub Actions Inactive
@ryan-crabbe-berri ryan-crabbe-berri temporarily deployed to integration-postgres April 2, 2026 23:00 — with GitHub Actions Inactive
@ryan-crabbe-berri ryan-crabbe-berri merged commit eb780a8 into litellm_ryan-march-31 Apr 3, 2026
51 of 58 checks passed
@ryan-crabbe-berri ryan-crabbe-berri deleted the litellm_docs-default-team-params branch April 3, 2026 23:07
fede-kamel pushed a commit to fede-kamel/litellm that referenced this pull request Apr 5, 2026
…am-params

docs: document default_team_params in config reference
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.

1 participant