[PR #672] [MERGED] fix: Allow specifying an external dburl in helm #1099

Closed
opened 2025-10-09 19:08:51 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/plankanban/planka/pull/672
Author: @bstinsonmhk
Created: 4/4/2024
Status: Merged
Merged: 4/5/2024
Merged by: @meltyshev

Base: masterHead: helm-db-value


📝 Commits (1)

  • 3d3f3c6 fix: Allow specifying an external dburl in helm

📊 Changes

3 files changed (+9 additions, -2 deletions)

View changed files

📝 charts/planka/Chart.yaml (+1 -1)
📝 charts/planka/templates/deployment.yaml (+5 -0)
📝 charts/planka/values.yaml (+3 -1)

📄 Description

Some helm users might want to turn off the built-in postgres installation, but the deployment template statically refers to a secret created by the postgresql chart.

If postgres.enabled is set to false, we should ask for a dburl parameter for an external postgres and use that in the environment instead.

The behavior if postgres.enabled == true remains unchanged.


🔄 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/plankanban/planka/pull/672 **Author:** [@bstinsonmhk](https://github.com/bstinsonmhk) **Created:** 4/4/2024 **Status:** ✅ Merged **Merged:** 4/5/2024 **Merged by:** [@meltyshev](https://github.com/meltyshev) **Base:** `master` ← **Head:** `helm-db-value` --- ### 📝 Commits (1) - [`3d3f3c6`](https://github.com/plankanban/planka/commit/3d3f3c67859f6c5dd5ce26dfd9c4518b9a43606c) fix: Allow specifying an external dburl in helm ### 📊 Changes **3 files changed** (+9 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `charts/planka/Chart.yaml` (+1 -1) 📝 `charts/planka/templates/deployment.yaml` (+5 -0) 📝 `charts/planka/values.yaml` (+3 -1) </details> ### 📄 Description Some helm users might want to turn off the built-in postgres installation, but the deployment template statically refers to a secret created by the postgresql chart. If postgres.enabled is set to false, we should ask for a dburl parameter for an external postgres and use that in the environment instead. The behavior if postgres.enabled == true remains unchanged. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2025-10-09 19:08:51 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/planka#1099