[PR #88] [MERGED] Move extraEnvVars to Workload from ConfigMap #191

Closed
opened 2026-10-18 21:07:04 +00:00 by bsakaguchi · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/open-webui/helm-charts/pull/88
Author: @saip92
Created: 9/28/2024
Status: ✅ Merged
Merged: 10/2/2024
Merged by: @james-westbrook

Base: main ← Head: saip92/move-env-vars-to-stateful-set


📝 Commits (2)

  • a16cdd4 Move extraEnvVars to Workload from ConfigMap
  • 119fdd2 Update chart version

📊 Changes

5 files changed (+19 additions, -29 deletions)

View changed files

📝 charts/open-webui/Chart.yaml (+1 -1)
📝 charts/open-webui/README.md (+1 -1)
➖ charts/open-webui/templates/configmap.yaml (+0 -22)
📝 charts/open-webui/templates/workload-manager.yaml (+16 -4)
📝 charts/open-webui/values.yaml (+1 -1)

📄 Description

The extraEnvVars value in the open-webui chart is inserted into a config map which is later mounted in the workload (deployment/stateful set) for its environment variables. But this does not support referencing secrets which are needed when using features like OIDC Authentication or PostgreSQL as the database.

This PR removes the config map and moves the environment variables directly into the workload itself.

fixes #86
fixes #87


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/open-webui/helm-charts/pull/88 **Author:** [@saip92](https://github.com/saip92) **Created:** 9/28/2024 **Status:** ✅ Merged **Merged:** 10/2/2024 **Merged by:** [@james-westbrook](https://github.com/james-westbrook) **Base:** `main` ← **Head:** `saip92/move-env-vars-to-stateful-set` --- ### 📝 Commits (2) - [`a16cdd4`](https://github.com/open-webui/helm-charts/commit/a16cdd4477789920ed4d00c598699d6d2dcb36f2) Move extraEnvVars to Workload from ConfigMap - [`119fdd2`](https://github.com/open-webui/helm-charts/commit/119fdd2f2db7c274a77d07ead89b3f41aed3c277) Update chart version ### 📊 Changes **5 files changed** (+19 additions, -29 deletions) <details> <summary>View changed files</summary> 📝 `charts/open-webui/Chart.yaml` (+1 -1) 📝 `charts/open-webui/README.md` (+1 -1) ➖ `charts/open-webui/templates/configmap.yaml` (+0 -22) 📝 `charts/open-webui/templates/workload-manager.yaml` (+16 -4) 📝 `charts/open-webui/values.yaml` (+1 -1) </details> ### 📄 Description The `extraEnvVars` value in the open-webui chart is inserted into a config map which is later mounted in the workload (deployment/stateful set) for its environment variables. But this does not support referencing secrets which are needed when using features like OIDC Authentication or PostgreSQL as the database. This PR removes the config map and moves the environment variables directly into the workload itself. fixes #86 fixes #87 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
bsakaguchi 2026-10-18 21:07:04 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
open-webui/helm-charts#191
No description provided.