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.

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

Update interchaintest to support latest Hermes config #1063

Closed
rootulp opened this issue Apr 9, 2024 · 0 comments · Fixed by #1067
Closed

Update interchaintest to support latest Hermes config #1063

rootulp opened this issue Apr 9, 2024 · 0 comments · Fixed by #1067

Comments

@rootulp
Copy link
Contributor

rootulp commented Apr 9, 2024

Context

I'm trying to use interchaintest with Hermes relayer because the default ibc-go relayer is hitting #1055 and I don't know how to override the --gas flag for the command that is creating clients. I'm setting up the Hermes relayer like so:

	relayer := interchaintest.NewBuiltinRelayerFactory(
		ibc.Hermes,
		zaptest.NewLogger(t),
		relayer.CustomDockerImage("informalsystems/hermes", "v1.8.2", "1000:1000"),
	).Build(t, client, network)

Problem

It looks like the Hermes config in this repo doesn't match the latest Hermes config options.

        	Error Trace:	/Users/rootulp/git/rootulp/celestiaorg/celestia-app/test/interchain/ica_test.go:61
        	Error:      	Received unexpected error:
        	            	failed to configure relayer relayerName for chain celestia: exit code 1:  The Hermes configuration file at path '/home/hermes/.hermes/config.toml' is invalid, reason: parse error: invalid CosmosSdk config: unknown field `websocket_addr`, expected one of `id`, `rpc_addr`, `grpc_addr`, `event_source`, `rpc_timeout`, `trusted_node`, `account_prefix`, `key_name`, `key_store_type`, `key_store_folder`, `store_prefix`, `default_gas`, `max_gas`, `genesis_restart`, `gas_adjustment`, `gas_multiplier`, `fee_granter`, `max_msg_num`, `max_tx_size`, `max_grpc_decoding_size`, `query_packets_chunk_size`, `clock_drift`, `max_block_time`, `trusting_period`, `client_refresh_rate`, `ccv_consumer_chain`, `memo_prefix`, `memo_overwrite`, `sequential_batch_tx`, `proof_specs`, `trust_threshold`, `gas_price`, `packet_filter`, `dynamic_gas_price`, `address_type`, `extension_options`, `compat_mode`, `clear_interval`, `excluded_sequences` for key `chains` at line 59 column 1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants