hyperlane-monorepo

Commit Graph

Author	SHA1	Message	Date
Trevor Porter	97a1d0f959	Change ethereum finality in config 14 -> 20 (#1959 ) ### Description See https://github.com/hyperlane-xyz/issues/issues/415 for context Essentially all validators are on 20 block finality but we changed the relayer to 14 [recently](https://github.com/hyperlane-xyz/hyperlane-monorepo/pull/1675/files#diff-67e27b51135e9801b172be277ee0f667ed7b5ea5cb600f987bceb03b75eb3d65R33). It results in a weird scenario where the relayer will retry a message a few times before there's a quorum from validators. Moving back to 20 in the relayer ### Drive-by changes * Added names to validator addresses which I found useful to know at a glance ### Related issues - Fixes https://github.com/hyperlane-xyz/issues/issues/415 ### Backward compatibility _Are these changes backward compatible?_ yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ Sorta. We should re-deploy agents from this commit ### Testing _What kind of testing have these changes undergone?_ None	2 years ago
Mattie Conover	52404d25a6	Better context to track down errors during init (#1911 ) ### Description - Additional context to error to track down what chain's configuration is invalid. ### Drive-by changes None ### Related issues - Fixes https://discord.com/channels/935678348330434570/961711527092682783/1082783165803937853 ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ Manual	2 years ago
Mattie Conover	1c004f4b10	Remove gelato support (#1946 ) ### Description We have not been using Gelato anymore so thought it would be nice to remove this code and avoid continuing to maintain it. I know I have had to update it a few times to maintain parity. As an added perk it should slightly speed up build times since it removes an entire crate. ### Drive-by changes None ### Related issues ### Backward compatibility _Are these changes backward compatible?_ Yes - As we do not use this code _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ Unit Tests	2 years ago
Trevor Porter	61714e9ecd	Deploy on chain fee quoting onto mainnet (#1954 ) ### Description Enforce gas amounts on mainnet ### Drive-by changes none ### Related issues - Fixes #1610 ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ Deployed & sent some test messages	2 years ago
Yorke Rhodes	5b021c6242	Add fork testing to CI (#1931 ) ### Description - Add fork tests to CI - [x] check - [x] deploy - [x] govern - [x] check (after govern) ### Related issues - Fixes https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/1653 ### Backward compatibility Yes ### Testing CI --------- Co-authored-by: nambrot <nambrot@googlemail.com>	2 years ago
Trevor Porter	ad26ba661b	Deploy to testnet3 with arbitrum enforcement (#1951 ) ### Description Start enforcing arbitrum gas amounts after #1949 ### Drive-by changes none ### Related issues related: #1610 ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ Deployed and sent some messages	2 years ago
Trevor Porter	ae6b35d551	Deploy key funder (#1948 ) ### Description Deployed to include #1945 ### Drive-by changes n/a ### Related issues n/a ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ deployed	2 years ago
Trevor Porter	81ad2295fe	Move to estimateRetryableTicket to get L2 gas for arbitrum chains (#1949 ) ### Description * Moves to `estimateRetryableTicket` to get Arbitrum L2 gas ### Drive-by changes none ### Related issues - Fixes #1728 ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ Unit tests - haven't deployed yet tho	2 years ago
Mattie Conover	2c0b12528b	Support default signer configuration (#1908 ) ### Description - Adds a `defaultSigner` to settings which will automatically set all chains without their own signer configuration to use it. - Adds validation to relayers at startup to ensure that all destination chains have a signer configuration set. - Updates e2e test to validate this works ### Drive-by changes - Support `StrOrInt` or metric port deserialization. ### Related issues - Relates to #1856 ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ E2E Test	2 years ago
Trevor Porter	6e0c44a828	Use Polygon overrides when IGP claiming in key funder (#1945 ) ### Description For some extra context: #1934 and #1933 Calling IGP.claim() was not using the Polygon tx defaults. Therefore we'd use a priority fee < 30 gwei and have txs be rejected :\| ### Drive-by changes none ### Related issues n/a ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ Ran locally	2 years ago
Trevor Porter	23db81414d	Deploy key funder (#1938 ) ### Description Deploy key funder to include #1935 ### Drive-by changes none ### Related issues n/a ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ deployed	2 years ago
Trevor Porter	06d522dd92	Testnet3 gas amount enforcement (#1941 ) ### Description Deploys testnet3 to fully enforce gas amounts Note this doesn't enforce for arbitrumgoerli yet -- when testing things it seemed there was some flakiness in gas amount enforcement for arbitrum that I allude to in https://github.com/hyperlane-xyz/hyperlane-monorepo/pull/1940. Working on a more robust fix here ### Drive-by changes none ### Related issues related #1610 - mainnet & arbitrum chains will come soon ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ Deployed!	2 years ago
Yorke Rhodes	19d9450031	Remove coverage-sol PR comments (#1939 ) ### Description Removes PR comments that do not work on external contributions	2 years ago
Sergei Patrikeev	d510a19546	fix: index the message with zero nonce. (#1925 ) ### Description Proposed fix of #1924 ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ Manual --------- Co-authored-by: Trevor Porter <trkporter@ucdavis.edu> Co-authored-by: Nam Chu Hoai <nambrot@googlemail.com>	2 years ago
Trevor Porter	355168d66c	Graceful handling of errors with key funder's IGP claim or L2 bridging (#1935 ) ### Description Noticed that if an IGP claim fails, it's not gracefully handled ### Drive-by changes none ### Related issues n/a ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ Ran locally	2 years ago
Nam Chu Hoai	80005f01e9	Add Sepolia (#1930 ) ### Description This PR deploys Hyperlane to Sepolia. Updates packages to 1.2.2 to include sepolia ### Drive-by changes - Fixes ContractVerifier - Adds multiple bytecode hashes that are valid in HyperlaneCoreChecker ### Related issues - Fixes #1878 ### Backward compatibility Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None	2 years ago
Trevor Porter	20e6857ac1	Support DefaultIsmInterchainGasPaymaster in checker & govern (#1926 ) ### Description * Adds DefaultIsmInterchainGasPaymaster to the checker and govern logic * I'm aware this conflicts with https://github.com/hyperlane-xyz/hyperlane-monorepo/pull/1907 -- I wrote this mostly so I could propose new multisig txs to update the overhead gas amounts on mainnet. Happy to either merge this before or after https://github.com/hyperlane-xyz/hyperlane-monorepo/pull/1907, whatever works easiest ### Drive-by changes * Some fixes in the wake of the sdk refactor, e.g.: * if some tx params like `maxPriorityFeePerGas` or `maxFeePerGas` were specified when we estimate gas, it can fail if the balance of the sender is insufficient. For cases where we want to estimate gas from the owner multisig which may not have native tokens, this caused issues. * We should've been passing the `from` address into the MultiProvider.estimateGas fn as a separate param ### Related issues n/a ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ Ran & proposed some txs on mainnet to update the destination gas overheads	2 years ago
Trevor Porter	ce16fa7421	Deploy kathy to use new contracts (#1923 ) ### Description Includes #1922 ### Drive-by changes none ### Related issues n/a ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ Deployed	2 years ago
Trevor Porter	5ccba5570d	Deploy new helloworld contracts (#1922 ) ### Description Deployed helloworld contracts on mainnet2 & testnet3 hyperlane contexts. The currently deployed version reads gas amounts from a mapping that isn't populated. The latest helloworld contracts just use a constant 50k gas. The deployed rc contexts already use this constant gas amount ### Drive-by changes n/a ### Related issues n/a ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ Sorta - we need to deploy testnet3 and mainnet2 kathy from a monorepo commit following this change so that the new contracts are used ### Testing _What kind of testing have these changes undergone?_ Deployed	2 years ago
Asa Oines	8c456c97f4	Minor SDK refactor (#1904 ) ### Description Move `sdk/src/deploy/X` to `sdk/src/X`, where appropriate Consumers of the SDK should not need to make any changes ### Drive-by changes None ### Related issues Using this as a base for moving some of the IGP deployment tooling out of `HyperlaneCoreDeployer` into `HyperlaneIgpDeployer` ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ None --------- Co-authored-by: Trevor Porter <trkporter@ucdavis.edu>	2 years ago
Trevor Porter	dd5478f740	Deploy agents, deploy on chain fee quoting to testnet3 rc (#1921 ) ### Description * Routine agent deploy to all envs / contexts * Fixed a JSON deserialization issue where a U256 must be a string and not a number * Added the on chain fee quoting gas enforcement policy to infra * Deployed to testnet3 rc with on chain fee quoting gas enforcement policy ### Drive-by changes n/a ### Related issues n/a ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ not really - but bc this includes an infra fix, you should deploy using this commit or later ### Testing _What kind of testing have these changes undergone?_ Deployed	2 years ago
Trevor Porter	cd75c25f40	Check IGP ownership, transfer IGP ownership in the initializer (#1895 ) ### Description * Adds the IGP & DefaultIsmIgp to the HyperlaneCoreChecker's list of ownables * Transfers ownership in the `initialize` function of the IGP - this way we won't ever get in a scenario where the proxy admin is the owner for fresh deploys ### Drive-by changes * The provided default mumbai rpc `https://rpc-mumbai.maticvigil.com` was giving problems for eth_getStorageAt, so changed it * Removed the DefaultIsmInterchainGasPaymaster from the list of proxies because it's not proxied (this is in #1765, but made this change in here too) ### Related issues n/a ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ Unit Tests	2 years ago
J M Rossy	406d365e03	Bump versions for 1.2.1 (#1915 ) - Increment package versions to 1.2.1 - Update submodules for their version bumps	2 years ago
Nam Chu Hoai	5152da508b	Validator metadata name matches key (#1918 ) ### Description If you use a different name, then MP lookup can fail. Thus we should validate that the key matches the name	2 years ago
Trevor Porter	1cbe5fd0a0	Update gas oracle exchange rates & IGP bytecode hash (#1919 ) ### Description Just updates mainnet gas oracle token exchange rates Gas prices were the same ### Drive-by changes none ### Related issues n/a ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ Deployed	2 years ago
Trevor Porter	7fb935e801	Arbitrum gas amount handling in gas enforcement policies (#1912 ) ### Description See #1728 for detailed explanation Uses Arbitrum's NodeInterface to get the L1 cost of a tx so we can isolate the L2 gas and use that for gas enforcement policies. Very open to alternative ideas in the way I wrote it as well ### Drive-by changes No ### Related issues - Fixes #1728 ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ Unit Tests - haven't deployed yet, but plan to deploy to rc testnet soon	2 years ago
Nam Chu Hoai	97c7f52801	Remove GasPaymentEnforcementPolicy::MeetsEstimatedCost (#1910 ) ### Description As discussed offline, to establish the social contract of the relayer accepting the quotes of its associated IGP on the origin chain, this PR removes the GasPaymentEnforcementPolicy that was assessing gas payments at tx submission time via off-chain oracles.	2 years ago
Trevor Porter	61a09a6554	Deploy key funder with IGP claiming & higher kathy balances (#1906 ) ### Description Deploys key funder on testnet3 and mainnet2 to include https://github.com/hyperlane-xyz/hyperlane-monorepo/pull/1883 ### Drive-by changes none ### Related issues n/a ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ Deployed	2 years ago
J M Rossy	319838e871	Improve MultiProvider explorer url handling (#1898 ) - Improve MultiProvider explorer url handling - Add ApiKey to ChainMetadata - Add zod schema validation for ChainMetadata - Fix bug in HyperlaneCore message fetching	2 years ago
Mattie Conover	0049cca1ba	Support On Chain Fee Quoting (#1658 ) ### Description * New gas enforcement policy that uses the on-chain-fee-quoting values. * Change gas enforcement policies to return the gas limit that should be used to submit the transaction. * Change gelato submitter to use the gas limit provided by the gas enforcement policy. * Now indexes the gas amounts for the payments A couple of notes: * We might want to test this with the e2e test. * I took a look into the retying provider and it looks like it already does not retry on failed transaction submissions so I don't think anything needs to be done there after all. ### Drive-by changes ### Related issues - Fixes #1535 ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ Yes - changes the internal database format for message payments ### Testing _What kind of testing have these changes undergone?_ Unit Tests --------- Co-authored-by: Trevor Porter <trkporter@ucdavis.edu>	2 years ago
Trevor Porter	59a90b1bb6	Default to not allowing LocalStorage checkpoint syncers in relayer (#1900 ) ### Description Open to better names Adds a setting to the relayer, `allow_local_checkpoint_syncers`, which determines whether local storage based checkpoint syncers will be allowed by the metadata builder. Originally, I wanted something a bit more clever, like being able to specify `HYP_RELAYER_VALIDCHECKPOINTSYNCERS=LocalStorage,S3` or something, where I'd like those variants to be matched to the variants found in https://github.com/hyperlane-xyz/hyperlane-monorepo/blob/main/rust/hyperlane-base/src/types/checkpoint_syncer.rs#L14. But that enum requires values, so things get ugly. One option would be to create a new enum like: ``` enum CheckpointSyncerTypes { LocalStorage, S3, } ``` And another option is to use something like strum's [EnumString](https://docs.rs/strum/latest/strum/derive.EnumString.html) (shoutout to @mattiecnvr). But this still is a bit clunky, so for now just making this a bool and we can figure out something more elegant later if we ever get to a point where we're supporting multiple types of checkpoint syncers ### Drive-by changes none ### Related issues - Fixes https://github.com/hyperlane-xyz/issues/issues/402 ### Backward compatibility _Are these changes backward compatible?_ Yes - although if you ever want to run a relayer that uses local storage now, you'll need to set `HYP_RELAYER_ALLOWLOCALCHECKPOINTSYNCERS=true` _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None - we always expect to not be reading from the local fs in deployed relayers ### Testing _What kind of testing have these changes undergone?_ Ran e2e tests	2 years ago
Trevor Porter	07f91490fc	Fix flaky fuzz test (#1903 ) ### Description I believe this is the fix for the flaky fuzz tests we've been experiencing. My theory is that the IGP's balance was overflowing by there being 2 subsequent `payForGas` calls with each payment being > `type(uint256).max / 2`. I found that this test was the culprit by running specific fuzz tests we have. Then made the fix and ran it multiple times and never saw an overflow again ### Drive-by changes n/a ### Related issues n/a ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ Unit Tests	2 years ago
Mattie Conover	599722a7ca	Load all configs by default (#1887 ) ### Description This removes `RUN_ENV` and `BASE_CONFIG` from the Rust config loading process and also from the helm charts and TS configs as appropriate. ### Drive-by changes - Flattens config dir structure - Removes the test config from the git repo. - Fixed a bug in the deploy script that caused the e2e pipeline to break when the config file was removed - Removes `environment` from the config files - Updates TS code to use `runEnv` for deployment - Updates TS code to type environment with `DeployEnvironment` instead of `string` ### Related issues - Relates to #1850 ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ Manual - Updated e2e test to verify these don't cause a config loading issue.	2 years ago
Asa Oines	0ab0ed0331	Pin foundry version in CI (#1901 ) ### Description Pin the foundry version in CI to the nightly from a few days ago. Unfortunately the only releases are nightly. ### Drive-by changes None	2 years ago
Trevor Porter	854cf163f6	Update token submodule commit (#1897 )	2 years ago
Trevor Porter	f5d1fbbd03	On chain gas oracles in the IGP (#1765 ) * Rough implementations of all gas oracles, done with StaticGasOracle & tests * more tests, beef up StorageGasOracle * Tests cleaned up, moved to Foundry test for IGP * rm StaticGasOracle, fix all tests * cleaning up * comment * yarn gas, better comments * Fix sdk test * Fix e2e * Rm LZ stuff * fix router test * Rm MockInterchainGasPaymaster * pr comments * Move to setGasConfigs instead of setGasConfig * fix test * nit * yarn gas * IGP deploy & checker tooling, script to update on-chain StorageGasOracle data (#1794) * Add IGP gas oracle to checker * Checker works * Support different gas oracle contracts in IGP deploy tooling too * Export some types * Build out set-storage-gas-oracle-values * nit * nit * rm comment * Add pricing data * nits * nearly there * haven't tested e2e yet, but added new proxy upgrade logic to set gas oracles in IGP in upgradeAndCall * pr comments * lots * Move away from upgradeAndCall for IGP upgrade, trying to debug annoying dependency issue * Got working, cleaned a little * Add test to ensure a new implementation is deployed * yarn hyperlane * nits * cleaning * Map in description * Dedupe submission type logic * yarn gas * Separate beneficiary in IGP (#1801) * Add IGP gas oracle to checker * Checker works * Support different gas oracle contracts in IGP deploy tooling too * Export some types * Build out set-storage-gas-oracle-values * nit * nit * Add separate beneficiary to IGP * Checker / govern / deploy tooling * nits * Add compare-storage-gas-oracle * successful test upgrade, minor changes * Nits * Fix tests * Performed testne3 upgrade. Need to clean up code, but should probably use this for the mainnet upgrade * nit * update prices * Update gas prices and token exchange rates for mainnet * mainnet upgrade. needs lots of cleaning, but here it is in all its glory * Starting to clean up * more cleanup * Use updated submodules * yarn gas * nit * PR comments * yarn hyperlane * yarn gas	2 years ago
Trevor Porter	5d9372603e	Fund kathy more on mainnet, add IGP claiming to key funder (#1883 ) Fund kathy more on mainnet, add claiming to key funder	2 years ago
Trevor Porter	63db3ceeb8	Remove testnet3 relayer blacklist (#1890 ) Remove testnet3 blacklist	2 years ago
Mattie Conover	836179c9ad	Better default stack traces (#1886 ) * Update to use color-eyre by default for stack traces * Use backtrace setting to define config formatting for debugging	2 years ago
Mattie Conover	6ad3b39114	Allow configs to use number or strings for numerics (#1885 ) * Allow strings or integers for configuration * Update TS for new integer change * Fixed finality blocks parsing	2 years ago
Mattie Conover	7a121a8d80	Make relayer require destination chains (#1884 ) * Make destination chains required for relayers * Update helm chart to include destination chains * Added notice for missing destination chain names	2 years ago
Asa Oines	33a77c35e1	Submit txs with EIP 1559 in relayer (#1869 )	2 years ago
Asa Oines	653ddc82e7	Serialize concatenated signature (#1877 )	2 years ago
Mattie Conover	55242e65ce	Better errors for missing config values (#1814 ) * Better errors for missing config values * Cleanup * Added more help on missing name * Fix lints * fix format * fix format	2 years ago
Yorke Rhodes	582d45d029	Proxy middlewares in SDK deployers (#1838 ) * Proxy middlewares in SDK deployers * Small mods * Fix build	2 years ago
Mattie Conover	03d32f4009	Eliminate trivial constructors (#1847 )	2 years ago
J M Rossy	b9dfbdb31b	Update token ref (#1873 )	2 years ago
Mattie Conover	1b45f8cad1	Rust Updates (#1845 ) * Update rust lang version * Fix new clippy lint * Update packages * Update rocksdb * Updates	2 years ago
J M Rossy	5f1278abd1	Fix lint issues in core package readme (#1839 )	2 years ago
J M Rossy	e322270477	Update infra package for SDK 1.2.0 (#1836 ) - Update infra code to work with update SDK types - Include some minor cosmetic code improvements	2 years ago

1 2 3 4 5 ...

1333 Commits (97a1d0f959c3b23c9a53a55f694f19666220e264) All Branches Search

1333 Commits (97a1d0f959c3b23c9a53a55f694f19666220e264)

All Branches