Skip to content

Commit

Permalink
fix: expose the zk and bk accessModes config in values (#1132)
Browse files Browse the repository at this point in the history
* fix: expose the zk and bk accessModes config in values

Signed-off-by: ericsyh <[email protected]>

* apply to sn-platform chart

Signed-off-by: ericsyh <[email protected]>

* fix lint

Signed-off-by: ericsyh <[email protected]>

---------

Signed-off-by: ericsyh <[email protected]>
  • Loading branch information
ericsyh authored Dec 4, 2023
1 parent d4c6a55 commit e14cc48
Show file tree
Hide file tree
Showing 6 changed files with 45 additions and 13 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,9 @@ spec:
numDirsPerVolume: {{ .Values.bookkeeper.volumes.journal.numDirsPerVolume | default 1 }}
volumeClaimTemplate:
accessModes:
- ReadWriteOnce
{{- range .Values.bookkeeper.volumes.journal.accessModes }}
- {{ . | quote }}
{{- end }}
resources:
requests:
storage: {{ .Values.bookkeeper.volumes.journal.size }}
Expand All @@ -140,7 +142,9 @@ spec:
numDirsPerVolume: {{ .Values.bookkeeper.volumes.ledgers.numDirsPerVolume | default 1 }}
volumeClaimTemplate:
accessModes:
- ReadWriteOnce
{{- range .Values.bookkeeper.volumes.ledgers.accessModes }}
- {{ . | quote }}
{{- end }}
resources:
requests:
storage: {{ .Values.bookkeeper.volumes.ledgers.size }}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -163,14 +163,18 @@ spec:
persistence:
data:
accessModes:
- ReadWriteOnce
{{- range .Values.zookeeper.volumes.data.accessModes }}
- {{ . | quote }}
{{- end }}
resources:
requests:
storage: {{ .Values.zookeeper.volumes.data.size }}
{{- include "pulsar.zookeeper.data.storage.class" . | nindent 6 }}
dataLog:
accessModes:
- ReadWriteOnce
{{- range .Values.zookeeper.volumes.dataLog.accessModes }}
- {{ . | quote }}
{{- end }}
resources:
requests:
{{- if .Values.zookeeper.volumes.useSeparateDiskForTxlog }}
Expand Down
12 changes: 10 additions & 2 deletions charts/sn-platform-slim/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -638,6 +638,8 @@ zookeeper:
data:
name: data
size: 50Gi
accessModes:
- ReadWriteOnce
# storageClassName: ""
## If the storage class is left undefined when using persistence
## the default storage class for the cluster will be used.
Expand All @@ -656,6 +658,8 @@ zookeeper:
dataLog:
name: datalog
size: 10Gi
accessModes:
- ReadWriteOnce
# storageClassName: ""
## If the storage class is left undefined when using persistence
## the default storage class for the cluster will be used.
Expand Down Expand Up @@ -859,6 +863,8 @@ bookkeeper:
numDirsPerVolume: 1
name: journal
size: 10Gi
accessModes:
- ReadWriteOnce
# storageClassName: ""
## If the storage class is left undefined when using persistence
## the default storage class for the cluster will be used.
Expand All @@ -875,11 +881,13 @@ bookkeeper:
# extraParameters:
# iopsPerGB: "50"
ledgers:
name: ledgers
size: 50Gi
# It determines the directory of ledgers data
numVolumes: 1
numDirsPerVolume: 1
name: ledgers
size: 50Gi
accessModes:
- ReadWriteOnce
# storageClassName: ""
## If the storage class is left undefined when using persistence
## the default storage class for the cluster will be used.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,9 @@ spec:
numDirsPerVolume: {{ .Values.bookkeeper.volumes.journal.numDirsPerVolume | default 1 }}
volumeClaimTemplate:
accessModes:
- ReadWriteOnce
{{- range .Values.bookkeeper.volumes.journal.accessModes }}
- {{ . | quote }}
{{- end }}
resources:
requests:
storage: {{ .Values.bookkeeper.volumes.journal.size }}
Expand All @@ -140,7 +142,9 @@ spec:
numDirsPerVolume: {{ .Values.bookkeeper.volumes.ledgers.numDirsPerVolume | default 1 }}
volumeClaimTemplate:
accessModes:
- ReadWriteOnce
{{- range .Values.bookkeeper.volumes.journal.accessModes }}
- {{ . | quote }}
{{- end }}
resources:
requests:
storage: {{ .Values.bookkeeper.volumes.ledgers.size }}
Expand Down
10 changes: 7 additions & 3 deletions charts/sn-platform/templates/zookeeper/zookeeper-cluster.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -163,14 +163,18 @@ spec:
persistence:
data:
accessModes:
- ReadWriteOnce
resources:
{{- range .Values.zookeeper.volumes.data.accessModes }}
- {{ . | quote }}
{{- end }}
resources:
requests:
storage: {{ .Values.zookeeper.volumes.data.size }}
{{- include "pulsar.zookeeper.data.storage.class" . | nindent 6 }}
dataLog:
accessModes:
- ReadWriteOnce
{{- range .Values.zookeeper.volumes.dataLog.accessModes }}
- {{ . | quote }}
{{- end }}
resources:
requests:
{{- if .Values.zookeeper.volumes.useSeparateDiskForTxlog }}
Expand Down
12 changes: 10 additions & 2 deletions charts/sn-platform/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -713,6 +713,8 @@ zookeeper:
data:
name: data
size: 50Gi
accessModes:
- ReadWriteOnce
# storageClassName: ""
## If the storage class is left undefined when using persistence
## the default storage class for the cluster will be used.
Expand All @@ -731,6 +733,8 @@ zookeeper:
dataLog:
name: datalog
size: 10Gi
accessModes:
- ReadWriteOnce
# storageClassName: ""
## If the storage class is left undefined when using persistence
## the default storage class for the cluster will be used.
Expand Down Expand Up @@ -934,6 +938,8 @@ bookkeeper:
numDirsPerVolume: 1
name: journal
size: 10Gi
accessModes:
- ReadWriteOnce
# storageClassName: ""
## If the storage class is left undefined when using persistence
## the default storage class for the cluster will be used.
Expand All @@ -950,11 +956,13 @@ bookkeeper:
# extraParameters:
# iopsPerGB: "50"
ledgers:
name: ledgers
size: 50Gi
# It determines the directory of ledgers data
numVolumes: 1
numDirsPerVolume: 1
name: ledgers
size: 50Gi
accessModes:
- ReadWriteOnce
# storageClassName: ""
## If the storage class is left undefined when using persistence
## the default storage class for the cluster will be used.
Expand Down

0 comments on commit e14cc48

Please sign in to comment.