Skip to content

[🐞 Bug]: RegisterAccountsUntil should check the stake key registration until the beginning of the current epoch. #67

Description

@satran004

Is there an existing issue for this?

  • I have searched the existing issues

Current (incorrect 🙈) Behavior

Currently, it checks the stake key registration until the stability window of the current epoch. I think it should check till the beginning of the current epoch.

We had similar code in Yaci Store, which caused a mismatch in the adapot calculation for epoch 884 on the Preview network. This issue has already been fixed in Yaci Store.

In epoch 884, 15 pools retired, and the reward accounts for 10 retired pools were registered and deregistered in epoch 884.

HashSet<String> registeredAccountsUntilNow = dbSyncDataProvider.getRegisteredAccountsUntilNow(epoch, poolRewardAddresses, stabilityWindow);

Expected Behavior (what you should have seen 🎉)

10 pool refunds should go to treasury.

Steps To Reproduce

Run adapot calculation of Preview epoch 884.

Environment

- OS:
- Java JDK:

Anything else?

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions