Skip to content

Commit

Permalink
chore(charts): update rollup for new geth version (#912)
Browse files Browse the repository at this point in the history
## Summary
Updates the rollup chart to support latest release of Geth.

## Background
Geth was updated to support bridge transactions & locking to specific
bridges on EVM.

## Changes
- Update geth genesis, and version.

## Testing
Running smoke test
  • Loading branch information
joroshiba authored Apr 3, 2024
1 parent b08a890 commit 1b72d6b
Show file tree
Hide file tree
Showing 4 changed files with 17 additions and 4 deletions.
7 changes: 7 additions & 0 deletions .github/workflows/docker-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -94,11 +94,18 @@ jobs:
install_only: true
- name: Install Foundry
uses: foundry-rs/foundry-toolchain@v1
- name: Log in to GHCR
uses: docker/login-action@v2
with:
registry: ghcr.io
username: ${{ github.actor }}
password: ${{ secrets.GITHUB_TOKEN }}
- name: Setup Smoke Test Environment
timeout-minutes: 5
run: |
TAG=sha-$(git rev-parse --short HEAD)
just deploy cluster
kubectl create secret generic regcred --from-file=.dockerconfigjson=$HOME/.docker/config.json --type=kubernetes.io/dockerconfigjson
echo "Deploying with astria images tagged $TAG"
just deploy smoke-test $TAG
- name: Run Smoke test
Expand Down
4 changes: 2 additions & 2 deletions charts/evm-rollup/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,13 +16,13 @@ type: application
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)

version: 0.10.4
version: 0.11.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
# follow Semantic Versioning. They should reflect the version the application is using.
# It is recommended to use it with quotes.
appVersion: "0.7.0"
appVersion: "0.9.0"

dependencies:
- name: celestia-node
Expand Down
4 changes: 3 additions & 1 deletion charts/evm-rollup/files/genesis/geth-genesis.json
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,9 @@
"astriaSequencerInitialHeight": {{ toString .Values.config.sequencer.initialBlockHeight | replace "\"" "" }},
"astriaRollupName": "{{ .Values.config.rollup.name }}",
"astriaCelestiaInitialHeight": {{ toString .Values.config.celestia.initialBlockHeight | replace "\"" "" }},
"astriaCelestiaHeightVariance": {{ toString .Values.config.celestia.heightVariance | replace "\"" "" }}
"astriaCelestiaHeightVariance": {{ toString .Values.config.celestia.heightVariance | replace "\"" "" }},
"astriaBridgeAddresses": [{{ .Values.config.rollup.genesis.bridgeAddress | quote }}],
"astriaBridgeAllowedAssetDenom": {{ .Values.config.rollup.genesis.bridgeAllowedAssetDenom | quote }}
{{- if not .Values.global.dev }}
{{- else }}
{{- end }}
Expand Down
6 changes: 5 additions & 1 deletion charts/evm-rollup/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ global:
images:
geth:
repo: ghcr.io/astriaorg/go-ethereum
tag: "0.7.0"
tag: "0.9.0"
devTag: latest
conductor:
repo: ghcr.io/astriaorg/conductor
Expand Down Expand Up @@ -59,6 +59,10 @@ config:
extraDataOverride: ""
# If set to true the genesis block will contain extra data
overrideGenesisExtraData: true
# Configure the sequencer bridge address and allowed asset denom if using
# the astria canonical bridge. Recommend removing alloc values if so.
bridgeAddress: ""
bridgeAllowedAssetDenom: nria
alloc:
- address: "0xaC21B97d35Bf75A7dAb16f35b111a50e78A72F30"
value:
Expand Down

0 comments on commit 1b72d6b

Please sign in to comment.