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

chore: helm chart added #4694

Closed
wants to merge 10 commits into from
Closed

chore: helm chart added #4694

wants to merge 10 commits into from

Conversation

mguptahub
Copy link
Collaborator

No description provided.

@mguptahub mguptahub marked this pull request as draft June 4, 2024 05:34
@mguptahub mguptahub marked this pull request as ready for review June 4, 2024 05:50
@karitham
Copy link

karitham commented Jun 7, 2024

Hi, I took the liberty to fork out a considerable amount of your chart. I would rather upstream it than maintain it myself, as it would have a bigger and better community, and more documentation.

I assume you have more constraints than I do (unsure what the questions.yaml file does for example).

Other than that I've standardized the chart, used proper dependencies etc.

Most of the issues I had with your chart (when trying it out) were the ingress setup does not work well when you already have cert-manager/an issuer.

The postgres/minio/redis components weren't scalable at all and didn't have all required options for me. (eg I want password auth)

URLs were in configmap but they might have contained passwords for postgres/redis.

It was missing configuration option to disable signups.

Feel free to reuse any part of what I did.

https://github.com/Karitham/plane-helm

@sriramveeraghanta sriramveeraghanta marked this pull request as draft June 12, 2024 08:17
@mguptahub mguptahub closed this Jun 14, 2024
@mguptahub mguptahub deleted the chore/helm-chart branch June 14, 2024 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants