Skip to content

Make LiteLLM admin password optional with a generated default#703

Merged
ak684 merged 1 commit into
mainfrom
alona/litellm-admin-password-optional
Jun 11, 2026
Merged

Make LiteLLM admin password optional with a generated default#703
ak684 merged 1 commit into
mainfrom
alona/litellm-admin-password-optional

Conversation

@ak684

@ak684 ak684 commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #690. The litellm_admin_password KOTS field was required with no default, which hard-blocks every upgrade from pre-0.7.44 until the admin invents a password — even on installs that never use the LiteLLM UI (hit on the R03 0.7.34→0.7.44 upgrade, and KOTS required is not satisfied by a default).

Change

  • Drop required: true; generate an initial value with value: '{{repl RandomString 32}}' — the same once-generated-then-sticky pattern the other bundled secrets (jwt_secret, redis_password, keycloak_admin_password) already use.
  • Field stays visible (not hidden) so the no-kubectl flow from Add KOTS LiteLLM admin password #690 is unchanged: an admin who wants the LiteLLM dashboard sets their own password in the KOTS config and deploys — the existing pod checksum annotation restarts openhands-litellm and the new password takes effect.

Behavior

  • Fresh installs / upgrades: no prompt, random password generated and stored in ConfigValues.
  • Worst case if never touched: the LiteLLM UI stays inaccessible to the admin — identical to the pre-Add KOTS LiteLLM admin password #690 status quo; reset-and-deploy is always available.
  • UI_PASSWORD in litellm-env-secrets is now never empty (verified via helm template render), so no chart changes needed.

No chart content changes — replicated/config.yaml only.

@ak684 ak684 requested a review from all-hands-bot June 11, 2026 01:07
@ak684 ak684 merged commit dd02f5e into main Jun 11, 2026
2 checks passed
@ak684 ak684 deleted the alona/litellm-admin-password-optional branch June 11, 2026 01:13
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.

2 participants