This is an automated email from the ASF dual-hosted git repository.

lhotari pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar-helm-chart.git


The following commit(s) were added to refs/heads/master by this push:
     new 0768986  Fix Oxia config so that it includes a list of all pods in the 
statefulset (#553)
0768986 is described below

commit 07689860f6d4b588f3767321d969d1caaaa3f69d
Author: Lari Hotari <[email protected]>
AuthorDate: Fri Nov 22 15:54:11 2024 +0200

    Fix Oxia config so that it includes a list of all pods in the statefulset 
(#553)
    
    * Fix Oxia config so that it includes a list of all pods in the statefulset
    
    * Test Oxia with 3 replicas since some issues only come up with more nodes
    
    * Make internal name not a fqdn
    
    * Fix issue with insufficient cpu requests in CI
---
 .ci/clusters/values-oxia.yaml     | 11 ++++++-----
 charts/pulsar/templates/_oxia.tpl | 12 ++++++++++--
 2 files changed, 16 insertions(+), 7 deletions(-)

diff --git a/.ci/clusters/values-oxia.yaml b/.ci/clusters/values-oxia.yaml
index 51afd8c..15d69dc 100644
--- a/.ci/clusters/values-oxia.yaml
+++ b/.ci/clusters/values-oxia.yaml
@@ -25,10 +25,11 @@ components:
   functions: false
 
 oxia:
-  initialShardCount: 1
-  replicationFactor: 1
+  initialShardCount: 3
+  replicationFactor: 3
   server:
-    replicas: 1
-    memoryLimit: 256Mi
-    dbCacheSizeMb: 128
+    replicas: 3
+    cpuLimit: 333m
+    memoryLimit: 200Mi
+    dbCacheSizeMb: 100
     storageSize: 1Gi
diff --git a/charts/pulsar/templates/_oxia.tpl 
b/charts/pulsar/templates/_oxia.tpl
index b8b8143..0f0fe25 100644
--- a/charts/pulsar/templates/_oxia.tpl
+++ b/charts/pulsar/templates/_oxia.tpl
@@ -88,8 +88,16 @@ namespaces:
     initialShardCount: {{ .Values.oxia.initialShardCount }}
     replicationFactor: {{ .Values.oxia.replicationFactor }}
 servers:
-  - public:  {{ template "pulsar.fullname" . }}-{{ .Values.oxia.component 
}}-svc.{{ template "pulsar.namespace" . }}.svc.cluster.local:{{ 
.Values.oxia.server.ports.public }}
-    internal:  {{ template "pulsar.fullname" . }}-{{ .Values.oxia.component 
}}-svc.{{ template "pulsar.namespace" . }}.svc.cluster.local:{{ 
.Values.oxia.server.ports.internal }}
+  {{- $servicename := printf "%s-%s-svc" (include "pulsar.fullname" .) 
.Values.oxia.component }}
+  {{- $fqdnSuffix := printf "%s.svc.cluster.local" (include "pulsar.namespace" 
.) }}
+  {{- $podnamePrefix := printf "%s-%s-server-" (include "pulsar.fullname" .) 
.Values.oxia.component }}
+  {{- range until (int .Values.oxia.server.replicas) }}
+  {{- $podnameIndex := . }}
+  {{- $podname := printf "%s%d.%s" $podnamePrefix $podnameIndex $servicename }}
+  {{- $podnameFQDN := printf "%s.%s" $podname $fqdnSuffix }}
+  - public: {{ $podnameFQDN }}:{{ $.Values.oxia.server.ports.public }}
+    internal: {{ $podname }}:{{ $.Values.oxia.server.ports.internal }}
+  {{- end }}
 {{- end }}
 
 {{/*

Reply via email to