webgrip/invoiceninja-application

Fork 0

[PR #7] [CLOSED] Generate Helm charts using community Invoice Ninja chart with complete values.yaml #46

New issue

Closed

opened 2026-06-08 09:15:48 +00:00 by ryangr0 · 0 comments

ryangr0 commented

2026-06-08 09:15:48 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/webgrip/invoiceninja-application/pull/7
Author: @Copilot
Created: 9/3/2025
Status: ❌ Closed

Base: main ← Head: copilot/fix-5

📝 Commits (2)

3253342 Initial plan
4376858 Implement Invoice Ninja Helm charts using community chart

📊 Changes

16 files changed (+585 additions, -56 deletions)

View changed files

➕ ops/helm/README.md (+159 -0)
➖ ops/helm/application-application/Chart.lock (+0 -15)
➖ ops/helm/application-application/Chart.yaml (+0 -20)
➕ ops/helm/invoiceninja-application/Chart.lock (+12 -0)
➕ ops/helm/invoiceninja-application/Chart.yaml (+15 -0)
➕ ops/helm/invoiceninja-application/charts/invoiceninja (+1 -0)
➕ ops/helm/invoiceninja-application/templates/_helpers.tpl (+19 -0)
➕ ops/helm/invoiceninja-application/values.yaml (+171 -0)
➕ ops/helm/invoiceninja-application/values.yaml.backup (+167 -0)
📝 ops/helm/invoiceninja-application/values.yaml.old (+0 -0)
➖ ops/secrets/application-application-secrets/templates/secret.yaml (+0 -13)
➖ ops/secrets/application-application-secrets/values.dec.yaml.example (+0 -8)
📝 ops/secrets/invoiceninja-application-secrets/Chart.yaml (+0 -0)
➕ ops/secrets/invoiceninja-application-secrets/templates/secret.yaml (+19 -0)
➕ ops/secrets/invoiceninja-application-secrets/values.dec.yaml.example (+14 -0)
➕ ops/secrets/invoiceninja-application-secrets/values.yaml (+8 -0)

📄 Description

This PR replaces the existing bwj app-template based Helm charts with new charts that use the actively maintained community Invoice Ninja chart as a dependency, providing a complete values.yaml with all configurable options ready to fill.

Changes Made

Main Chart: `ops/helm/invoiceninja-application/`

Replaced dependency: Changed from bjw-s-labs/app-template to the community Saddamus/invoiceninja-helm chart
Complete values.yaml: Exposes all upstream chart configuration options with defaults and clear # REQUIRED placeholders
Organization image policy: Uses webgrip/invoiceninja-application:latest following our standards
Standard integrations: Configured with Traefik ingress + cert-manager for TLS
Proper dependencies: Includes MySQL 14.0.3, Redis 22.0.7, and common-helpers

Secrets Chart: `ops/secrets/invoiceninja-application-secrets/`

Install-first pattern: Designed to be deployed before the main chart
Complete secret management: Handles all sensitive configuration (database passwords, app keys, mail credentials)
Clear documentation: values.dec.yaml.example with detailed REQUIRED field descriptions

Community Chart Integration

Source: Uses Saddamus/invoiceninja-helm (actively maintained community chart)
Version: Pinned to v0.1.0 with Invoice Ninja app version 5.0.52
Integration method: Packaged as local subchart to ensure dependency availability
No modifications: Used as-is to maintain future update compatibility

Configuration Required

Operators must fill these fields before deployment:

Domain & URLs: _shared_config.hostname, _shared_config.url, ingress hosts
Database: invoiceninja.dbpass, matching secret values
Application: APP_KEY (Laravel generated), APP_URL
Mail: SMTP configuration (host, username, password, from address)
Secrets: Database passwords, app key, mail password

Validation

Both charts pass all required validations:

✅ helm lint passes for both charts
✅ helm template generates valid Kubernetes manifests
✅ All upstream chart options exposed with defaults
✅ 16 REQUIRED fields clearly marked in main chart
✅ 4 REQUIRED secret fields documented

Installation Order

Install secrets: helm install invoiceninja-application-secrets ops/secrets/invoiceninja-application-secrets/
Install application: helm install invoiceninja-application ops/helm/invoiceninja-application/

Complete installation documentation is provided in ops/helm/README.md.

Fixes #5.

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

_{🔄 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/webgrip/invoiceninja-application/pull/7 **Author:** [@Copilot](https://github.com/apps/copilot-swe-agent) **Created:** 9/3/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `copilot/fix-5` --- ### 📝 Commits (2) - [`3253342`](https://github.com/webgrip/invoiceninja-application/commit/3253342602151845b5b966b3fdfd9a59d0eafb70) Initial plan - [`4376858`](https://github.com/webgrip/invoiceninja-application/commit/437685844b68b8d20998bf18a0bf0dcdc5177b43) Implement Invoice Ninja Helm charts using community chart ### 📊 Changes **16 files changed** (+585 additions, -56 deletions) <details> <summary>View changed files</summary> ➕ `ops/helm/README.md` (+159 -0) ➖ `ops/helm/application-application/Chart.lock` (+0 -15) ➖ `ops/helm/application-application/Chart.yaml` (+0 -20) ➕ `ops/helm/invoiceninja-application/Chart.lock` (+12 -0) ➕ `ops/helm/invoiceninja-application/Chart.yaml` (+15 -0) ➕ `ops/helm/invoiceninja-application/charts/invoiceninja` (+1 -0) ➕ `ops/helm/invoiceninja-application/templates/_helpers.tpl` (+19 -0) ➕ `ops/helm/invoiceninja-application/values.yaml` (+171 -0) ➕ `ops/helm/invoiceninja-application/values.yaml.backup` (+167 -0) 📝 `ops/helm/invoiceninja-application/values.yaml.old` (+0 -0) ➖ `ops/secrets/application-application-secrets/templates/secret.yaml` (+0 -13) ➖ `ops/secrets/application-application-secrets/values.dec.yaml.example` (+0 -8) 📝 `ops/secrets/invoiceninja-application-secrets/Chart.yaml` (+0 -0) ➕ `ops/secrets/invoiceninja-application-secrets/templates/secret.yaml` (+19 -0) ➕ `ops/secrets/invoiceninja-application-secrets/values.dec.yaml.example` (+14 -0) ➕ `ops/secrets/invoiceninja-application-secrets/values.yaml` (+8 -0) </details> ### 📄 Description This PR replaces the existing bwj app-template based Helm charts with new charts that use the actively maintained community Invoice Ninja chart as a dependency, providing a complete `values.yaml` with all configurable options ready to fill. ## Changes Made ### Main Chart: `ops/helm/invoiceninja-application/` - **Replaced dependency**: Changed from `bjw-s-labs/app-template` to the community `Saddamus/invoiceninja-helm` chart - **Complete values.yaml**: Exposes all upstream chart configuration options with defaults and clear `# REQUIRED` placeholders - **Organization image policy**: Uses `webgrip/invoiceninja-application:latest` following our standards - **Standard integrations**: Configured with Traefik ingress + cert-manager for TLS - **Proper dependencies**: Includes MySQL 14.0.3, Redis 22.0.7, and common-helpers ### Secrets Chart: `ops/secrets/invoiceninja-application-secrets/` - **Install-first pattern**: Designed to be deployed before the main chart - **Complete secret management**: Handles all sensitive configuration (database passwords, app keys, mail credentials) - **Clear documentation**: `values.dec.yaml.example` with detailed REQUIRED field descriptions ### Community Chart Integration - **Source**: Uses `Saddamus/invoiceninja-helm` (actively maintained community chart) - **Version**: Pinned to v0.1.0 with Invoice Ninja app version 5.0.52 - **Integration method**: Packaged as local subchart to ensure dependency availability - **No modifications**: Used as-is to maintain future update compatibility ## Configuration Required Operators must fill these fields before deployment: **Domain & URLs**: `_shared_config.hostname`, `_shared_config.url`, ingress hosts **Database**: `invoiceninja.dbpass`, matching secret values **Application**: `APP_KEY` (Laravel generated), `APP_URL` **Mail**: SMTP configuration (host, username, password, from address) **Secrets**: Database passwords, app key, mail password ## Validation Both charts pass all required validations: - ✅ `helm lint` passes for both charts - ✅ `helm template` generates valid Kubernetes manifests - ✅ All upstream chart options exposed with defaults - ✅ 16 REQUIRED fields clearly marked in main chart - ✅ 4 REQUIRED secret fields documented ## Installation Order 1. Install secrets: `helm install invoiceninja-application-secrets ops/secrets/invoiceninja-application-secrets/` 2. Install application: `helm install invoiceninja-application ops/helm/invoiceninja-application/` Complete installation documentation is provided in `ops/helm/README.md`. Fixes #5.  --- ✨ Let Copilot coding agent [set things up for you](https://github.com/webgrip/invoiceninja-application/issues/new?title=✨+Set+up+Copilot+instructions&body=Configure%20instructions%20for%20this%20repository%20as%20documented%20in%20%5BBest%20practices%20for%20Copilot%20coding%20agent%20in%20your%20repository%5D%28https://gh.io/copilot-coding-agent-tips%29%2E%0A%0A%3COnboard%20this%20repo%3E&assignees=copilot) — coding agent works faster and does higher quality work when set up for your repo. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

ryangr0

2026-06-08 09:15:48 +00:00

closed this issue
added the
pull-request
label

No labels

pull-request

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

webgrip/invoiceninja-application#46

No description provided.

Rows
Columns