search

truecharts / public

Community Helm Chart Repository
https://truecharts.org
GNU Affero General Public License v3.0
1.14k stars 617 forks source link

Common Deployed Infrastructure Sharing Accross TrueChart Apps. #7838

Closed mandusm closed 1 year ago

mandusm commented 1 year ago

Is your feature request related to a problem?

Compute resources like CPU and memory are valuable and should be used frugally by creating duplicates of the same common infrastructure. TrueChart applications currently deploy infrastructure dependencies like SQL databases for each TrueCharts application, creating duplicates of common infrastructure and causing a high overhead and misuse of TrueNAS resources.

Describe the solution you'd like

Using kubernetes namespacing, DNS service discovery and HELM chart functionality, determine if an Application requires DB infrastructure, then let HELM check if common infra has already been deployed and reuse that, or if not, deploy new common infrastructure.

Describe alternatives you've considered

Allow users to pass all available chart configurations using the TrueNAS web interfaces. In other words, expose all options in the values.yaml for advanced users to modify.

Additional context

No response

I've read and agree with the following

PrivatePuffin commented 1 year ago

This is not how kubernetes is normally used. Our new postgresql backend doesnt even allow for multiple databases etc.

tldr: this is the opposite of what the TrueCharts Microservice philosophy stands for.

PrivatePuffin commented 1 year ago

Also: the overhead for our postgresql deployment is minimal. We picked postgresql for a reason.

other databases we wouldnt even care about enough to put any significant time into..

PrivatePuffin commented 1 year ago

Im even going to lock this, as its not up for any debate or feedback.

even if we had the manpower to implement this, which we dont, this goes literally against every reasonable kubernetes best practice.

And on top of that, it goes far byond what this project does: building helm charts.

PrivatePuffin commented 1 year ago

Your free to start a project to dynamically share a database server based on crs or discovery. It just isnt this project.

PrivatePuffin commented 1 year ago

The alternative is in-part possible deploying as normal helm already.

we also dont plan to add all values.yaml settings. A TrueNAS App is not a custom cluster/chart in the same way as a helm chart is.

we’ve seperated exposed settings based on target audience and the fact every setting we expose might lead to needless breaking changes in the future due to GUI issues.