[PR #136] [MERGED] Fix 'helm-release' CI to disabling chart release for forked repos #208

New Issue

2026-02-15T19:16:52-05:00

yindo commented

2026-02-15 19:16:52 -05:00

📋 Pull Request Information

Original PR: https://github.com/open-webui/helm-charts/pull/136
Author: @jyje
Created: 1/7/2025
Status: ✅ Merged
Merged: 1/12/2025
Merged by: @westbrook-ai

Base: main ← Head: fix/skip-forked-release

📝 Commits (1)

5ae7d88 fix(ci): skip release actions for forked repositories

📊 Changes

1 file changed (+1 additions, -0 deletions)

View changed files

📝 .github/workflows/helm-release.yml (+1 -0)

📄 Description

Hello, folks. I'd like to suggest disabling chart releases for forked repositories by default.

After forking this repository and enabling Actions, I noticed that the main branch's workflow attempted to release charts using unconfigured or void variables, causing the pipeline to fail. This behavior seems common across other repositories with similar setups.

The proposed change is simple: it adds a repository slug check to ensure the workflow only proceeds in the original repository unless explicitly configured otherwise. This helps prevent unnecessary actions and failures in forked repositories. If someone wants to release forked chart, they can comment the if: ... line.

But it’s a simple but significant change, so rejecting this is totally okay. Please check it, and any feedback would be welcome.

_{🔄 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/136 **Author:** [@jyje](https://github.com/jyje) **Created:** 1/7/2025 **Status:** ✅ Merged **Merged:** 1/12/2025 **Merged by:** [@westbrook-ai](https://github.com/westbrook-ai) **Base:** `main` ← **Head:** `fix/skip-forked-release` --- ### 📝 Commits (1) - [`5ae7d88`](https://github.com/open-webui/helm-charts/commit/5ae7d88c6de4f6152a09cd60cf6aa2995d77c8ef) fix(ci): skip release actions for forked repositories ### 📊 Changes **1 file changed** (+1 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/helm-release.yml` (+1 -0) </details> ### 📄 Description Hello, folks. I'd like to suggest disabling chart releases for forked repositories by default. After forking this repository and enabling Actions, I noticed that the `main` branch's workflow attempted to release charts using unconfigured or void variables, causing the pipeline to fail. This behavior seems common across other repositories with similar setups. The proposed change is simple: it adds a repository slug check to ensure the workflow only proceeds in the original repository unless explicitly configured otherwise. This helps prevent unnecessary actions and failures in forked repositories. If someone wants to release forked chart, they can comment the `if: ...` line. But it’s a simple but significant change, so rejecting this is totally okay. Please check it, and any feedback would be welcome. --- <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:52 -05:00

yindo closed this issue

2026-02-15 19:16:52 -05:00

yindo referenced this issue

2026-02-15 19:17:04 -05:00

[PR #208] [MERGED] Add support for SSO/OIDC #250

Sign in to join this conversation.