[RelayMiner] Use gas for claim and proof txs

[red-0ne]

Summary

This pull request includes several changes to the pkg/client package, focusing on adding gas limit and gas price parameters to transaction signing and broadcasting functions, as well as making related updates in tests and configurations.

It also updates the BroadcastTx method in pkg/client/tx/context.go to use BroadcastTxSync instead of BroadcastTxAsync for better error handling during the check-tx ABCI operation.

Minor Fix: Changed the URL in makefiles/relay.mk to include a trailing slash.

Issue

Type of change

Select one or more from the following:

• New feature, functionality or library
• Consensus breaking; add the consensus-breaking label if so. See [Infra] Automatically add the consensus-breaking label #791 for details
• Bug fix
• Code health or cleanup
• Documentation
• Other (specify)

Testing

• Documentation: make docusaurus_start; only needed if you make doc changes
• Unit Tests: make go_develop_and_test
• LocalNet E2E Tests: make test_e2e
• DevNet E2E Tests: Add the devnet-test-e2e label to the PR.

Sanity Checklist

• I have tested my changes using the available tooling
• I have commented my code
• I have performed a self-review of my own code; both comments & source code
• I create and reference any new tickets, if applicable
• I have left TODOs throughout the codebase, if applicable

[github-actions]

The image is going to be pushed after the next commit.

You can use make trigger_ci to push an empty commit.

If you also want to run E2E tests, please add devnet-test-e2e label.

[Olshansk]

Thank you for handling this so quickly!

[github-actions]

The CI will now also run the e2e tests on devnet, which increases the time it takes to complete all CI checks.

You may need to run make trigger_ci to submit an empty commit that'll trigger the tests.

GCP workloads (requires changing the namespace to 1018)
Grafana network dashboard for devnet-issue-1018

[okdas]

{"level":"info","message":"Starting relay miner..."}
{"level":"info","message":"starting relayer sessions manager"}
{"level":"info","endpoint":"localhost:6060","message":"starting a pprof endpoint"}
{"level":"info","endpoint":":9090","message":"serving metrics"}
{"level":"info","message":"starting relayer proxy"}
{"level":"info","server host":"0.0.0.0:8545","message":"starting relay proxy server"}
{"level":"info","message":"starting ring cache"}
{"level":"info","session_end_height":34020,"claim_window_open_height":34022,"message":"waiting & blocking until the earliest claim commit height offset seed block height"}
{"level":"info","session_end_height":34020,"claim_window_open_height":34022,"claim_window_open_block_hash":"cd1ef990487790b33d764349f0ef61e30dfd835d4383228209d5e140584fe6f5","message":"observed earliest claim commit height offset seed block height"}
{"level":"info","session_end_height":34020,"claim_window_open_height":34022,"claim_window_open_block_hash":"cd1ef990487790b33d764349f0ef61e30dfd835d4383228209d5e140584fe6f5","earliest_claim_commit_height":34022,"message":"waiting & blocking until the earliest claim commit height for this supplier"}
{"level":"info","session_end_height":34020,"claim_window_open_height":34022,"claim_window_open_block_hash":"cd1ef990487790b33d764349f0ef61e30dfd835d4383228209d5e140584fe6f5","earliest_claim_commit_height":34022,"message":"observed earliest claim commit height"}
{"level":"error","error":"out of gas in location: ReadFlat; gasWanted: 100000, gasUsed: 100905: out of gas: error during ABCI check tx","message":"failed to create claims"}
{"level":"error","error":"out of gas in location: ReadFlat; gasWanted: 100000, gasUsed: 100905: out of gas: error during ABCI check tx"}

Seems like we need to bump the fees if we're not going to make it configurable at the moment.

[bryanchriswhite]

Very cool - thanks @red-0ne! 🚀

[Olshansk]

Looks great! Requesting changes just because I know this is still an active WIP.

[Olshansk]

Awesome work, ty @red-0ne!