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

[testing] Add scheduled jobs to test bootstrap from mainnet and testnet #3172

Draft
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

marun
Copy link
Contributor

@marun marun commented Jul 4, 2024

Why this should be merged

Periodic bootstrap testing is intended to insure that bootstrapping issues are detected prior to releasing to testnet and mainnet.

How this works

  • adds a new test binary to start a node and wait for it to finish bootstrapping
  • adds a new check-bootstrap custom action to simplify job creation
  • adds new jobs to test bootstrap from mainnet and testnet for full- and state-sync

TODO

@marun marun added testing This primarily focuses on testing ci This focuses on changes to the CI process labels Jul 4, 2024
@marun marun self-assigned this Jul 4, 2024
@marun marun force-pushed the testing-bootstrap branch 10 times, most recently from 757eaa6 to 592daa7 Compare July 4, 2024 05:38
@marun marun force-pushed the testing-bootstrap branch 3 times, most recently from 2b74717 to 260ab20 Compare July 9, 2024 18:04
@marun marun marked this pull request as ready for review July 10, 2024 22:55
})

// Create a new single-node network that will bootstrap from the specified network
network := &tmpnet.Network{
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use of tmpnet to start the nodes is primarily to take advantage of its configuration of metrics and log collection in CI. If reviewers feel strongly it should be possible to extract the monitoring configuration for reuse instead.

Base automatically changed from ci-monitoring-actions to master July 11, 2024 12:04
@marun marun force-pushed the testing-bootstrap branch 2 times, most recently from 3cdd0ae to 89a5670 Compare July 11, 2024 18:48
@marun marun changed the title [testing] Add a job to test state sync bootstrap of testnet [testing] Add scheduled jobs to test bootstrap from mainnet and testnet Jul 11, 2024
@marun marun force-pushed the testing-bootstrap branch 2 times, most recently from 4928b06 to 5a46786 Compare July 16, 2024 02:14
@marun marun force-pushed the testing-bootstrap branch 3 times, most recently from 5e9a28b to b245d28 Compare July 24, 2024 00:57
@marun marun force-pushed the testing-bootstrap branch 3 times, most recently from 585a15c to f79f576 Compare July 25, 2024 03:20
Copy link
Contributor

@abi87 abi87 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm once the conflict and lint issue is solved. Left a couple of optional suggestions

@marun marun marked this pull request as draft August 21, 2024 04:55
@marun
Copy link
Contributor Author

marun commented Aug 21, 2024

Moving to draft, am actively working on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci This focuses on changes to the CI process testing This primarily focuses on testing
Projects
Status: In Progress 🏗
Development

Successfully merging this pull request may close these issues.

None yet

3 participants