From 05750ad9f6f66293f47eb0e8a5ffbf0709dc8c35 Mon Sep 17 00:00:00 2001
From: Michal Hajas <mhajas@redhat.com>
Date: Wed, 14 Aug 2024 15:37:29 +0200
Subject: [PATCH] HA guide changes

Signed-off-by: Michal Hajas <mhajas@redhat.com>
---
 doc/kubernetes/collector/build.sh             |  8 ++++++
 provision/infinispan/Utils.yaml               |  1 -
 .../infinispan/ispn-helm/kc-25-caches.yaml    | 17 ------------
 .../minikube/keycloak/templates/keycloak.yaml | 27 ++++++++++++++++---
 provision/minikube/keycloak/values.yaml       |  1 +
 5 files changed, 32 insertions(+), 22 deletions(-)
 delete mode 100644 provision/infinispan/ispn-helm/kc-25-caches.yaml

diff --git a/doc/kubernetes/collector/build.sh b/doc/kubernetes/collector/build.sh
index 50ced4ed5..703e873d7 100755
--- a/doc/kubernetes/collector/build.sh
+++ b/doc/kubernetes/collector/build.sh
@@ -16,6 +16,13 @@ helm template --debug ${STARTDIR}/../../../provision/minikube/keycloak \
   --set dbUrl=\<AWS_AURORA_URL_HERE\> \
   --set keycloakImage=\<KEYCLOAK_IMAGE_HERE\> \
   --set useAWSJDBCWrapper=true \
+  --set multiSite=true \
+  --set keycloakDocumentation=true \
+  --set infinispan.customConfig=false \
+  --set infinispan.remoteStore.enabled=false \
+  --set infinispan.remoteStore.host=infinispan.keycloak.svc \
+  --set infinispan.remoteStore.password=secure_password \
+  --set infinispan.site=keycloak \
   --set jvmDebug=false \
   --set cryostat=false \
   --set instances=3 \
@@ -39,6 +46,7 @@ helm template --debug ${STARTDIR}/../../../provision/minikube/keycloak \
   --set dbUrl=\<AWS_AURORA_URL_HERE\> \
   --set keycloakImage=\<KEYCLOAK_IMAGE_HERE\> \
   --set useAWSJDBCWrapper=true \
+  --set multiSite=true \
   --set jvmDebug=false \
   --set cryostat=false \
   --set heapInitMB=64 \
diff --git a/provision/infinispan/Utils.yaml b/provision/infinispan/Utils.yaml
index 3d1fb3094..16f0635cc 100644
--- a/provision/infinispan/Utils.yaml
+++ b/provision/infinispan/Utils.yaml
@@ -111,7 +111,6 @@ tasks:
         --set alertmanager.webhook.url={{ .ACCELERATOR_WEBHOOK_URL }}
         --set alertmanager.webhook.username={{ .ACCELERATOR_WEBHOOK_USERNAME }}
         --set alertmanager.webhook.password={{ .ACCELERATOR_WEBHOOK_PASSWORD }}
-        {{if eq .KC_KC25_MODE "true"}}--values ispn-helm/kc-25-caches.yaml{{end}}
         ./ispn-helm
     preconditions:
       - test -f ".task/kubecfg/{{.ROSA_CLUSTER_NAME}}"
diff --git a/provision/infinispan/ispn-helm/kc-25-caches.yaml b/provision/infinispan/ispn-helm/kc-25-caches.yaml
deleted file mode 100644
index 5c2d6ab79..000000000
--- a/provision/infinispan/ispn-helm/kc-25-caches.yaml
+++ /dev/null
@@ -1,17 +0,0 @@
-caches:
-  sessions:
-    owners: 2
-    memory:
-      maxCount: -1
-  offlineSessions:
-    owners: 2
-    memory:
-      maxCount: -1
-  clientSessions:
-    owners: 2
-    memory:
-      maxCount: -1
-  offlineClientSessions:
-    owners: 2
-    memory:
-      maxCount: -1
diff --git a/provision/minikube/keycloak/templates/keycloak.yaml b/provision/minikube/keycloak/templates/keycloak.yaml
index 5d31e37fa..104f7c073 100644
--- a/provision/minikube/keycloak/templates/keycloak.yaml
+++ b/provision/minikube/keycloak/templates/keycloak.yaml
@@ -87,14 +87,12 @@ spec:
     - name: tracing-sampler-type
       value: "parentbased_traceidratio"
     {{- end }}
-    {{- if .Values.persistentSessions }}
-    - name: spi-user-sessions-infinispan-use-caches
-      value: "false"
-    {{- end }}
+    # end::keycloak[]
     - name: http-metrics-histograms-enabled
       value: 'true'
     - name: http-metrics-slos
       value: '5,10,25,50,250,500'
+    # tag::keycloak[]
 {{- if or (eq .Values.database "infinispan") (eq .Values.database "postgres+infinispan") }}
     - name: storage-hotrod-host
       value: infinispan
@@ -145,6 +143,27 @@ spec:
     {{ end }}
     # end::keycloak-ispn[]
 {{- end }}
+{{- if .Values.keycloakDocumentation }}
+    # end::keycloak[]
+    # This block is just for documentation purposes as we need both versions of Infinispan config, with and without numbers to corresponding options
+    # tag::keycloak[]
+    - name: cache-remote-host
+      value: {{ quote .Values.infinispan.remoteStore.host }}
+    - name: cache-remote-port
+      value: {{ quote .Values.infinispan.remoteStore.port }}
+    - name: cache-remote-username
+      secret:
+        name: remote-store-secret
+        key: username
+    - name: cache-remote-password
+      secret:
+        name: remote-store-secret
+        key: password
+    {{- if .Values.infinispan.site }}
+    - name: spi-connections-infinispan-quarkus-site-name
+      value: {{ .Values.infinispan.site }}
+    {{ end }}
+{{- end }}
 {{- if .Values.infinispan.jgroupsTls }}
     - name: cache-embedded-mtls-enabled
       value: "true"
diff --git a/provision/minikube/keycloak/values.yaml b/provision/minikube/keycloak/values.yaml
index 4dec02f0a..c63e08531 100644
--- a/provision/minikube/keycloak/values.yaml
+++ b/provision/minikube/keycloak/values.yaml
@@ -41,6 +41,7 @@ predefinedAdmin: true
 persistentSessions: false
 externalInfinispan: false
 multiSite: false
+keycloakDocumentation: false
 infinispan:
   jgroupsTls: false
   customConfig: false