Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clean up tempalting and indenting in horizon chart #103

Merged
merged 1 commit into from
Oct 23, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions charts/horizon/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,9 @@

{{- define "core.config" -}}
{{- if eq .Values.global.network "testnet" -}}
{{- template "core.testnetConfig" }}
{{- include "core.testnetConfig" . }}
{{- else if eq .Values.global.network "pubnet" -}}
{{- template "core.pubnetConfig" }}
{{- include "core.pubnetConfig" . }}
{{- else -}}
{{- .Values.ingest.coreConfig }}
{{- end -}}
Expand Down
9 changes: 4 additions & 5 deletions charts/horizon/templates/core-configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,13 +2,12 @@
apiVersion: v1
kind: ConfigMap
metadata:
name: {{ template "common.fullname" . }}-core
name: {{ include "common.fullname" . }}-core
namespace: {{ .Release.Namespace }}
labels:
app: {{ template "common.fullname" . }}-core
chart: {{ template "common.chart" . }}
app: {{ include "common.fullname" . }}-core
chart: {{ include "common.chart" . }}
release: {{ .Release.Name }}
heritage: {{ .Release.Service }}
data:
stellar-core.cfg: |
{{ include "core.config" . | indent 4 }}
stellar-core.cfg: | {{ include "core.config" . | nindent 4 }}
6 changes: 3 additions & 3 deletions charts/horizon/templates/ingest-configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,11 @@
apiVersion: v1
kind: ConfigMap
metadata:
name: {{ template "common.fullname" . }}-ingest-env
name: {{ include "common.fullname" . }}-ingest-env
namespace: {{ .Release.Namespace }}
labels:
app: {{ template "common.fullname" . }}-ingest
chart: {{ template "common.chart" . }}
app: {{ include "common.fullname" . }}-ingest
chart: {{ include "common.chart" . }}
release: {{ .Release.Name }}
heritage: {{ .Release.Service }}
data:
Expand Down
58 changes: 27 additions & 31 deletions charts/horizon/templates/ingest-statefulset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,42 +3,40 @@
apiVersion: apps/v1
kind: StatefulSet
metadata:
name: {{ template "common.fullname" . }}-ingest
name: {{ include "common.fullname" . }}-ingest
namespace: {{ .Release.Namespace }}
labels:
app: {{ template "common.fullname" . }}-ingest
chart: {{ template "common.chart" . }}
app: {{ include "common.fullname" . }}-ingest
chart: {{ include "common.chart" . }}
release: {{ .Release.Name }}
heritage: {{ .Release.Service }}
spec:
replicas: {{ .Values.ingest.replicaCount }}
serviceName: {{ template "common.fullname" . }}-ingest
serviceName: {{ include "common.fullname" . }}-ingest
selector:
matchLabels:
app: {{ template "common.fullname" . }}-ingest
app: {{ include "common.fullname" . }}-ingest
template:
metadata:
labels:
app: {{ template "common.fullname" . }}-ingest
app: {{ include "common.fullname" . }}-ingest
release: {{ .Release.Name }}
horizon_network: {{ .Values.global.network }}
{{- if .Values.ingest.labels }}
{{- range $key, $value := .Values.ingest.labels }}
{{- if .Values.ingest.labels }}
{{- range $key, $value := .Values.ingest.labels }}
{{ $key }}: {{ $value | quote }}
{{- end }}
{{- end }}
{{- end }}
{{- end }}
{{- if .Values.ingest.annotations }}
annotations:
{{- range $key, $value := .Values.ingest.annotations }}
{{- range $key, $value := .Values.ingest.annotations }}
{{ $key }}: {{ $value | quote }}
{{- end }}
{{- end }}
{{- end }}
spec:
{{- if .Values.ingest.serviceAccountName }}
serviceAccountName: {{ .Values.ingest.serviceAccountName }}
{{- end }}
{{- if .Values.global.imagePullSecrets }}
imagePullSecrets: {{ toYaml .Values.global.imagePullSecrets | nindent 8 }}
imagePullSecrets: {{ toYaml .Values.global.imagePullSecrets | nindent 6 }}
{{- end }}
containers:
- name: horizon
Expand All @@ -61,23 +59,22 @@ spec:
- secretRef:
name: {{ .Values.ingest.existingSecret }}
- configMapRef:
name: {{ template "common.fullname" . }}-ingest-env
name: {{ include "common.fullname" . }}-ingest-env
readinessProbe:
httpGet:
port: {{ .Values.ingest.horizonConfig.port }}
path: /health
initialDelaySeconds: 5
timeoutSeconds: 5
{{- if .Values.ingest.resources }}
resources:
{{ toYaml .Values.ingest.resources | indent 10 }}
resources: {{ toYaml .Values.ingest.resources | nindent 10 }}
{{- end }}
volumeMounts:
- mountPath: /config
name: core-config
{{- if .Values.ingest.persistence.enabled }}
- mountPath: /var/lib/stellar
name: {{ template "common.fullname" . }}-ingest-var-lib-stellar
name: {{ include "common.fullname" . }}-ingest-var-lib-stellar
{{- end }}
{{- if .Values.ingest.coreExporter.enabled }}
- name: stellar-core-prometheus-exporter
Expand All @@ -87,18 +84,17 @@ spec:
- containerPort: 9473
name: metrics
{{- if .Values.ingest.coreExporter.resources }}
resources:
{{ toYaml .Values.ingest.coreExporter.resources | indent 10 }}
resources: {{ toYaml .Values.ingest.coreExporter.resources | nindent 10 }}
{{- end }}
{{- end }}
volumes:
- name: core-config
configMap:
name: {{ template "common.fullname" . }}-core
name: {{ include "common.fullname" . }}-core
{{- if .Values.ingest.persistence.enabled }}
volumeClaimTemplates:
- metadata:
name: {{ template "common.fullname" . }}-ingest-var-lib-stellar
name: {{ include "common.fullname" . }}-ingest-var-lib-stellar
spec:
accessModes: ["ReadWriteOnce"]
storageClassName: {{ .Values.ingest.persistence.storageClass }}
Expand All @@ -110,11 +106,11 @@ spec:
apiVersion: v1
kind: Service
metadata:
name: {{ template "common.fullname" . }}-ingest-core
name: {{ include "common.fullname" . }}-ingest-core
namespace: {{ .Release.Namespace }}
labels:
app: {{ template "common.fullname" . }}-ingest-core
chart: {{ template "common.chart" . }}
app: {{ include "common.fullname" . }}-ingest-core
chart: {{ include "common.chart" . }}
release: {{ .Release.Name }}
heritage: {{ .Release.Service }}
spec:
Expand All @@ -124,16 +120,16 @@ spec:
port: 11626
targetPort: 11626
selector:
app: {{ template "common.fullname" . }}-ingest
app: {{ include "common.fullname" . }}-ingest
---
apiVersion: v1
kind: Service
metadata:
name: {{ template "common.fullname" . }}-ingest
name: {{ include "common.fullname" . }}-ingest
namespace: {{ .Release.Namespace }}
labels:
app: {{ template "common.fullname" . }}-ingest
chart: {{ template "common.chart" . }}
app: {{ include "common.fullname" . }}-ingest
chart: {{ include "common.chart" . }}
release: {{ .Release.Name }}
heritage: {{ .Release.Service }}
spec:
Expand All @@ -143,5 +139,5 @@ spec:
port: 80
targetPort: {{ .Values.ingest.horizonConfig.port }}
selector:
app: {{ template "common.fullname" . }}-ingest
app: {{ include "common.fullname" . }}-ingest
{{- end }}
20 changes: 10 additions & 10 deletions charts/horizon/templates/ingress.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,17 +3,17 @@
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
name: {{ template "common.fullname" . }}-ingest
name: {{ include "common.fullname" . }}-ingest
namespace: {{ .Release.Namespace }}
{{- if .Values.ingest.ingress.annotations }}
annotations:
{{- range $key, $value := .Values.ingest.ingress.annotations }}
{{- range $key, $value := .Values.ingest.ingress.annotations }}
{{ $key }}: {{ $value | quote }}
{{- end }}
{{- end }}
{{- end }}
spec:
tls:
- secretName: {{ template "common.fullname" . }}-ingest-cert
- secretName: {{ include "common.fullname" . }}-ingest-cert
hosts:
- {{ .Values.ingest.ingress.host }}
rules:
Expand All @@ -24,7 +24,7 @@ spec:
pathType: Prefix
backend:
service:
name: {{ template "common.fullname" . }}-ingest
name: {{ include "common.fullname" . }}-ingest
port:
number: 80
{{- end }}
Expand All @@ -33,17 +33,17 @@ spec:
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
name: {{ template "common.fullname" . }}-web
name: {{ include "common.fullname" . }}-web
namespace: {{ .Release.Namespace }}
{{- if .Values.web.ingress.annotations }}
annotations:
{{- range $key, $value := .Values.web.ingress.annotations }}
{{ $key }}: {{ $value | quote }}
{{- end }}
{{ $key }}: {{ $value | quote }}
{{- end }}
{{- end }}
spec:
tls:
- secretName: {{ template "common.fullname" . }}-web-cert
- secretName: {{ include "common.fullname" . }}-web-cert
hosts:
- {{ .Values.web.ingress.host }}
rules:
Expand All @@ -54,7 +54,7 @@ spec:
pathType: Prefix
backend:
service:
name: {{ template "common.fullname" . }}-web
name: {{ include "common.fullname" . }}-web
port:
number: 80
{{- end }}
10 changes: 5 additions & 5 deletions charts/horizon/templates/web-configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,11 @@
apiVersion: v1
kind: ConfigMap
metadata:
name: {{ template "common.fullname" . }}-web-env
name: {{ include "common.fullname" . }}-web-env
namespace: {{ .Release.Namespace }}
labels:
app: {{ template "common.fullname" . }}-web
chart: {{ template "common.chart" . }}
app: {{ include "common.fullname" . }}-web
chart: {{ include "common.chart" . }}
release: {{ .Release.Name }}
heritage: {{ .Release.Service }}
data:
Expand All @@ -24,9 +24,9 @@ data:
PER_HOUR_RATE_LIMIT: {{ .Values.web.horizonConfig.perHourRateLimit | quote}}
PORT: {{ .Values.web.horizonConfig.port | quote}}
SSE_UPDATE_FREQUENCY: {{ .Values.web.horizonConfig.sseUpdateFrequency | quote}}
{{ if .Values.web.horizonConfig.stellarCoreUrl -}}
{{- if .Values.web.horizonConfig.stellarCoreUrl -}}
STELLAR_CORE_URL: {{ .Values.web.horizonConfig.stellarCoreUrl | quote}}
{{- else }}
STELLAR_CORE_URL: http://{{ template "common.fullname" . }}-ingest-core:11626
STELLAR_CORE_URL: http://{{ include "common.fullname" . }}-ingest-core:11626
{{- end }}
{{- end }}
38 changes: 18 additions & 20 deletions charts/horizon/templates/web-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,42 +3,41 @@
apiVersion: apps/v1
kind: Deployment
metadata:
name: {{ template "common.fullname" . }}-web
name: {{ include "common.fullname" . }}-web
namespace: {{ .Release.Namespace }}
labels:
app: {{ template "common.fullname" . }}-web
chart: {{ template "common.chart" . }}
app: {{ include "common.fullname" . }}-web
chart: {{ include "common.chart" . }}
release: {{ .Release.Name }}
heritage: {{ .Release.Service }}
spec:
replicas: {{ .Values.web.replicaCount }}
selector:
matchLabels:
app: {{ template "common.fullname" . }}-web
app: {{ include "common.fullname" . }}-web
template:
metadata:
labels:
app: {{ template "common.fullname" . }}-web
app: {{ include "common.fullname" . }}-web
release: {{ .Release.Name }}
horizon_network: {{ .Values.global.network }}
{{- if .Values.web.labels }}
{{- if .Values.web.labels }}
{{- range $key, $value := .Values.web.labels }}
{{ $key }}: {{ $value | quote }}
{{- end }}
{{- end }}
{{- end }}
{{- if .Values.web.annotations }}
annotations:
{{- range $key, $value := .Values.web.annotations }}
{{ $key }}: {{ $value | quote }}
{{- end }}
{{ $key }}: {{ $value | quote }}
{{- end }}
{{- end }}
spec:
{{- if .Values.web.serviceAccountName }}
serviceAccountName: {{ .Values.web.serviceAccountName }}
{{- end }}
{{- if .Values.global.imagePullSecrets }}
imagePullSecrets: {{ toYaml .Values.global.imagePullSecrets | nindent 8 }}
{{- end }} containers:
imagePullSecrets: {{ toYaml .Values.global.imagePullSecrets | nindent 6 }}
{{- end }}
containers:
- name: horizon
image: {{ include "common.horizonImage" . | quote }}
{{- if .Values.web.cliArgs }}
Expand All @@ -57,26 +56,25 @@ spec:
- secretRef:
name: {{ .Values.web.existingSecret }}
- configMapRef:
name: {{ template "common.fullname" . }}-web-env
name: {{ include "common.fullname" . }}-web-env
readinessProbe:
httpGet:
port: {{ .Values.web.horizonConfig.port }}
path: /health
initialDelaySeconds: 5
timeoutSeconds: 5
{{- if .Values.web.resources }}
resources:
{{ toYaml .Values.web.resources | indent 10 }}
resources: {{ toYaml .Values.web.resources | nindent 10 }}
{{- end }}
---
apiVersion: v1
kind: Service
metadata:
name: {{ template "common.fullname" . }}-web
name: {{ include "common.fullname" . }}-web
namespace: {{ .Release.Namespace }}
labels:
app: {{ template "common.fullname" . }}-web
chart: {{ template "common.chart" . }}
app: {{ include "common.fullname" . }}-web
chart: {{ include "common.chart" . }}
release: {{ .Release.Name }}
heritage: {{ .Release.Service }}
spec:
Expand All @@ -86,5 +84,5 @@ spec:
port: 80
targetPort: {{ .Values.web.horizonConfig.port }}
selector:
app: {{ template "common.fullname" . }}-web
app: {{ include "common.fullname" . }}-web
{{- end }}
Loading