Skip to content

Add e2e sync tests for lookup and range sync to CI #7550

@jimmygchen

Description

@jimmygchen

I made an attempt to add the following tests on to in #7530

  • genesis sync on a local devnet (covers current and next fork)
    • brief shutdown and restart (covers lookup sync)
    • longer shutdown and restart (covers range sync)

I tried using kurtosis-sync-tests and ran into a some issues with assertoor and I ended up removing the tests from the PR so we can move forward.

Given the simplicity of these tests, I think we could just have a simple bash script (similar to the checkpoint sync one in #7530) to do this (without having to debug assertoor):

  1. Start a network
  2. Immediately shutdown a node for some time
  3. Restart the node and poll its sync status until it's fully sync

The matrix I think we should cover:

strategy:
matrix:
fork: [electra, fulu]
offline_secs: [ 120, 300]

The commit for the previous attempt is here
6c7e839

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions