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

Closed
opened 2026-02-15 19:16:46 -05:00 by yindo · 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: @westbrook-ai

Base: mainHead: 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:** [@westbrook-ai](https://github.com/westbrook-ai) **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>
yindo added the pull-request label 2026-02-15 19:16:46 -05:00
yindo closed this issue 2026-02-15 19:16:46 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: open-webui/helm-charts#185