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.

Sign up for GitHub

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

Jump to bottom

Feat/assertor testing #12667

Open
wants to merge 56 commits into
base: release/2.61
Choose a base branch
from
Open

Feat/assertor testing #12667

wants to merge 56 commits into from
+322 −0

Conversation

tosettil-polimi
Copy link
Collaborator

@tosettil-polimi tosettil-polimi commented Nov 7, 2024 •
edited
Loading

Add Kurtosis Assertoor Testing Infrastructure

Overview

This PR introduces automated testing infrastructure using Kurtosis Assertoor for validating Erigon's functionality and compatibility. The implementation includes configuration for both standard and Pectra-specific test scenarios.

Changes

  • Added GitHub Actions workflow for Kurtosis Assertoor testing
  • Created two configuration files (first one imported from main branch):
    • Standard testing configuration (config.properties)
    • Pectra-specific testing configuration (config-pectra.properties)
  • Implemented Docker build and test pipeline

Implementation Details

  • Test workflow triggers on:
    • Pushes to release/2.61 branch
    • Manual workflow dispatch
  • Configures test matrix with multiple clients:
    • Execution Layer: Geth, Erigon, Nethermind
    • Consensus Layer: Lighthouse, Nimbus
  • Includes artifact retention for test logs and outputs

@tosettil-polimi tosettil-polimi changed the base branch from main to release/2.61 November 7, 2024 15:56
@tosettil-polimi
Copy link
Collaborator Author

tosettil-polimi commented Nov 7, 2024 •
edited
Loading

Here the new version after closing #12662.

@tosettil-polimi
feat(ci-cd): build like version 2
24ad646
@tosettil-polimi
fix(ci-cd): name
cab39f3
@tosettil-polimi
fix(ci-cd): run kurtosis assertor path
30fe026
@tosettil-polimi
fix(ci-cd): amd64 + target base image
c6f6ad0
@tosettil-polimi
feat(ci-cd): build erigon only
be53d80
@tosettil-polimi
feat(ci-cd): import kurtosis
82f86b8
@tosettil-polimi
fix(ci-cd): build
240f961
@tosettil-polimi
fix(ci-cd): build version
e4438a9
@tosettil-polimi
fix(ci-cd): build version latest + without
83b785f
@tosettil-polimi
fix(ci-cd): loading to local image store
5ebb30b
@tosettil-polimi
lint(ci-cd): buildx params
ac407ca
@tosettil-polimi
fix(ci-cd): build only amd
e8ae7b1
@tosettil-polimi
Merge branch 'release/2.61' into feat/assertor-testing
270baa3
@tosettil-polimi
fix(ci-cd): platform
16bb5c5
@tosettil-polimi
fix(ci-cd): build
ce2a653
@tosettil-polimi
feat(test-kurtosis-assertor): removed
39702e0
@tosettil-polimi
lint(config.properties): pectra test
8b6a702
@tosettil-polimi
lint(ci-cd): Build job name
6e52886
@tosettil-polimi
fix(ci-cd): remove timeout
7b078ce
@tosettil-polimi
lint(config.properties): pectra test first
a04483f
@tosettil-polimi
Revert "feat(test-kurtosis-assertor): removed"
5d2fae8 
This reverts commit 39702e0.
@mriccobene mriccobene self-requested a review November 14, 2024 16:07
@tosettil-polimi tosettil-polimi marked this pull request as ready for review November 19, 2024 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Perlets9 Perlets9 Awaiting requested review from Perlets9

@mriccobene mriccobene Awaiting requested review from mriccobene

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@tosettil-polimi