Skip to content

Chart refactor with breaking changes#126

Merged
johnjeffers merged 38 commits into
mainfrom
jj/chart-cleanup
Jun 5, 2026
Merged

Chart refactor with breaking changes#126
johnjeffers merged 38 commits into
mainfrom
jj/chart-cleanup

Conversation

@johnjeffers

@johnjeffers johnjeffers commented May 23, 2026

Copy link
Copy Markdown
Contributor

This PR hardens and reorganizes the FusionAuth Helm chart:

  • Splits chart helper logic into focused database, deployment, labels, search, and init-container templates.
  • Adds structured database credential values under database.dbUser.* and database.rootUser.*
  • Adds search.basicAuth.* values, including existing-secret support (closes Add existingSecret for elasticsearch #125)
  • Adds support for networkPolicy (closes Add NetworkPolicy Support to Helm Charts #127)
  • Adds support for HTTPRoute (closes Make chart compatible with gateway API #120)
  • Adds support for global and per-image registry override (closes Add support for image.registry override to facilitate chart internalization #128)
  • Adds support for arbitrary extraObjects. (closes Support extraDeploy for Custom Resources (e.g., Secrets via AKS CSI from Key Vault) #129)
  • Prevents environment from overriding chart-managed settings.
  • Adds validation for conflicting reserved names, invalid credential combinations, ingress requirements, HPA configuration, PDB configuration, and kickstart data.
  • Removes service.spec support and ExternalName service support to prevent invalid service configurations.
  • Updates supported Kubernetes APIs to require Kubernetes >=1.23.
  • Removes the low-value Helm test connection hook.
  • Generates chart README from template chart/README.md.gotmpl using helm-docs.
  • Adds helm unittest coverage for database, search, init-container, and negative validation behavior.
  • Adds CI workflows for chart tests, unit test matrix, and generated docs verification.
  • Updates release packaging to stamp dev placeholders with release chart/app versions.

Breaking / Upgrade Notes

  • Minimum Kubernetes version is now 1.23.
  • service.spec has been removed.
  • service.type=ExternalName is no longer supported.
  • Chart-managed database Secrets changed from <release>-credentials to:
    • <release>-db-credentials
    • <release>-db-root-credentials
  • Chart-managed environment variables can no longer be overridden through environment; use chart values instead.

@johnjeffers johnjeffers requested review from a team as code owners May 23, 2026 05:53
@johnjeffers johnjeffers mentioned this pull request Jun 3, 2026
@johnjeffers johnjeffers merged commit a6a3da0 into main Jun 5, 2026
9 checks passed
@johnjeffers johnjeffers deleted the jj/chart-cleanup branch June 5, 2026 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants