search

pydio / cells

Future-proof content collaboration platform
https://pydio.com
GNU Affero General Public License v3.0
1.83k stars 181 forks source link

Docs for k8s installs #550

Open arno4000 opened 8 months ago

arno4000 commented 8 months ago

Where are the docs of the helm values? I have read https://pydio.com/en/docs/cells/v4/kubernetes-quick-install, but I haven't found the docs of the helm values. On https://pydio.com/en/docs/cells/v4/k8s-advanced-parameters only the helm charts which are used as dependencies are documented. For example, how can I change the storage class or the size of the PVCs?

In my opinion, it should not be necessary to use helm pull to download the helm chart from the repo and then extract it to see the whole helm values. Is there a GitHub link of the helm chart, which I haven't found?

EDIT: it seems impossible to install the helm chart on an already existing k8s cluster. Because the helm chart deploys its own ingress controller and cert manager, with no obvious way to disable this.

arno4000 commented 8 months ago

The docker compose deployment just works. It should be possible to deploy the helm chart without deploying a half k8s cluster. Because almost all users will have already an ingress controller and cert manager etc.

cdujeu commented 8 months ago

see here: https://github.com/pydio/cells/tree/main/tools/kubernetes/cells

arno4000 commented 8 months ago

Thanks for the link. But I still have the problem that there seems to be no way to use an already existing ingress controller and clusterissuer. When enabling the ingress in the helm chart, I get the following error:

Error: INSTALLATION FAILED: Unable to continue with install: IngressClass "nginx" in namespace "" exists and cannot be imported into the current release: invalid ownership metadata; annotation validation error: key "meta.helm.sh/release-name" must equal "cells": current value is "ingress-nginx"; annotation validation error: key "meta.helm.sh/release-namespace" must equal "pydio": current value is "ingress-nginx"
arana198 commented 6 months ago

+1 for the issue.

Also have to say this is worst helm chart implementation I have seen.

cdujeu commented 6 months ago

Hey @arana198, thanks for your kind comment... You are more than welcome to suggest improvements (you know, we call it open source) ! ;-)

arana198 commented 6 months ago

I have created a PR with changes https://github.com/pydio/cells/pull/567 however I need write access which I haven't been granted yet

cdujeu commented 6 months ago

thx for submitting @arana198 ! you'll have to bear with us reviewing your PR ;-), we are currently pretty busy did you sign the CLA ? -c

arana198 commented 6 months ago

Yes I did, twice.

cdujeu commented 6 months ago

cool thx

arana198 commented 6 months ago

Hello, How long does it take to get write access after signing the CLA?

cdujeu commented 6 months ago

hello you don't get write access - we review your PR and eventually merge it. -c

arana198 commented 5 months ago

Hello there, is there a timeframe for this review?