[PR #110] [MERGED] Allow service accounts #208

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

📋 Pull Request Information

Original PR: https://github.com/open-webui/helm-charts/pull/110
Author: @alexnederlof
Created: 11/24/2024
Status: ✅ Merged
Merged: 11/24/2024
Merged by: @james-westbrook

Base: main ← Head: allow-service-accounts


📝 Commits (3)

  • afcd804 Allow service account to be set
  • d0cf4ab Merge branch 'main' into alexnederlof-allow-service-accounts
  • 746fe34 Merge in main, change create to enable for consistency

📊 Changes

4 files changed (+26 additions, -3 deletions)

View changed files

📝 charts/open-webui/Chart.yaml (+1 -1)
➕ charts/open-webui/templates/service-account.yaml (+14 -0)
📝 charts/open-webui/templates/workload-manager.yaml (+5 -2)
📝 charts/open-webui/values.yaml (+6 -0)

📄 Description

I was missing service accounts from the helm chart, so added the default stuff to use those. This helps people running on Amazon EKS that use this to authenticate against AWS services.

Tested it by running helm template . with and without the config.

This change is backwards compatible (ie, it's a no-op if you don't enable serviceAccount)

@0xThresh can you take a look?


🔄 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/110 **Author:** [@alexnederlof](https://github.com/alexnederlof) **Created:** 11/24/2024 **Status:** ✅ Merged **Merged:** 11/24/2024 **Merged by:** [@james-westbrook](https://github.com/james-westbrook) **Base:** `main` ← **Head:** `allow-service-accounts` --- ### 📝 Commits (3) - [`afcd804`](https://github.com/open-webui/helm-charts/commit/afcd804aaf125fe47b44a96291308a7528686983) Allow service account to be set - [`d0cf4ab`](https://github.com/open-webui/helm-charts/commit/d0cf4abd119d7805f4b22dc04249e59b84b1c8a3) Merge branch 'main' into alexnederlof-allow-service-accounts - [`746fe34`](https://github.com/open-webui/helm-charts/commit/746fe34aa195fd4ce33cde138292462b74bcb716) Merge in main, change create to enable for consistency ### 📊 Changes **4 files changed** (+26 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `charts/open-webui/Chart.yaml` (+1 -1) ➕ `charts/open-webui/templates/service-account.yaml` (+14 -0) 📝 `charts/open-webui/templates/workload-manager.yaml` (+5 -2) 📝 `charts/open-webui/values.yaml` (+6 -0) </details> ### 📄 Description I was missing service accounts from the helm chart, so added the default stuff to use those. This helps people running on Amazon EKS that use this to authenticate against AWS services. Tested it by running `helm template .` with and without the config. This change is backwards compatible (ie, it's a no-op if you don't enable serviceAccount) @0xThresh can you take a look? --- <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:12 +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#208
No description provided.