Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Calculate ccm gas limit #3935

Merged
merged 16 commits into from
Sep 14, 2023
Merged

feat: Calculate ccm gas limit #3935

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
7db71ed
WIP:
syan095 Sep 1, 2023
f2cc9cb
Connected the calculated gas limit for CCM messages to Ethereum Trans…
syan095 Sep 4, 2023
00239ea
Merge remote-tracking branch 'origin/main' into feature/ccm-gas-limit
syan095 Sep 4, 2023
9e079b1
minor improvement to comments
syan095 Sep 4, 2023
68ad01a
Removed the default gas limit for CCM messages.
syan095 Sep 5, 2023
9e32512
Fixed minor typo
syan095 Sep 5, 2023
9a8fdae
Refactored so that the gas limit calculation is done when building Tr…
syan095 Sep 6, 2023
779563a
Merged latest from main.
syan095 Sep 6, 2023
3b335c4
Chore/mainnet ccm gaslimit test (#3887)
albert-llimos Sep 11, 2023
cab939c
chore: merge from main
albert-llimos Sep 11, 2023
af9a0fb
chore: add into() and small refactoring improvement
albert-llimos Sep 11, 2023
a61d85b
chore: add random gas consumption tests
albert-llimos Sep 11, 2023
8da0758
fix: more generic gas limit, avoid panics
dandanlen Sep 11, 2023
6941f3c
directly use Eip1559TransactionRequest
AlastairHolmes Sep 11, 2023
f59c32e
Renamed a parameter
syan095 Sep 11, 2023
048c7a6
Merge branch 'main' into feature/ccm-gas-limit
dandanlen Sep 14, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .cargo/config.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
[env]
CF_ETH_CONTRACT_ABI_ROOT = { value = "eth-contract-abis", relative = true }
CF_ETH_CONTRACT_ABI_TAG = "perseverance-0.9-rc6"
CF_ETH_CONTRACT_ABI_TAG = "v0.8.0"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a bit confusing... looks like we downgraded the contracts?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes I understand, but any follow-up tags can't be named perseverance-0.9-xyz as perseverance contracts have been deployed and will not changed. That would be even more confusing/wrong. So the best option to me seems going back to the normal tags and we launch mainnet with v1.0.0.

Copy link
Collaborator

@dandanlen dandanlen Sep 12, 2023 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about 0.10.0?



# Note: If you just want to save typing command commands, you can install tab completions for most shells. Type
Expand Down
2 changes: 1 addition & 1 deletion bouncer/run.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@ set -e
./commands/observe_block.ts 5
./commands/setup_vaults.ts
./commands/setup_swaps.ts
./tests/all_concurrent_tests.ts
./tests/gaslimit_ccm.ts
./tests/all_concurrent_tests.ts
./tests/rotates_through_btc_swap.ts

if [[ $LOCALNET == false ]]; then
Expand Down
2 changes: 1 addition & 1 deletion bouncer/shared/eth_abis.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ function loadContractCached(abiPath: string) {
return cached;
};
}
const CF_ETH_CONTRACT_ABI_TAG = 'perseverance-0.9-rc6';
const CF_ETH_CONTRACT_ABI_TAG = 'v0.8.0';
export const getErc20abi = loadContractCached('../eth-contract-abis/IERC20.json');
export const getGatewayAbi = loadContractCached(
`../eth-contract-abis/${CF_ETH_CONTRACT_ABI_TAG}/IStateChainGateway.json`,
Expand Down
Loading
Loading