Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cnpg-db helm chart including documentation, first draft #165

Closed
wants to merge 1 commit into from

Conversation

wibed
Copy link

@wibed wibed commented Nov 14, 2023

kindly review, works ootb.

helm chart for the postgres database accompanying the operator.
it is supposed to mirror the official crd as closely as possible to facilitate maintenance as changes occur.

there is a small documentation within the folder "examples" explaining the most central
parts the helm charts tries to facilitate, just to get a grasp a little bit faster.

but essentially it is a plain templated representation of the cloudnative-pg crd's .

@wibed wibed force-pushed the cnpg-db_helmchart branch from d4e25b6 to 97f9b70 Compare November 18, 2023 08:56
@onedr0p
Copy link

onedr0p commented Nov 18, 2023

Duplicate of #106

@wibed
Copy link
Author

wibed commented Nov 18, 2023

Duplicate of #106

its not! my version is a plain template of the crd's including most elements outlined there.
the pr you referred is much closer to wrapper including custom names and references injecting values into definitions instead of using plain yaml and a helper.tpl like this pr does.

@wibed
Copy link
Author

wibed commented Jan 8, 2024

what is the status on this pr?

@itay-grudev
Copy link
Collaborator

Closed as completed in #179. See also #178.

@itay-grudev itay-grudev added the chart( cluster ) Related to the cluster chart label May 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chart( cluster ) Related to the cluster chart
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants