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

Stuck packet #1296

Merged
merged 10 commits into from
Oct 11, 2023
Merged

Stuck packet #1296

merged 10 commits into from
Oct 11, 2023

Conversation

agouin
Copy link
Member

@agouin agouin commented Sep 13, 2023

Adds flags --stuck-packet-chain-id, --stuck-packet-height-start, and --stuck-packet-height-end for starting up the CosmosChainProcessor by querying a specific range for a chain ID before jumping to current and relaying anything that needs to be handled by the queried range.

Note: This is for advanced usage only and should only be used for mitigation scenarios, as the relayer will not have a full picture of what happened between --stuck-packet-height-end and the current chain tip, which could lead to failures if packets were already handled since the stuck packet height range.

break out of query cycle if stuck packet height has been queried

Add debug logs

get hack working
@agouin agouin marked this pull request as ready for review September 13, 2023 19:29
Copy link
Member

@jtieri jtieri left a comment

Choose a reason for hiding this comment

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

this is an awesome feature! had a few comments for ya but i'll approve now so i'm not a blocker on merging

cmd/flags.go Show resolved Hide resolved
relayer/chains/cosmos/cosmos_chain_processor.go Outdated Show resolved Hide resolved
@agouin agouin enabled auto-merge (squash) October 11, 2023 21:49
@agouin agouin merged commit 6bd9c02 into main Oct 11, 2023
18 checks passed
@agouin agouin deleted the andrew/stride_hack_refresh_2 branch October 11, 2023 22:15
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 this pull request may close these issues.

None yet

2 participants