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

Token pools should be able to reactivate themselves #659

Closed
awrichar opened this issue Apr 1, 2022 · 2 comments
Closed

Token pools should be able to reactivate themselves #659

awrichar opened this issue Apr 1, 2022 · 2 comments
Assignees

Comments

@awrichar
Copy link
Contributor

awrichar commented Apr 1, 2022

The token connectors are currently operating under a false assumption that they can delete pool-specific event subscriptions, re-send a "token-pool" event to FireFly, and expect it to reactivate the pools (therefore recreating the subscriptions for that pool).

This does not happen. FireFly only triggers activation in response to a pool creation event if the pool was previously in "unknown" state:

if err = em.assets.ActivateTokenPool(ctx, existingPool, pool.Event.Info); err != nil {

Need to revisit this dialogue and determine the best way for connectors to recreate event subscriptions in this scenario.

@awrichar awrichar self-assigned this Apr 1, 2022
@awrichar
Copy link
Contributor Author

awrichar commented Apr 1, 2022

Note that pools are (re)activated when processing a definition message for a token pool. Replaying the chain of batch pinned messages would result in pools being properly activated.

However, due to a combination of this bug and hyperledger/firefly-tokens-erc1155#62, restarting the token connector can continuously get into this state with subscriptions missing.

@awrichar
Copy link
Contributor Author

awrichar commented Apr 1, 2022

Closing as invalid. For now, auto-migration of token subscriptions will be removed, and manual migration will be supported only by users directly (re)activating the pools with the token connector.

@awrichar awrichar closed this as completed Apr 1, 2022
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

No branches or pull requests

1 participant