Skip to content

Commit

Permalink
Merge pull request #276 from 0xPolygon/CDK-507-fix-create-rollup-para…
Browse files Browse the repository at this point in the history
…meters-population-in-kurtosis-cdk

fix: Remove redundant copy step for `create_rollup_parameters.json`
  • Loading branch information
praetoriansentry authored Sep 23, 2024
2 parents 28e021a + e317b42 commit 06ad0ac
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions templates/contract-deploy/run-contract-setup.sh
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,10 @@ pushd /opt/zkevm-contracts || exit 1
# setup a foundry toml in case we do a gas token or dac deployment
printf "[profile.default]\nsrc = 'contracts'\nout = 'out'\nlibs = ['node_modules']\n" > foundry.toml

cp /opt/contract-deploy/deploy_parameters.json /opt/zkevm-contracts/deployment/v2/deploy_parameters.json
cp /opt/contract-deploy/create_rollup_parameters.json /opt/zkevm-contracts/deployment/v2/create_rollup_parameters.json
sed -i 's#http://127.0.0.1:8545#{{.l1_rpc_url}}#' hardhat.config.ts

# Deploy gas token
# TODO in the future this should be configurable. I.e. we should be able to specify a token address that has already been deployed
# {{if .zkevm_use_gas_token_contract}}
Expand All @@ -120,10 +124,6 @@ forge create \
jq --slurpfile c gasToken-erc20.json '.gasTokenAddress = $c[0].deployedTo' /opt/contract-deploy/create_rollup_parameters.json > /opt/zkevm-contracts/deployment/v2/create_rollup_parameters.json
# {{end}}

cp /opt/contract-deploy/deploy_parameters.json /opt/zkevm-contracts/deployment/v2/deploy_parameters.json
cp /opt/contract-deploy/create_rollup_parameters.json /opt/zkevm-contracts/deployment/v2/create_rollup_parameters.json
sed -i 's#http://127.0.0.1:8545#{{.l1_rpc_url}}#' hardhat.config.ts

is_first_rollup=0 # an indicator if this deployment is doing the first setup of the agglayer etc
if [[ ! -e /opt/zkevm/combined.json ]]; then
echo_ts "It looks like this is the first rollup so we'll deploy the LxLy and Rollup Manager"
Expand Down

0 comments on commit 06ad0ac

Please sign in to comment.