Skip to content

Commit

Permalink
fix(charts): fix hermes config (#1778)
Browse files Browse the repository at this point in the history
## Summary
The hermes config didn't work when path filters were used because some
standard config properties were below the seperated path filter section.

## Background
Allow use of path filters on deployed hermes instances.

## Changelogs
No updates required.

---------

Co-authored-by: quasystaty <ido@astria.org>
  • Loading branch information
joroshiba and quasystaty1 authored Nov 21, 2024
1 parent 3e85e8d commit fb66cf8
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 15 deletions.
2 changes: 1 addition & 1 deletion charts/hermes/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ type: application
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)
version: 0.5.1
version: 0.5.2

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
Expand Down
20 changes: 9 additions & 11 deletions charts/hermes/files/config.toml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
log_level = '{{ .Values.global.logLevel }}'

[rest]

# Whether or not to enable the REST service. Default: false
enabled = {{ .Values.rest.enabled }}

Expand Down Expand Up @@ -110,6 +109,15 @@ trust_threshold = '{{ $chain.trustThreshold }}'
{{- if $chain.memoPrefix }}
memo_prefix = '{{ $chain.memoPrefix }}'
{{- end }}
{{- if $chain.feeGranter }}
fee_granter = '{{ $chain.feeGranter }}'
{{- end }}
{{- if $chain.compatMode }}
compat_mode = '{{ $chain.compatMode }}'
{{- end }}
{{- if $chain.clearInterval }}
clear_interval = '{{ $chain.clearInterval }}'
{{- end }}

{{- if $chain.packetFilter }}
{{- if $chain.packetFilter.policy }}
Expand All @@ -132,14 +140,4 @@ recv = [
{{- end }}
{{- end }}
{{- end }}
{{- if $chain.feeGranter }}
fee_granter = '{{ $chain.feeGranter }}'
{{- end }}
{{- if $chain.compatMode }}
compat_mode = '{{ $chain.compatMode }}'
{{- end }}
{{- if $chain.clearInterval }}
clear_interval = '{{ $chain.clearInterval }}'
{{- end }}

{{ end }}
2 changes: 1 addition & 1 deletion charts/hermes/templates/configmaps.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ metadata:
namespace: {{ include "hermes.namespace" . }}
data:
config.toml: |
{{- tpl (.Files.Get "files/config.toml") $ | nindent 4 }}
{{- tpl (.Files.Get "files/config.toml") $ | nindent 4 }}
---
{{- if not .Values.secretProvider.enabled }}
{{- range $chainId, $chain := .Values.chains }}
Expand Down
4 changes: 2 additions & 2 deletions charts/hermes/templates/servicemonitor.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,14 +2,14 @@
apiVersion: monitoring.coreos.com/v1
kind: ServiceMonitor
metadata:
name: hermes-relayer-metrics
name: {{ include "hermes.fullname" . }}-metrics
labels:
app: {{ include "hermes.fullname" . }}
{{- with .Values.serviceMonitor.additionalLabels }}
{{- toYaml . | nindent 4 }}
{{- end }}
spec:
jobLabel: hermes-relayer-metric
jobLabel: {{ include "hermes.fullname" . }}-metric
namespaceSelector:
matchNames:
- {{ include "hermes.namespace" . }}
Expand Down

0 comments on commit fb66cf8

Please sign in to comment.