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

Test that zcashd and Zebra can sync from Zebra #1400

Closed
Tracked by #2867 ...
teor2345 opened this issue Nov 26, 2020 · 4 comments
Closed
Tracked by #2867 ...

Test that zcashd and Zebra can sync from Zebra #1400

teor2345 opened this issue Nov 26, 2020 · 4 comments
Labels
C-enhancement Category: This is an improvement

Comments

@teor2345
Copy link
Contributor

Once we've merged GetBlocks and GetHeaders support #1347 , Zebra should be able to serve blocks to other nodes.

We should test that zcashd and Zebra can sync from genesis, using a fully synced Zebra.

This is a network integration test, so its out of scope for the first alpha. But it would be nice to make sure there isn't any really bad behaviour.

@teor2345 teor2345 added C-enhancement Category: This is an improvement S-needs-triage Status: A bug report needs triage labels Nov 26, 2020
@mpguerra mpguerra added this to the Sync and Network milestone Dec 17, 2020
@mpguerra mpguerra removed this from the Sync and Network milestone Jan 5, 2021
@mpguerra mpguerra removed the S-needs-triage Status: A bug report needs triage label Feb 2, 2021
@mpguerra
Copy link
Contributor

@ZcashFoundation/zebra-team has anyone tried this already?

@oxarbitrage
Copy link
Contributor

I never tried this but i think is a good idea to sync zcash and zebra only with zebra seeders. We should have this done for the first beta so we should prioritize.

@teor2345
Copy link
Contributor Author

I run tests like this all the time, and I see Zebra syncing from Zebra.

But I don't do it in an isolated environment where there are no extra zcashd instances.

@mpguerra mpguerra added this to the 2021 Sprint 21 milestone Oct 15, 2021
@mpguerra mpguerra added the P-Low label Oct 15, 2021
@mpguerra mpguerra removed this from the 2021 Sprint 21 milestone Oct 25, 2021
@teor2345
Copy link
Contributor Author

This would be a useful test, but it's partly covered by our other tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: This is an improvement
Projects
None yet
Development

No branches or pull requests

3 participants