Skip to content

Commit

Permalink
fix: broken kurtosis CI dependency (#874)
Browse files Browse the repository at this point in the history
The current workflow expects the
`templates/cdk-erigon/config-sequencer.yaml` file. This file has been
removed and consolidated. The command that it's running shouldn't be
needed anymore since we removed `zkevm.sequencer-initial-fork-id` from
the file already.

Co-authored-by: Valentin Staykov <[email protected]>
  • Loading branch information
praetoriansentry and V-Staykov authored Jul 30, 2024
1 parent ba6346e commit 9733d9a
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions .github/workflows/ci_zkevm.yml
Original file line number Diff line number Diff line change
Expand Up @@ -96,12 +96,16 @@ jobs:
/usr/local/bin/yq -i '.args.data_availability_mode = "rollup"' cdk-erigon-sequencer-params.yml
/usr/local/bin/yq -i '.args.cdk_erigon_node_image = "cdk-erigon:local"' cdk-erigon-sequencer-params.yml
/usr/local/bin/yq -i '.args.zkevm_bridge_service_image = "hermeznetwork/zkevm-bridge-service:v0.5.0-RC8"' cdk-erigon-sequencer-params.yml
sed -i '/zkevm\.sequencer-initial-fork-id/d' ./templates/cdk-erigon/config-sequencer.yaml
- name: Deploy Kurtosis CDK package
working-directory: ./kurtosis-cdk
run: kurtosis run --enclave cdk-v1 --args-file cdk-erigon-sequencer-params.yml --image-download always .

- name: Override gas limit for test transactions
working-directory: ./kurtosis-cdk
run: |
sed -i 's/--gas-limit [0-9]*/--gas-limit 100000/gi' .github/actions/monitor-cdk-verified-batches/batch_verification_monitor.sh
- name: Monitor verified batches
working-directory: ./kurtosis-cdk
shell: bash
Expand Down Expand Up @@ -170,4 +174,4 @@ jobs:
uses: actions/upload-artifact@v3
with:
name: logs_${{ github.run_id }}
path: ./kurtosis-cdk/ci_logs
path: ./kurtosis-cdk/ci_logs

0 comments on commit 9733d9a

Please sign in to comment.