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

[SPEC] Remove state of linked anchors from relayer configs #197

Closed
5 tasks
drewstone opened this issue Jun 8, 2022 · 1 comment · Fixed by #355
Closed
5 tasks

[SPEC] Remove state of linked anchors from relayer configs #197

drewstone opened this issue Jun 8, 2022 · 1 comment · Fixed by #355
Assignees
Labels
p3 🔵 Issues should be resolved eventually spec 🆕 Specification details for future implementation

Comments

@drewstone
Copy link
Contributor

drewstone commented Jun 8, 2022

(this shouldn't be taken on right now)

Overview

Relayers don't need to store the linked anchor config, instead we can get this information from the edge list of a deployed anchor. The main challenge is to handle dynamic additions to the bridge. The relayer would need to create a new event watcher on the fly if it hasn't set up a watcher for a new addition to an anchor's edge list.

The new anchor addition flow would also required hooking up the AnchorUpdateProposal to the force_submit_unsigned_proposal so that we can get it signed for inclusion into all existing anchors. Finally, we must finish the loop by updating the new anchor with the state of all existing anchors as well.

Checklist

  • Spec a dynamic addition of a new anchor onto an existing bridge
  • Remove configuration and use on-chain
  • Define flow for adding new anchors to an existing bridge
  • Spawn new event watchers when new anchors are added to an existing bridge (and in general)
  • more to spec...
@dutterbutter dutterbutter added needs more info ℹ️ Add additional information spec 🆕 Specification details for future implementation p3 🔵 Issues should be resolved eventually and removed needs more info ℹ️ Add additional information labels Jun 8, 2022
@dutterbutter dutterbutter moved this to Not Started 🕧 in Webb Universe Jun 8, 2022
@dutterbutter dutterbutter moved this from Not Started 🕧 to Planning 🗺️ in Webb Universe Jun 16, 2022
@dutterbutter dutterbutter moved this from Planning 🗺️ to Not Started 🕧 in Webb Universe Oct 24, 2022
@shekohex shekohex moved this from Not Started 🕧 to Building 🏗️ in Webb Universe Mar 10, 2023
@shekohex
Copy link
Collaborator

Closed by #355

@github-project-automation github-project-automation bot moved this from Building 🏗️ to Completed ✅ in Webb Universe Mar 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p3 🔵 Issues should be resolved eventually spec 🆕 Specification details for future implementation
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants