Skip to content

Commit fc322e6

Browse files
committed
Fix chart paths and set httpRoute.enabled to false by default
This commit addresses feedback from @wrenix in PR opencloud-eu#31: - Updates all chart paths in README.md to use the correct ./charts/opencloud and ./charts/opencloud-dev references - Sets httpRoute.enabled to false by default following standard Helm chart practices - Improves documentation consistency across files - Updates repository URL to the official opencloud-eu/helm repository
1 parent 0c15a08 commit fc322e6

File tree

2 files changed

+8
-7
lines changed

2 files changed

+8
-7
lines changed

charts/opencloud/README.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ Please ensure that your PR follows best practices and includes necessary documen
6666
To install the chart with the release name `opencloud`:
6767

6868
```bash
69-
helm install opencloud . \
69+
helm install opencloud ./charts/opencloud \
7070
--namespace opencloud \
7171
--create-namespace \
7272
--set httpRoute.gateway.name=opencloud-gateway \
@@ -604,11 +604,11 @@ Finally, install OpenCloud using Helm:
604604

605605
```bash
606606
# Clone the repository
607-
git clone https://github.com/your-repo/opencloud-helm.git
607+
git clone https://github.com/opencloud-eu/helm.git opencloud-helm
608608
cd opencloud-helm
609609

610610
# Install OpenCloud
611-
helm install opencloud . \
611+
helm install opencloud ./charts/opencloud \
612612
--namespace opencloud \
613613
--create-namespace \
614614
--set httpRoute.gateway.name=opencloud-gateway \
@@ -669,7 +669,7 @@ https://docs.opencloud.eu/docs/admin/getting-started/docker/docker
669669
Deployment from the file system:
670670

671671
```
672-
$ helm install opencloud -n opencloud --create-namespace ./opencloud-dev --set=adminPassword="<MY-SECURE-PASSWORD>" --set=url="<PUBLIC-URL>"
672+
$ helm install opencloud -n opencloud --create-namespace ./charts/opencloud-dev --set=adminPassword="<MY-SECURE-PASSWORD>" --set=url="<PUBLIC-URL>"
673673
```
674674

675675
It is important that the public-url is reachable, and forwarded to the backend-service opencloud-service:443,
@@ -678,7 +678,7 @@ otherwise login will not be possible or the message "missing or invalid config"
678678
For testing with the default settings port-forwarding from localhost can be used:
679679

680680
```
681-
$ helm install opencloud -n opencloud --create-namespace ./opencloud-dev
681+
$ helm install opencloud -n opencloud --create-namespace ./charts/opencloud-dev
682682
683683
Release "opencloud" does not exist. Installing it now.
684684
NAME: opencloud
@@ -732,7 +732,7 @@ $ helm uninstall -n opencloud opencloud
732732
release "opencloud" uninstalled
733733
```
734734

735-
The data PVC is configured to be kept, so it will survive uninstall and install of opencloud-dev
735+
The data PVC is configured to be kept, so it will survive uninstall and install of charts/opencloud-dev
736736

737737
## 📜 License
738738

charts/opencloud/values.yaml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -506,7 +506,8 @@ opencloud:
506506
# Requires Gateway API resources and suitable controller installed within the cluster
507507
# (see: https://gateway-api.sigs.k8s.io/guides/)
508508
httpRoute:
509-
enabled: true
509+
# Enable HTTPRoute resources (set to true to create Gateway API routes)
510+
enabled: false
510511

511512
# Gateway configuration
512513
gateway:

0 commit comments

Comments
 (0)