search

EmerisHQ / tracelistener

UNIX named pipes-based real-time state listener for Cosmos SDK blockchains
GNU Affero General Public License v3.0
33 stars 8 forks source link

Tracelistener sometimes misses unbonded delegations #65

Closed akc2267 closed 2 years ago

akc2267 commented 2 years ago

a user reported inaccurate ATOM values, as Emeris shows staked ATOM that had already been unbonded and moved out of the account: reported account: https://api.emeris.com/v1/account/56d58bd08f77b6dde9c3afb3788bf5de6b0bc686/unbondingdelegations slack thread: https://allinbits.slack.com/archives/C02AXJ0K9F1/p1646841505875939?thread_ts=1646841505.875939&cid=C02AXJ0K9F1

this is likely due to TL missing a "delete" trace once the unbonding was complete

akc2267 commented 2 years ago

this might be related to this issue: https://app.zenhub.com/workspaces/emeris-back-end-619cd009cc778000112d6259/issues/emerishq/demeris-backend/366

akc2267 commented 2 years ago

relevant investigation: https://www.notion.so/allinbits/Tracelistener-overview-discussion-ce2dd3cdd2514a018401e39ab4adafce#1f55725bb23742b59bb1d68fe3cf0488

gsora commented 2 years ago

This has been fixed in code, not yet deployed due to GitHub actions being unreliable.

akc2267 commented 2 years ago

next steps:

  1. relaunch tracelistener across staging chains
  2. test staking/redelegating/undelegating on some staging chains
  3. if successful, schedule upgrade in prod chains
  4. test and confirm in Prod
akc2267 commented 2 years ago

also need to confirm if we need to do a full bulk re-import. Will need to find addresses that have staking accuracy issues in the past and see if these issues still persist

Pitasi commented 2 years ago

@akc2267 yes, we will need to bulk import all the chains after upgrading tracelistener.