[PR #1466] [MERGED] Make chart idempotent. #7475

New Issue

GiteaMirror · 2025-11-11T17:27:36-06:00

GiteaMirror commented

2025-11-11 17:27:36 -06:00

📋 Pull Request Information

Original PR: https://github.com/open-webui/open-webui/pull/1466
Author: @jmferrer
Created: 4/9/2024
Status: ✅ Merged
Merged: 4/9/2024
Merged by: @tjbck

Base: dev ← Head: make-chart-idempotent

📝 Commits (1)

d6275ee Make chart idempotent.

📊 Changes

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

View changed files

📝 kubernetes/helm/templates/webui-pvc.yaml (+2 -0)

📄 Description

Pull Request Checklist

Description: Fix problem that makes deployment fail when the chart is upgraded and storageClass is not configured, probably because you want to use default storageClass
Changelog: Done
Documentation: No documentation
Dependencies: No dependencies

Description

Fix this problem when helm upgrade --install is launched twice:

Error: UPGRADE FAILED: cannot patch "open-webui" with kind PersistentVolumeClaim: PersistentVolumeClaim "open-webui" is invalid: spec: Forbidden: spec is immutable after creation except resources.requests for bound claims
  core.PersistentVolumeClaimSpec{
  	... // 2 identical fields
  	Resources:        {Requests: {s"storage": {i: {...}, s: "2Gi", Format: "BinarySI"}}},
  	VolumeName:       "pvc-6316c4dc-cc28-44bc-96cc-8717f9015b9e",
- 	StorageClassName: &"microk8s-hostpath",
+ 	StorageClassName: nil,
  	VolumeMode:       &"Filesystem",
  	DataSource:       nil,
  	DataSourceRef:    nil,
  }

Changelog Entry

Fixed

pvc management is not idempotent

Changed

Force storageClass name only if is configured in values.yaml

_{🔄 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/open-webui/pull/1466 **Author:** [@jmferrer](https://github.com/jmferrer) **Created:** 4/9/2024 **Status:** ✅ Merged **Merged:** 4/9/2024 **Merged by:** [@tjbck](https://github.com/tjbck) **Base:** `dev` ← **Head:** `make-chart-idempotent` --- ### 📝 Commits (1) - [`d6275ee`](https://github.com/open-webui/open-webui/commit/d6275ee94137f049255498a86886899e67b90ee8) Make chart idempotent. ### 📊 Changes **1 file changed** (+2 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `kubernetes/helm/templates/webui-pvc.yaml` (+2 -0) </details> ### 📄 Description ## Pull Request Checklist - [ ] **Description:** Fix problem that makes deployment fail when the chart is upgraded and storageClass is not configured, probably because you want to use default storageClass - [ ] **Changelog:** Done - [ ] **Documentation:** No documentation - [ ] **Dependencies:** No dependencies --- ## Description Fix this problem when `helm upgrade --install` is launched twice: ``` Error: UPGRADE FAILED: cannot patch "open-webui" with kind PersistentVolumeClaim: PersistentVolumeClaim "open-webui" is invalid: spec: Forbidden: spec is immutable after creation except resources.requests for bound claims core.PersistentVolumeClaimSpec{ ... // 2 identical fields Resources: {Requests: {s"storage": {i: {...}, s: "2Gi", Format: "BinarySI"}}}, VolumeName: "pvc-6316c4dc-cc28-44bc-96cc-8717f9015b9e", - StorageClassName: &"microk8s-hostpath", + StorageClassName: nil, VolumeMode: &"Filesystem", DataSource: nil, DataSourceRef: nil, } ``` --- ### Changelog Entry ### Fixed - pvc management is not idempotent ### Changed - Force storageClass name only if is configured in values.yaml --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

GiteaMirror added the pull-request label 2025-11-11 17:27:36 -06:00

GiteaMirror closed this issue

2025-11-11 17:27:36 -06:00

GiteaMirror referenced this issue

2025-11-11 18:08:05 -06:00

[PR #7475] [MERGED] 0.4.7 #8861

GiteaMirror referenced this issue

2025-11-11 18:08:26 -06:00

[PR #7536] [MERGED] style: Add proper favicon and iOS home screen support #8875

Sign in to join this conversation.