Skip to content

Conversation

lmnzx
Copy link
Member

@lmnzx lmnzx commented Sep 2, 2025

Issue Addressed

Run Testnet in Anchor's CI

Proposed Changes

Added a new CI Workflow to Run Assertoor

Additional Info

New github workflow is to run a local testnet with kurtosis and run a few test with assertoor

The tests are to check:

  • Check chain stability
  • Every client pair proposed a block
  • Transaction Test
  • Blob Transaction Test

Copy link

cla-assistant bot commented Sep 2, 2025

CLA assistant check
All committers have signed the CLA.

@dknopik dknopik changed the base branch from stable to release-v0.3.0 September 2, 2025 10:37
@dknopik dknopik changed the title Local Net CI Workflow ci: Local Net CI Workflow Sep 2, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new CI workflow to run a local testnet with Assertoor testing framework in Anchor's continuous integration pipeline. The workflow builds a Docker image and runs local testnet tests to ensure network functionality.

Key changes:

  • New GitHub Actions workflow for local testnet testing
  • Docker image building and artifact management for the testnet
  • Integration with Assertoor testing framework and Kurtosis for network orchestration

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment on lines 37 to 38
repository: dknopik/ssv-mini
ref: fixes
Copy link
Preview

Copilot AI Sep 2, 2025

Choose a reason for hiding this comment

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

Using a personal repository ('dknopik/ssv-mini') and a non-specific branch reference ('fixes') in production CI creates dependency risks. Consider using a stable, official repository or fork it under the organization's namespace with a specific commit hash or tagged version.

Suggested change
repository: dknopik/ssv-mini
ref: fixes
repository: your-org/ssv-mini
ref: v1.2.3

Copilot uses AI. Check for mistakes.

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

yeah, at least we should use a specific commit hash here

Copy link
Contributor

Choose a reason for hiding this comment

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

Why are we using a personal branch?

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

updated to use the specific commit hash

Copy link
Member

Choose a reason for hiding this comment

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

Why are we using a personal branch?

currently, we had to make slight adjustments in order to get the config working. this is currently done in a slightly hacky way to get this MVP for our CI.

We will continue to improve on this initial state and eventually upstream all changes to ssv-mini

Copy link
Contributor

Choose a reason for hiding this comment

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

What about we move it to sigp?

Copy link
Contributor

Choose a reason for hiding this comment

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

@diegomrsantos
Copy link
Contributor

Issue Addressed

Run Testnet in Anchor's CI

Proposed Changes

Added a new CI Workflow to Run Assertoor

Could you please briefly explain what is being tested in this PR for people not familiar with it (like me)?

@realbigsean realbigsean closed this Sep 2, 2025
@realbigsean realbigsean reopened this Sep 2, 2025
@realbigsean realbigsean closed this Sep 2, 2025
@realbigsean realbigsean reopened this Sep 2, 2025
@dknopik
Copy link
Member

dknopik commented Sep 4, 2025

IMO this is good to go, but before merging I want to test intentionally breaking anchor it to see if the CI catches it 😈

will do soon

@dknopik
Copy link
Member

dknopik commented Sep 10, 2025

@lmnzx

I broke block production, but the CI seems to pass anyway. Can you please take a look?

@diegomrsantos
Copy link
Contributor

The tests are to check:

  • Check chain stability
  • Every client pair proposed a block
  • Transaction Test
  • Blob Transaction Test

Where are these tests defined?

@dknopik
Copy link
Member

dknopik commented Sep 10, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants