Skip to content

Commit

Permalink
fix(chart): evm volumes fix (#1097)
Browse files Browse the repository at this point in the history
## Summary
Variety of chart fixes found while deploying forma:
- gas limit needs the hacky string number treatment
- volumes weren't deploying correctly
- update to not push faucet secret when disabled.
  • Loading branch information
joroshiba authored May 22, 2024
1 parent c77d6e0 commit d85d594
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 4 deletions.
2 changes: 1 addition & 1 deletion charts/evm-rollup/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.17.2
version: 0.18.0

# 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
2 changes: 1 addition & 1 deletion charts/evm-rollup/files/genesis/geth-genesis.json
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@
{{- end }}
},
"difficulty": "0",
"gasLimit": "{{ .Values.config.rollup.genesis.gasLimit }}",
"gasLimit": "{{ toString .Values.config.rollup.genesis.gasLimit | replace "\"" "" }}",
"alloc": {
{{- range $index, $value := .Values.config.rollup.genesis.alloc }}
{{- if $index }},{{- end }}
Expand Down
2 changes: 2 additions & 0 deletions charts/evm-rollup/templates/secretproviderclass.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ spec:
{{- $_ := set $ "key" .Values.config.sequencer.privateKey.secret }}
{{- tpl $.Values.secretProvider.parametersTemplate $ | nindent 4 }}
---
{{- if .Values.config.faucet.enabled }}
apiVersion: secrets-store.csi.x-k8s.io/v1
kind: SecretProviderClass
metadata:
Expand All @@ -26,3 +27,4 @@ spec:
{{- tpl $.Values.secretProvider.parametersTemplate $ | nindent 4 }}
---
{{- end }}
{{- end }}
4 changes: 2 additions & 2 deletions charts/evm-rollup/templates/volumes.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ spec:
- astria-dev-cluster-control-plane
- astria-dev-cluster-worker
---
{{- if .Values.config.blockscout.enabled }}
{{- if $.Values.config.blockscout.enabled }}
apiVersion: v1
kind: PersistentVolume
metadata:
Expand Down Expand Up @@ -76,7 +76,7 @@ spec:
requests:
storage: {{ $value.size }}
---
{{- if .Values.config.blockscout.enabled }}
{{- if $.Values.config.blockscout.enabled }}
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
Expand Down

0 comments on commit d85d594

Please sign in to comment.