sbtc-developer-release icon indicating copy to clipboard operation
sbtc-developer-release copied to clipboard

[sBTC DR] StacksTransactionUpdate event processed too often

Open friedger opened this issue 1 year ago • 2 comments

Expected Behavior

Each event should be handled only once

Actual Behavior

2023-10-27T06:24:15.913047Z  INFO update: romeo::state: Processing event=StacksTransactionUpdate(0c6d2636b8e5b92a640b6b8fbf59efa7e6e580ea4eb47bd21c09f291059aeb4b, Broadcasted)
2023-10-27T06:24:15.913121Z DEBUG update: romeo::state: Ignoring Broadcasted status update for a Stacks transaction that is not acknowledged: 0c6d2636b8e5b92a640b6b8fbf59efa7e6e580ea4eb47bd21c09f291059aeb4b event=StacksTransactionUpdate(0c6d2636b8e5b92a640b6b8fbf59efa7e6e580ea4eb47bd21c09f291059aeb4b, Broadcasted)
2023-10-27T06:24:15.913148Z DEBUG update: romeo::state: Ignoring Broadcasted status update for a Stacks transaction that is not acknowledged: 0c6d2636b8e5b92a640b6b8fbf59efa7e6e580ea4eb47bd21c09f291059aeb4b event=StacksTransactionUpdate(0c6d2636b8e5b92a640b6b8fbf59efa7e6e580ea4eb47bd21c09f291059aeb4b, Broadcasted)
2023-10-27T06:24:15.913166Z DEBUG update: romeo::state: Ignoring Broadcasted status update for a Stacks transaction that is not acknowledged: 0c6d2636b8e5b92a640b6b8fbf59efa7e6e580ea4eb47bd21c09f291059aeb4b event=StacksTransactionUpdate(0c6d2636b8e5b92a640b6b8fbf59efa7e6e580ea4eb47bd21c09f291059aeb4b, Broadcasted)
2023-10-27T06:24:15.913184Z DEBUG update: romeo::state: Ignoring Broadcasted status update for a Stacks transaction that is not acknowledged: 0c6d2636b8e5b92a640b6b8fbf59efa7e6e580ea4eb47bd21c09f291059aeb4b event=StacksTransactionUpdate(0c6d2636b8e5b92a640b6b8fbf59efa7e6e580ea4eb47bd21c09f291059aeb4b, Broadcasted)
2023-10-27T06:24:15.913203Z DEBUG update: romeo::state: Ignoring Broadcasted status update for a Stacks transaction that is not acknowledged: 0c6d2636b8e5b92a640b6b8fbf59efa7e6e580ea4eb47bd21c09f291059aeb4b event=StacksTransactionUpdate(0c6d2636b8e5b92a640b6b8fbf59efa7e6e580ea4eb47bd21c09f291059aeb4b, Broadcasted)

Steps to Reproduce

Let romeo run for a few hours and do some deposits and withdrawals

friedger avatar Oct 27 '23 06:10 friedger

with the panic removed (#333 ) and after restarting and indexing from the previous contract i see a lot of these lines in the log;

2023-10-28T15:53:03.882353Z DEBUG update: romeo::state: Ignoring Broadcasted status update for a Stacks transaction that is not acknowledged: 0000000000000000000000000000000000000000000000000000000000000000 event=StacksTransactionUpdate(0000000000000000000000000000000000000000000000000000000000000000, Broadcasted)

mijoco-btc avatar Oct 28 '23 16:10 mijoco-btc

Hm, this is definitely a regression as I've never seen this behavior before. I'll look into it.

stjepangolemac avatar Oct 30 '23 07:10 stjepangolemac