hyperlane-monorepo

Commit Graph

Author	SHA1	Message	Date
J M Rossy	05a943b4ad	CLI pre-flight check fix (#3089 ) ### Description Only check origin balances for CLI send commands ### Related issues Fixes https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3079 ### Backward compatibility Yes ### Testing Tested command locally	11 months ago
Daniel Savu	c2cf7be8f2	feat(cosmos): ism dry-run (#3077 ) ### Description Implements `dry_run_verify` for the aggregation ISM on cosmwasm. One remaining issue is that the estimated gas is hardcoded to `1`, because we're actually [just querying](`37fea49429/contracts/isms/aggregate/src/lib.rs (L108)`) via rpc rather than simulating a tx. The `verify` tx isn't marked as a contract [entrypoint](https://book.cosmwasm.com/basics/entry-points.html) so it can't be called from outside iiuc. (here's the [verify](`37fea49429/contracts/isms/aggregate/src/lib.rs (L124)`) fn for reference). Worth mentioning that the query interface for the aggregation ISM is named incorrectly - it should return fields called `threshold` and `modules`, but instead copies the response from the multisig ISM and returns `threshold` and `validators`. This can be particularly misleading because validators have 20-bytes long addresses, whereas modules (contracts) have 32-bytes long addresses. ### Related issues - Fixes https://github.com/hyperlane-xyz/issues/issues/807 ### Backward compatibility yes ### Testing E2E. The ISM setup is `routing` -> `aggregation (1/1)` -> `multisig (1/1)`	11 months ago
Daniel Savu	f73ee0b273	last round of PR remediations (#2791 ) Remediations for https://github.com/hyperlane-xyz/hyperlane-monorepo/pull/2746, in addition to https://github.com/hyperlane-xyz/hyperlane-monorepo/pull/2780	11 months ago
Daniel Savu	5c8d483b37	feat: cargo release profile in ci (#2980 )	11 months ago
Yorke Rhodes	f7d285e3a5	Add test recipients to SDK addresses (#3086 ) ### Description Add test recipients to SDK addresses --------- Co-authored-by: J M Rossy <jm.rossy@gmail.com>	11 months ago
Yorke Rhodes	655b6a0cd8	Redeploy routing ISM factories (#3085 )	11 months ago
J M Rossy	1918560609	Reduce verbosity of smartprovider logging (#3084 )	11 months ago
Daniel Savu	d18f7ae8e1	Cosmos gas price config (#3042 ) ### Description Adds a cosmos-specific config item for setting the minimum gas price, in the format returned by the `cosmos.base.node.v1beta1.Service/Config` cosmos-sdk grpc endpoint. ### Related issues - Fixes https://github.com/hyperlane-xyz/issues/issues/810 ### Backward compatibility No. Will break existing cosmos configs. ### Testing None yet, will test in e2e	11 months ago
Daniel Savu	8e44bc1b81	feat: sealevel merkle indexer + e2e (#3053 ) Depends on https://github.com/hyperlane-xyz/hyperlane-monorepo/pull/3048 Fixes https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3046 Fixes https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/2754	11 months ago
Daniel Savu	1f651b7afc	chore: remove neutron-related deployments from testnet4 context (#3071 ) ### Description Removes most occurences of lines that contain the `neutron` string from the testnet4 context. Where validators are set (e.g. [here](`3e685f9fb0/typescript/infra/config/environments/testnet4/validators.ts (L29)`)), the neutron context is required for type safety but can stay empty	11 months ago
J M Rossy	08ba0d32b9	Remove dead goerli arbiscan link (#3073 )	11 months ago
Trevor Porter	6f9b4e6660	Bump desired AVAX balance (#3068 ) ### Description Avalanche is like 5x more expensive now, see relayer txs https://snowtrace.io/address/0x74Cae0ECC47B02Ed9B9D32E000Fd70B9417970C5. A single funding of 0.3 is only enough to cover like 2 messages ### Drive-by changes <!-- Are there any minor or drive-by changes also included? --> ### Related issues <!-- - Fixes #[issue number here] --> ### Backward compatibility <!-- Are these changes backward compatible? Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling? Yes/No --> ### Testing <!-- What kind of testing have these changes undergone? None/Manual/Unit Tests -->	11 months ago
Yorke Rhodes	0f47fc21b0	Update issue templates (#3066 )	11 months ago
Yorke Rhodes	10c0ab17d1	Deploy eclipse fi warp (#3062 )	11 months ago
github-actions[bot]	0d9d1d26fa	Version Packages (#3028 ) # Releases ## @hyperlane-xyz/sdk@3.4.0 ### Minor Changes - b832e57ae: Replace Fallback and Retry Providers with new SmartProvider with more effective fallback/retry logic ### Patch Changes - 7919417ec: Granular control of updating predeployed routingIsms based on routing config mismatch - Add support for routingIsmDelta which filters out the incompatibility between the onchain deployed config and the desired config. - Based on the above, you either update the deployed Ism with new routes, delete old routes, change owners, etc. - `moduleMatchesConfig` uses the same - fd4fc1898: - Upgrade Viem to 1.20.0 - Add optional restUrls field to ChainMetadata - Add deepCopy util function - Add support for cosmos factory token addresses - e06fe0b32: Supporting DefaultFallbackRoutingIsm through non-factory deployments - 79c96d718: Remove healthy RPC URLs and remove NeutronTestnet - Updated dependencies [`fd4fc1898`] - Updated dependencies [`e06fe0b32`] - @hyperlane-xyz/utils@3.4.0 - @hyperlane-xyz/core@3.4.0 ## @hyperlane-xyz/core@3.4.0 ### Patch Changes - e06fe0b32: Supporting DefaultFallbackRoutingIsm through non-factory deployments - Updated dependencies [`fd4fc1898`] - @hyperlane-xyz/utils@3.4.0 ## @hyperlane-xyz/cli@3.4.0 ### Patch Changes - e06fe0b32: Supporting DefaultFallbackRoutingIsm through non-factory deployments - dcf8b800a: Fixes for commands with --yes flag - 9c7dbcb94: Remove domainId and protocolType setting when creating chain config - Updated dependencies [`7919417ec`] - Updated dependencies [`fd4fc1898`] - Updated dependencies [`e06fe0b32`] - Updated dependencies [`b832e57ae`] - Updated dependencies [`79c96d718`] - @hyperlane-xyz/sdk@3.4.0 - @hyperlane-xyz/utils@3.4.0 ## @hyperlane-xyz/helloworld@3.4.0 ### Patch Changes - Updated dependencies [`7919417ec`] - Updated dependencies [`fd4fc1898`] - Updated dependencies [`e06fe0b32`] - Updated dependencies [`b832e57ae`] - Updated dependencies [`79c96d718`] - @hyperlane-xyz/sdk@3.4.0 - @hyperlane-xyz/core@3.4.0 ## @hyperlane-xyz/utils@3.4.0 ### Patch Changes - fd4fc1898: - Upgrade Viem to 1.20.0 - Add optional restUrls field to ChainMetadata - Add deepCopy util function - Add support for cosmos factory token addresses ## @hyperlane-xyz/infra@3.4.0 ### Patch Changes - e06fe0b32: Supporting DefaultFallbackRoutingIsm through non-factory deployments - Updated dependencies [`7919417ec`] - Updated dependencies [`fd4fc1898`] - Updated dependencies [`e06fe0b32`] - Updated dependencies [`b832e57ae`] - Updated dependencies [`79c96d718`] - @hyperlane-xyz/sdk@3.4.0 - @hyperlane-xyz/utils@3.4.0 - @hyperlane-xyz/helloworld@3.4.0 Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>	11 months ago
J M Rossy	fd4fc1898c	Upgrade Viem & fix neutron chainmetadata (#3049 ) ### Description - Upgrade Viem and wagmi chains versions - Add optional RestUrl field to ChainMetadata - Set rest url for Neutron - Add simple deepCopy function Prerequisite for https://github.com/hyperlane-xyz/hyperlane-warp-ui-template/issues/80 ### Backward compatibility Yes ### Testing Tested in Warp UI	11 months ago
J M Rossy	dcf8b800a9	Fixes for commands with --yes flag (#3063 ) ### Description Consistently respect --yes flag for all user prompts in the core and warp deploy commands ### Related issues Fixes #3056 ### Backward compatibility Yes ### Testing Locally with CI test alterations	11 months ago
Nam Chu Hoai	9c7dbcb947	Default domainId and protocolType (#3041 ) ### Description We only support EVM chains today, so folks pretty much should always use the chainId as the domainId and not choose any other protocol type other than ethereum. So removing the optionality for now until that changes	11 months ago
Daniel Savu	8d8e3b8bc1	Deploy relayer balance fix (#3037 )	11 months ago
Daniel Savu	42ea929785	Sealevel v3 contracts (#3048 ) ### Description MVP contracts for sealevel v3 - note this doesn't include hooks so IGP payments still have to be made separately. However contracts will now be compatible with the metadata format used by the other V3 deployments. ### Drive-by changes The unit tests in `rust/sealevel/programs/ism/multisig-ism-message-id/src/metadata.rs` are refactored to reuse struct creation logic. ### Related issues - Fixes https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3044 - Fixes https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3045 ### Backward compatibility No, since this upgrades sealevel v2 to v3. ### Testing Unit tests	12 months ago
Nam Chu Hoai	e65c2a321c	Manage core governance (#3029 ) ### Description Changes that were required to run the deployer/check to enroll more validators. Some of the changes like removing artifacts from the SDK or changing the chains in infra probably shouldn't be checked in. ### Drive-by changes - Added another error message for mailbox initialization check (since there is no view call to make) - Ignoring validator announce violations in the governor ### Related issues Fixes https://github.com/hyperlane-xyz/issues/issues/673 --------- Co-authored-by: Yorke Rhodes <yorke@hyperlane.xyz>	12 months ago
Kunal Arora	cbea29091d	fix:add forge tests for `TestSendReceiver` (#3055 ) ### Description - Added `TestSendReceiverTest` after having removed `testSendReceiver.hardhat-test.ts` (latter uses blockhash randomness and can fail in CI infrequently but very frustratingly whereas the former is fuzzed and conditionally checked) ### Drive-by changes None ### Related issues None ### Backward compatibility Yes ### Testing Fuzz tests --------- Co-authored-by: Yorke Rhodes <yorke@hyperlane.xyz>	12 months ago
Nam Chu Hoai	a888cf7ce9	Actually default to none policy (#2921 ) (#3058 ) ### Description Cherry-picks #2921 which got merged into the wrong branch	12 months ago
Yorke Rhodes	ef2ece300e	Fix slither CI action (#3054 )	12 months ago
Kunal Arora	7919417ecd	feat: add routing delta for more granular routingIsm config updates (#3017 ) ### Description - add support for `routingIsmDelta` which filters out the incompatibility between the onchain deployed config and the desired config. - based on the above, you either update the deployed Ism with new routes, delete old routes, change owners, etc. - moduleMatchesConfig uses the same ### Drive-by changes - checking exclusion of domains in the onchain deployed config ### Related issues - fixes https://github.com/hyperlane-xyz/issues/issues/818 ### Backward compatibility Yes ### Testing Unit tests --------- Co-authored-by: Yorke Rhodes <yorke@hyperlane.xyz>	12 months ago
J M Rossy	b832e57aef	Replace Fallback/Retry providers with SmartProvider (#3032 ) ### Description - Port `HyperlaneSmartProvider` from explorer - Use as default ethers v5 provider - Includes `HyperlaneJsonRpcProvider` and `HyperlaneEtherscanProvider` for more granular control ### Related issues Fixes https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/2024 ### Backward compatibility No, the RetryProvider was removed ### Testing Unit tests with Foundry/anvil E2E tests with Goerli via mocha E2E tests via metadata-health CI	12 months ago
Ford	79bd348585	chore(docs): typo fixes (#3051 )	12 months ago
NicholasDotSol	dae8b26ed6	StandardHookMetadata.sol small doc fix (#3043 ) ### Description small fix to docs	12 months ago
J M Rossy	79c96d7187	Metadata Health Checks (#3030 ) ### Description - Create test to ensure SDK metadata RPCs and Explorers are healthy - Exercise explorer link utils - Enable `sort-keys` eslint rule for SDK metadata - Cleanup metadata and remove unhealthy RPC urls - Remove NeutronTestnet ### Related issues Fixes https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/2997 ### Backward compatibility Yes	12 months ago
Trevor Porter	95ab709d96	Deploy kathy (#3040 ) ### Description Deploy kathy - seems testnet4 kathy was having issues, this fixes it ### Drive-by changes <!-- Are there any minor or drive-by changes also included? --> ### Related issues <!-- - Fixes #[issue number here] --> ### Backward compatibility <!-- Are these changes backward compatible? Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling? Yes/No --> ### Testing <!-- What kind of testing have these changes undergone? None/Manual/Unit Tests -->	12 months ago
Daniel Savu	86b7f98850	fix: cosmos balance parsing (#3036 ) @tkporter reported that cosmos balance strings were being interpreted as hex when parsed, causing them to appear much larger. This fixes by parsing as decimal. Tested manually by checking the metrics endpoint.	12 months ago
Trevor Porter	313a2ab357	Kathy keys using correct env/context helloworld chains, small fixes (#3035 ) ### Description - Noticed an issue with #3023 where mainnet3 helloworld addresses were always used. This uses the correct environment and context - Also noticed errors with key funder infra because the neutron rpc url wasn't present - fixed the previous hack to accommodate this - Filter out non evm or blessed chains from key funder as well ### Drive-by changes n/a ### Related issues related to #3024 ### Backward compatibility yes ### Testing ran locally	12 months ago
Nam Chu Hoai	980ef1fc4e	Revert "Add figlet to CLI (#3008 )" (#3033 ) This reverts commit `c606b6a485`. ### Description <!-- What's included in this PR? --> ### Drive-by changes <!-- Are there any minor or drive-by changes also included? --> ### Related issues <!-- - Fixes #[issue number here] --> ### Backward compatibility <!-- Are these changes backward compatible? Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling? Yes/No --> ### Testing <!-- What kind of testing have these changes undergone? None/Manual/Unit Tests -->	12 months ago
Trevor Porter	1a64cef37e	Refactor key management in infra and key funder (#3023 ) ### Description * Remove the `bank` role, which we haven't used since the inception of abacus / hyperlane * Big changes to `key-utils.ts` so that there's a single source of truth on what kind of keys are used depending on the role & chain. Before this was sprinkled in a few different places * You can now get an object of `{ [chain]: { [role]: keys[] } }`, so it's super clear what kind of key relates to which chain. For example, before we would use the AWS-based relayer key for EVM chains, and then a GCP-based relayer key for non-EVM chains. But this wasn't really honored by key funder - it had no way of knowing to only fund the AWS relayer on EVM chains, and only fund the GCP relayer on non-EVM chains. Same situation for Kathy, where we want to use AWS keys for EVM chains but the GCP key for non-EVM chains * On v2 and prior to that we were using the AWS-based key for Kathy. Originally, we also launched v3 this way. However it was changed on v3 to use the GCP key for Kathy, causing us to fund both types of addresses on v3. This makes it more clear that we should be using the AWS-based key for Kathy on EVM chains ### Drive-by changes <!-- Are there any minor or drive-by changes also included? --> ### Related issues <!-- - Fixes #[issue number here] --> ### Backward compatibility <!-- Are these changes backward compatible? Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling? Yes/No --> ### Testing <!-- What kind of testing have these changes undergone? None/Manual/Unit Tests -->	12 months ago
Kunal Arora	e06fe0b327	fix:enable `DefaultFallbackRoutingIsm` through non-factory deployment (#3009 ) ### Description - `DefaultFallbackRoutingIsm` needs the mailbox address which meant `DefaultFallbackRoutingIsmFactory` needed the mailbox address which is not ideal. So, I switched to non-factory deployments for this specific ISM type. - CLI does the ISM deployment inside core deployment instead of ISM first and core then. - Using the coreDeployer's `cachedAddresses` to store the latest ie toplevel ISM. ### Drive-by changes - fixed bug in `ismFactory.fromAddressMap` to not use the multiprovider filtered by contract map. This is undesirable as CLI user can provide chain selection and artifacts which doesn't include all the chains in the configs (and when we call `multiprovider.getDomainId` for the origin which may now be missing from the filtered MP, we get an error). ### Related issues - fixes https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/2895 ### Backward compatibility Yes ### Testing Manual through CLI	12 months ago
github-actions[bot]	8b16adee48	Version Packages (#3002 ) # Releases ## @hyperlane-xyz/cli@3.3.0 ### Minor Changes - 7e620c9df: Allow CLI to accept hook as a config ### Patch Changes - f44589e45: Improve warp and kurtosis deploy command UX - 2da6ccebe: Allow users to only configure validators for their chain - Don't restrict user to having two chains for ism config - If the user accidentally picks two chains, we prompt them again to confirm if they don't want to use the hyperlane validators for their multisigConfig - 9f2c7ce7c: Removing agentStartBlocks and using mailbox.deployedBlock() instead - 9705079f9: Improve UX of the send and status commands - c606b6a48: Add figlet to CLI - Updated dependencies [`7e620c9df`] - Updated dependencies [`350175581`] - Updated dependencies [`9f2c7ce7c`] - @hyperlane-xyz/sdk@3.3.0 - @hyperlane-xyz/utils@3.3.0 ## @hyperlane-xyz/core@3.3.0 ### Patch Changes - 350175581: Rename StaticProtocolFee hook to ProtocolFee for clarity - @hyperlane-xyz/utils@3.3.0 ## @hyperlane-xyz/helloworld@3.3.0 ### Patch Changes - Updated dependencies [`7e620c9df`] - Updated dependencies [`350175581`] - Updated dependencies [`9f2c7ce7c`] - @hyperlane-xyz/sdk@3.3.0 - @hyperlane-xyz/core@3.3.0 ## @hyperlane-xyz/sdk@3.3.0 ### Patch Changes - 7e620c9df: Allow CLI to accept hook as a config - 350175581: Rename StaticProtocolFee hook to ProtocolFee for clarity - 9f2c7ce7c: Removing agentStartBlocks and using mailbox.deployedBlock() instead - Updated dependencies [`350175581`] - @hyperlane-xyz/core@3.3.0 - @hyperlane-xyz/utils@3.3.0 ## @hyperlane-xyz/utils@3.3.0 ## @hyperlane-xyz/infra@3.3.0 ### Patch Changes - 7e620c9df: Allow CLI to accept hook as a config - 9f2c7ce7c: Removing agentStartBlocks and using mailbox.deployedBlock() instead - Updated dependencies [`7e620c9df`] - Updated dependencies [`350175581`] - Updated dependencies [`9f2c7ce7c`] - @hyperlane-xyz/sdk@3.3.0 - @hyperlane-xyz/helloworld@3.3.0 - @hyperlane-xyz/utils@3.3.0 Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>	12 months ago
Daniel Savu	e791336115	Deploy relayer balance metrics (#3027 )	12 months ago
Daniel Savu	36de5bcc3c	chore: reduce agents log verbosity, reword (#3026 ) There are way too many `No message found in DB for leaf index` logs to have them at debug level. Also the wording makes it seem like an error when it's not.	12 months ago
Daniel Savu	98bb228a96	Sealevel balance metrics (#3025 ) Publishes sealevel balance as a relayer metric Drive-by Changes - Creates `HyperlaneSealevelError`, a `hyperlane-sealevel` specific type to avoid including sealevel errors in `hyperlane-core`'s `ChainCommunicationError`. It's pretty empty now, but we have a starting point now. This follows the approach used in `hyperlane-cosmos` too. - Includes an rpc provider instance in `SealevelProvider`, and uses it to add `get_balance` on `SealevelProvider`. Testing None, as sealevel e2e is currently disabled	12 months ago
Daniel Savu	77aa58c581	Relayer balance metrics (#2976 ) Done: - scaffolding for fetching custom agent metrics - abstractions for building a metrics fetcher for a given VM - querying cosmos balances; e2e tested. - querying evm balances; e2e tested. - Note that as a result, evm addresses are now no longer zero-padded when printed in the logs. This may break existing log queries - fixed a nasty bug on ubuntu where wasmd (osmosisd dependency, part of the grpc query flow) would panic when a block is specified via `x-cosmos-block-height`. The fix was to bump the version of osmosisd from `19.0.0` to `20.5.0`. Note that when running e2e on Mac OS, the osmosis version in use is still 19.0.0. That's because we need a fork that publishes a darwin target binary (currently pointing [here](https://github.com/hashableric/osmosis/releases/download/v19.0.0-mnts/osmosisd-19.0.0-mnts-darwin-arm64.tar.gz)) For follow up PR: - sealevel balance querying I'm open to all renaming suggestions, I just tried to speed through and didn't ponder names too much Relates to https://github.com/hyperlane-xyz/issues/issues/701 Closes https://github.com/hyperlane-xyz/issues/issues/702 (because the balance becomes available in the metrics endpoint for polling)	12 months ago
Kunal Arora	2da6ccebe8	fix:don't prompt user for multisig config if chain is core (#3021 ) ### Description - don't restrict user to having two chains for ism config - if the user accidentally picks two chains, we prompt them again to confirm if they don't want to use the hyperlane validators for their multisigconfig ### Drive-by changes None ### Related issues - fixes https://github.com/hyperlane-xyz/issues/issues/811 ### Backward compatibility Yes ### Testing Manual between arbitrumgoerli and anvil1	12 months ago
Yorke Rhodes	c606b6a485	Add figlet to CLI (#3008 )	12 months ago
J M Rossy	c65b598719	Remove demand option for router arg of send transfer (#3018 ) Fix for missing bit of #3006	12 months ago
J M Rossy	9705079f99	Improve UX of send and status commands (#3006 ) ### Description - Improve shared context utility with conditional typing and core artifact handling - Improve UX of send and status commands with prompts - Prompt for key if required and not provided ### Drive-by Changes Update the token readme for https://github.com/hyperlane-xyz/issues/issues/715 ### Backward compatibility Yes ### Testing Manual	12 months ago
Kunal Arora	d0c1f8f11e	Rename `hook-config.yaml` to `hooks.yaml` in `ci-test.sh` (#3016 ) ### Description - CI fails otherwise ### Drive-by changes <!-- Are there any minor or drive-by changes also included? --> ### Related issues <!-- - Fixes #[issue number here] --> ### Backward compatibility <!-- Are these changes backward compatible? Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling? Yes/No --> ### Testing Manual	12 months ago
J M Rossy	eb83cf891c	Enable for-in lint rule (#3013 ) ### Description https://eslint.org/docs/latest/rules/guard-for-in ### Drive-by changes Fix lint error in SDK Sealevel adapter ### Backward compatibility Yes	12 months ago
Kunal Arora	7e620c9dfa	feat:configure hooks in the CLI (#2964 ) ### Description - enable configuring hooks in the CLI (merkle, igp, protocolFee, aggregation, routing) - use preset by default without prompting ### Drive-by changes <!-- Are there any minor or drive-by changes also included? --> ### Related issues <!-- - Fixes #[issue number here] --> ### Backward compatibility <!-- Are these changes backward compatible? Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling? Yes/No --> ### Testing <!-- What kind of testing have these changes undergone? None/Manual/Unit Tests -->	12 months ago
Trevor Porter	9fc0866b29	Fix base bridging and a race condition in key funder (#3012 ) ### Description Turns out bridging from L1 to Base has been broken the whole time :( it's because the version of @eth-optimism/sdk we were using was 1.8.0 (according to yarn.lock) which was released in late 2022 https://github.com/ethereum-optimism/optimism/releases/tag/%40eth-optimism%2Fsdk%401.8.0, far before Base was launched. So we'd always get this error in key funder because we rely on the optimism SDK knowing about Base's chain ID: ``` {"chain":"base","error":"Error: cannot get contract AddressManager for unknown L2 chain ID 8453, you must provide an address\n at getOEContract (/hyperlane-monorepo/node_modules/@eth-optimism/sdk/src/utils/contracts.ts:58:11)\n at getAllOEContracts (/hyperlane-monorepo/node_modules/@eth-optimism/sdk/src/utils/contracts.ts:121:46)\n at new CrossChainMessenger (/hyperlane-monorepo/node_modules/@eth-optimism/sdk/src/cross-chain-messenger.ts:170:39)\n at ContextFunder.bridgeToOptimism (/hyperlane-monorepo/typescript/infra/scripts/funding/fund-keys-from-deployer.ts:652:33)\n at ContextFunder.bridgeToL2 (/hyperlane-monorepo/typescript/infra/scripts/funding/fund-keys-from-deployer.ts:637:23)\n at processTicksAndRejections (node:internal/process/task_queues:96:5)\n at async ContextFunder.bridgeIfL2 (/hyperlane-monorepo/typescript/infra/scripts/funding/fund-keys-from-deployer.ts:492:9)\n at async gracefullyHandleError (/hyperlane-monorepo/typescript/infra/scripts/funding/fund-keys-from-deployer.ts:774:5)\n at async /hyperlane-monorepo/typescript/infra/scripts/funding/fund-keys-from-deployer.ts:394:29\n at async Promise.all (index 9)","level":"error","message":"Error bridging to L2"} ``` The fix was just to upgrade to a newer optimism SDK version A mystery to me is that we'd get that log ^ but sometimes the key funder pod would show as having ran successfully. My best guess here is there was a race condition in `fund` when we had a variable local to the function called `failureOccurred` that many promises which were `Promise.all`'d would read and write to it via `failureOccurred \|\|= await someFallibleFn()`. I changed the logic here to not have multiple concurrent promises contend for the variable Also made a small change to not include mantapacific in the list of relayer keys for the Hyperlane context. It's not ever used, and had a downstream effect of us trying to fund the Kathy key on mantapacific, which we don't want to actually do ### Drive-by changes n/a ### Related issues n/a ### Backward compatibility ye ### Testing ran locally	12 months ago
Trevor Porter	7297c8d6f2	Deploy kathy w/ PolygonZkEvm fix (#3011 ) ### Description Deploy Kathy with #3001 ### Drive-by changes Small fix to deploy kathy successfully - because the relayer chains doesn't include neutron and kathy will try to create a multiprovider for all chains the env, we need to use the environment chain names as the helm `hyperlane.chains` value ### Related issues <!-- - Fixes #[issue number here] --> ### Backward compatibility <!-- Are these changes backward compatible? Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling? Yes/No --> ### Testing <!-- What kind of testing have these changes undergone? None/Manual/Unit Tests -->	12 months ago
Trevor Porter	e21e020b89	Fix Kathy sending from PolygonZkEvm & require MultiProtocolProvider chains to match addresses in MultiProtocolApp (#3001 ) ### Description Fixes 2 issues: 1. Estimates gas in Kathy by explicitly specifying the `from` address due to this bug with PolygonZkEvm when using a non-zero value https://github.com/0xPolygonHermez/zkevm-node/issues/2869 2. Now that we've added neutron & mantapacific as mainnet chains but we don't have helloworld deployments on these chains, Kathy was trying to send to & from these chains. To fix, I changed the constructor of `MultiProtocolApp` to intersect the multiProvider to only work with the chains specified in `addresses` ### Drive-by changes n/a ### Related issues n/a ### Backward compatibility ye ### Testing Ran kathy locally successfully & sent from polygonzkevm	12 months ago

1 2 3 4 5 ...

1961 Commits (99fe93a5ba0adb796b225ef035c1c1ec53990cc7) All Branches Search

1961 Commits (99fe93a5ba0adb796b225ef035c1c1ec53990cc7)

All Branches