Skip to content

Commit 10a3b08

Browse files
suse-coderYour Namemichaelstingl
authored
Fix oci url and fluxcd config values (opencloud-eu#115)
* Fix oci url and fluxcd config values * Bump up Version * Bump up Version * Bump up Version * fix: correct busybox image reference and fix typos - Use initContainerImage values for busybox init container instead of minio image registry/tag - Fix typo: 'Upgrate' -> 'Upgrade' in timoni README - Fix template spacing for consistency (add space after opening brackets) Note: initContainerImage is already defined in values.yaml (line 75-88) Co-authored-by: Michael Stingl <[email protected]> --------- Co-authored-by: Your Name <[email protected]> Co-authored-by: Michael Stingl <[email protected]>
1 parent 0e21105 commit 10a3b08

File tree

8 files changed

+66
-20
lines changed

8 files changed

+66
-20
lines changed

charts/opencloud-microservices/README.md

Lines changed: 48 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -126,17 +126,6 @@ This repository contains the following chart:
126126

127127
### Microservices Chart (`charts/opencloud-microservices`)
128128

129-
**Architecture**: Pod-per-service
130-
- Every single service in its own pod
131-
- Full Gateway API integration
132-
- NATS service discovery required
133-
- Keycloak for authentication
134-
- MinIO for object storage
135-
- Integrated LDAP
136-
- Document editing with Collabora and/or OnlyOffice
137-
- Slightly higher resource usage due to microservices pod overhead
138-
### Microservices Chart (`charts/opencloud-microservices`)
139-
140129
**Architecture**: Pod-per-service
141130
- Every single service in its own pod
142131
- Full Gateway API integration
@@ -148,6 +137,51 @@ This repository contains the following chart:
148137
- Slightly higher resource usage due to microservices pod overhead
149138
- See [architectural warnings](./charts/opencloud-microservices/README.md#architectural-considerations)
150139

140+
# 🔐 Mandatory secret changes for production
141+
142+
Set all of the following to strong, unique values before deploying to production. Rotate regularly and never commit real values to VCS. Names are taken from deployments/timoni/secret.yaml.
143+
144+
1) LDAP / IDM
145+
- Secret: ldap-bind-secrets
146+
- Key: reva-ldap-bind-password
147+
- Note: Must match the LDAP admin password below
148+
149+
- Secret: opencloud-ldap-secrets
150+
- Key: adminPassword
151+
- Key: configPassword
152+
- Note: LDAP Administrator credentials
153+
154+
2) Object Storage (MinIO / S3)
155+
- Secret: s3secret
156+
- Key: accessKey
157+
- Key: secretKey
158+
- Note: Used by services for S3 access
159+
160+
- Secret: opencloud-minio-secrets - for testing only
161+
- Key: rootPassword
162+
- Note: MinIO root account password
163+
164+
3) Authentication (Keycloak) - for testing only
165+
- Secret: opencloud-keycloak-admin-secrets
166+
- Key: adminPassword
167+
- Note: Keycloak admin password
168+
169+
- Secret: opencloud-keycloak-postgresql-secrets - for testing only
170+
- Key: postgresqlPassword
171+
- Note: Password for Keycloak’s PostgreSQL database user
172+
173+
4) Messaging / Queues
174+
- Secret: opencloud-amqp-secret
175+
- Key: amqpUrl
176+
- Note: Contains credentials in URL form; replace with a strong user/password and secure endpoint
177+
178+
5) Document Editing (OnlyOffice)
179+
- Secret: opencloud-onlyoffice-secrets
180+
- Key: inbox
181+
- Key: outbox
182+
- Key: session
183+
- Note: Tokens used by OnlyOffice/WOPI integration
184+
151185

152186
## 🚀 Installation
153187

@@ -174,12 +208,14 @@ timoni bundle apply -f ./charts/opencloud-microservices/deployments/timoni/openc
174208

175209
The charts are also available in the GitHub Container Registry (GHCR) as OCI artifacts:
176210

211+
Change the repo url to: ghcr.io/opencloud-eu/helm-charts/opencloud-microservices
212+
177213
```bash
178214
cd charts/opencloud-microservices/deployments
179215
helmfile sync
180216

181217
```
182-
You can also install it with timoni instead of helm:
218+
You can also install it with timoni and fluxcd instead of helm:
183219
```bash
184220
kubectl apply -f ./charts/opencloud-microservices/deployments/timoni/ && \
185221
timoni bundle apply -f ./charts/opencloud-microservices/deployments/timoni/opencloud.cue --runtime ./charts/opencloud-microservices/deployments/timoni/runtime.cue

charts/opencloud-microservices/deployments/helm/helmfile.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@ repositories:
1313
releases:
1414
# --- OpenCloud Microservice Chart Release ---
1515
# Deploys the main OpenCloud application with all its integrated services.
16-
- name: opencloud-helm
17-
namespace: opencloud-helm # Dedicated namespace for the OpenCloud deployment.
16+
- name: opencloud
17+
namespace: opencloud # Dedicated namespace for the OpenCloud deployment.
1818
chart: ../../ # Path to the OpenCloud Helm chart.
1919
values:
2020
# --- General Settings ---

charts/opencloud-microservices/deployments/timoni/README.md

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
1-
# Install
2-
1+
# Install/Upgrade
32
kubectl apply -f ./charts/opencloud-microservices/deployments/timoni/ && \
43
timoni bundle apply -f ./charts/opencloud-microservices/deployments/timoni/opencloud.cue --runtime ./charts/opencloud-microservices/deployments/timoni/runtime.cue
54

charts/opencloud-microservices/deployments/timoni/configmap.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -93,6 +93,7 @@ data:
9393
# Collabora Configuration
9494
###############################################################################
9595
COLLABORA_URI: "https://collabora.opencloud.test"
96+
COLLABORA_DOMAIN: "collabora.opencloud.test"
9697
COLLABORA_ICON_URI: "https://collabora.opencloud.test/favicon.ico"
9798
COLLABORA_ENABLED: "true"
9899
COLLABORA_INSECURE: "true"

charts/opencloud-microservices/deployments/timoni/opencloud.cue

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,11 +21,11 @@ bundle: {
2121
namespace: "opencloud"
2222
values: {
2323
repository: {
24-
url: "oci://ghcr.io/suse-coder/helm-charts"
24+
url: "oci://ghcr.io/opencloud-eu/helm-charts"
2525
}
2626
chart: {
2727
name: "opencloud-microservices"
28-
version: "*"
28+
version: "0.1.0"
2929
}
3030
sync: {
3131
timeout: 10
@@ -56,7 +56,12 @@ bundle: {
5656
}
5757
}
5858
}
59+
collabora: {
60+
enabled: bool @timoni(runtime:bool:COLLABORA_ENABLED)
61+
domain: string @timoni(runtime:string:COLLABORA_DOMAIN)
62+
}
5963
onlyoffice: {
64+
enabled: bool @timoni(runtime:bool:ONLYOFFICE_ENABLED)
6065
domain: string @timoni(runtime:string:ONLYOFFICE_DOMAIN)
6166
persistence: {
6267
size: string @timoni(runtime:string:ONLYOFFICE_PERSISTENCE_SIZE)
@@ -499,3 +504,4 @@ bundle: {
499504
}
500505
}
501506
}
507+

charts/opencloud-microservices/deployments/timoni/runtime.cue

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ runtime: {
5151
"LDAP_URI": "obj.data.LDAP_URI"
5252
"OIDC_ISSUER_URI": "obj.data.OIDC_ISSUER_URI"
5353
"COLLABORA_URI": "obj.data.COLLABORA_URI"
54+
"COLLABORA_DOMAIN": "obj.data.COLLABORA_DOMAIN"
5455
"COLLABORA_ICON_URI": "obj.data.COLLABORA_ICON_URI"
5556
"WOPI_INGRESS_DOMAIN": "obj.data.WOPI_INGRESS_DOMAIN"
5657
"WOPI_COLLABORA_TLS_HOST": "obj.data.WOPI_COLLABORA_TLS_HOST"
@@ -135,6 +136,7 @@ runtime: {
135136
LDAP_URI: "ldap://openldap.openldap.svc.cluster.local:389"
136137
OIDC_ISSUER_URI: "https://keycloak.opencloud.test/realms/openCloud"
137138
COLLABORA_URI: "https://collabora.opencloud.test"
139+
COLLABORA_DOMAIN: "collabora.opencloud.test"
138140
COLLABORA_ICON_URI: "https://collabora.opencloud.test/favicon.ico"
139141
WOPI_INGRESS_DOMAIN: "wopi.opencloud.test"
140142
WOPI_COLLABORA_TLS_HOST: "wopi-collabora.kube.opencloud.test"

charts/opencloud-microservices/deployments/timoni/secret.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
# The ldap-bind-secrets and opencloud-ldap-secrets adminPassword have to be the same
12
apiVersion: v1
23
kind: Secret
34
metadata:

charts/opencloud-microservices/templates/minio/deployment.yaml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,8 @@ spec:
1313
fsGroup: 1000
1414
initContainers:
1515
- name: init-minio-bucket
16-
image: busybox
16+
image: "{{ .Values.initContainerImage.repository }}:{{ .Values.initContainerImage.tag }}"
17+
imagePullPolicy: {{ .Values.initContainerImage.pullPolicy | default "IfNotPresent" }}
1718
securityContext:
1819
runAsUser: 1000
1920
runAsGroup: 1000
@@ -25,7 +26,7 @@ spec:
2526
containers:
2627
- name: minio
2728
image: "{{ .Values.minio.image.registry }}/{{ .Values.minio.image.repository }}:{{ .Values.minio.image.tag | default .Values.minio.version }}"
28-
imagePullPolicy: {{.Values.minio.image.pullPolicy | default "IfNotPresent" }}
29+
imagePullPolicy: {{ .Values.minio.image.pullPolicy | default "IfNotPresent" }}
2930
args:
3031
- server
3132
- --console-address

0 commit comments

Comments
 (0)