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

Docs for k8s installs #550

Open
arno4000 opened this issue Mar 16, 2024 · 12 comments
Open

Docs for k8s installs #550

arno4000 opened this issue Mar 16, 2024 · 12 comments

Comments

@arno4000
Copy link

arno4000 commented Mar 16, 2024

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
Copy link
Author

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
Copy link
Member

cdujeu commented Mar 18, 2024

@arno4000
Copy link
Author

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
Copy link

+1 for the issue.

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

@cdujeu
Copy link
Member

cdujeu commented May 15, 2024

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

@arana198
Copy link

I have created a PR with changes #567 however I need write access which I haven't been granted yet

@cdujeu
Copy link
Member

cdujeu commented May 17, 2024

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
Copy link

arana198 commented May 17, 2024

Yes I did, twice.

@cdujeu
Copy link
Member

cdujeu commented May 17, 2024

cool thx

@arana198
Copy link

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

@cdujeu
Copy link
Member

cdujeu commented May 22, 2024

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

@arana198
Copy link

Hello there, is there a timeframe for this review?

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

No branches or pull requests

3 participants