Feature: Add Collabora as a subchart #611

[kriegalex]

Description of the change

• Add a dependency to the 1.1.20 collabora chart from their official repository
• Add the subchart config to values.yaml
• Add the configuration options documentation to README

Benefits

• Install collabora as a subchart of nextcloud via helm

Possible drawbacks

• People may try to enable it without knowing what is is and how to configure it (mainly extra_params, server_name, aliasgroups and annotations.

Applicable issues

• fixes Feature: Add Collabora as a subchart #611

Additional information

Tested on my local kubernetes cluster with a HTTPS nginx ingress. Nextcloud can communicate with Collabora. Not tested with HAProxy (though it is only a change in annotations specific to HAProxy).

Checklist

• I have read the CONTRIBUTING.md doc.
• DCO has been signed off on the commit.
• Chart version bumped in Chart.yaml according to semver.
• Parameters are documented in the README.md

[jessebot]

Oh, awesome! Thanks, @kriegalex for this PR!

Right now the workflow is failing:
https://github.com/nextcloud/helm/actions/runs/10956242340/job/30421859233?pr=623#step:7:16

You can fix this by adding the helm repo for collabora to this section of the workflow file, similar to how we do for bitnami:

helm/.github/workflows/lint-test.yaml

Lines 39 to 41 in b25b4ad

	- name: Add dependency chart repos
	run: |
	helm repo add bitnami https://charts.bitnami.com/bitnami

[jessebot]

looks great, but we need to ensure the tests still work, so I left a comment about how to update the GHA workflow. If you can, it would also be neat to have a test for collabora itself as well, but we can also hold off on that for a separate PR if you'd like. 🙏

update: added the commit to maybe fix the tests :)

[kriegalex]

Thanks for the change on lint tests. I'm still learning k8s, didn't think of that.

requested changes done

[provokateurin]

Publishing the release failed: https://github.com/nextcloud/helm/actions/runs/12341432621/job/34440050803

[jessebot]

looking!

[jessebot]

let's see, the error was:

Error: no repository definition for https://collaboraonline.github.io/online

That does seem to be right according to the docs:
https://github.com/CollaboraOnline/online/tree/master/kubernetes/helm/collabora-online#deploying-collabora-online-in-kubernetes

Ah! It's the actual release config that needs to have the collabora online helm repo added:

helm/.github/workflows/release.yaml

Lines 35 to 38 in 10cae24

	- name: Add dependency chart repos
	run: |
	helm repo add bitnami https://charts.bitnami.com/bitnami

PR incoming!

[jessebot]

PR to fix release here: #667

PR to release this change here: #668