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

feat(rln-relay): allow passing in a path to the tree db #1772

Closed
1 task done
rymnc opened this issue Jun 1, 2023 · 0 comments · Fixed by #1805
Closed
1 task done

feat(rln-relay): allow passing in a path to the tree db #1772

rymnc opened this issue Jun 1, 2023 · 0 comments · Fixed by #1805

Comments

@rymnc
Copy link
Contributor

rymnc commented Jun 1, 2023

Problem

With the integration of pmtree into zerokit and the downstream to nwaku, we should now accept a path to the tree db to avoid onchain resyncs.

Suggested solution

  • A new cmd line arg to nwaku --rln-relay-tree-db which accepts an absolute path to the tree db.
  • Validation to ensure that the same db config is used (already handled on the zerokit end)
  • If the db doesn't exist at the given path, we need to persist the db there
  • Set metadata like LAST_BLOCK_PROCESSED into the tree upon each block being processed so that nwaku knows where to resync from.

Alternatives considered

Additional context

Add any other context or screenshots about the feature request here.

Acceptance criteria

  • Operators are able to pass in the path to an existing db
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant