cells icon indicating copy to clipboard operation
cells copied to clipboard

Docs for k8s installs

Open arno4000 opened this issue 1 year ago • 12 comments

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 avatar Mar 16 '24 12:03 arno4000

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.

arno4000 avatar Mar 16 '24 13:03 arno4000

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

cdujeu avatar Mar 18 '24 12:03 cdujeu

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"

arno4000 avatar Mar 18 '24 19:03 arno4000

+1 for the issue.

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

arana198 avatar May 15 '24 12:05 arana198

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

cdujeu avatar May 15 '24 13:05 cdujeu

Sorry @cdujeu, I realised the comment was a bit out of order, I didn't mean to be rude. I am making some changes will raise a PR soon

arana198 avatar May 15 '24 13:05 arana198

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

arana198 avatar May 17 '24 10:05 arana198

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

cdujeu avatar May 17 '24 10:05 cdujeu

Yes I did, twice.

arana198 avatar May 17 '24 10:05 arana198

cool thx

cdujeu avatar May 17 '24 10:05 cdujeu

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

arana198 avatar May 21 '24 22:05 arana198

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

cdujeu avatar May 22 '24 06:05 cdujeu

Hello there, is there a timeframe for this review?

arana198 avatar Jun 16 '24 12:06 arana198