Simplify installation process and improve documentation

iv-org / invidious-helm-chart

The helm chart for setting up Invidious with kubernetes.

GNU Affero General Public License v3.0

4 stars 3 forks source link

Simplify installation process and improve documentation #1

Closed perennialtech closed 4 months ago

perennialtech commented 4 months ago

This PR aims to simplify the deployment of the Helm chart and improve the accompanying documentation.

values.yaml
- Updated external links.
- Added a note on potential deadlock issues when replicaCount is greater than 1.
README.md
- Added a section on removing PostgreSQL PVCs.
- Removed section on creating PostgreSQL init scripts manually as they are now included by default.

JuniorJPDJ commented 4 months ago

I'm not sure if holding sql files inside the chart is a good idea as it will become out of date quickly. If you can propose (or even better - implement) some solution to make it not become out of date I'm OK with your solution :D

Thanks for the contribution!

unixfox commented 4 months ago

IIRC the sql files are included in the docker image no? Maybe we should.

JuniorJPDJ commented 4 months ago

hmm, maybe we should merge this and then create issue to fix later, as this PR is correct?

unixfox commented 4 months ago

ok for me