Skip to content

fix(litellm): enable model storage#60

Merged
vitramir merged 1 commit into
mainfrom
noa/issue-59
Mar 9, 2026
Merged

fix(litellm): enable model storage#60
vitramir merged 1 commit into
mainfrom
noa/issue-59

Conversation

@casey-brooks
Copy link
Copy Markdown
Contributor

Summary

  • enable LiteLLM model storage in DB by default

Testing

  • NIXPKGS_ALLOW_UNFREE=1 nix shell --impure nixpkgs#terraform -c terraform fmt -check -recursive
  • NIXPKGS_ALLOW_UNFREE=1 nix shell --impure nixpkgs#terraform -c terraform -chdir=stacks/platform validate

Fixes #59

@casey-brooks casey-brooks requested a review from a team as a code owner March 9, 2026 18:11
@casey-brooks
Copy link
Copy Markdown
Contributor Author

Summary

  • set LiteLLM general_settings.store_model_in_db to true

Testing

  • NIXPKGS_ALLOW_UNFREE=1 nix shell --impure nixpkgs#terraform -c terraform fmt -check -recursive
  • NIXPKGS_ALLOW_UNFREE=1 nix shell --impure nixpkgs#terraform -c terraform -chdir=stacks/platform validate

Copy link
Copy Markdown

@noa-lucent noa-lucent left a comment

Choose a reason for hiding this comment

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

Clean, minimal change that directly addresses #59. The store_model_in_db = true addition to general_settings is correctly placed and matches the issue specification exactly. No concerns.

@vitramir vitramir merged commit 9eac1a0 into main Mar 9, 2026
1 check passed
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.

Fix: Enable STORE_MODEL_IN_DB for LiteLLM proxy

3 participants