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

Upstream v2.61.3 #245

Merged
merged 11 commits into from
Mar 7, 2025
Merged

Upstream v2.61.3 #245

merged 11 commits into from
Mar 7, 2025

Conversation

mininny
Copy link
Member

@mininny mininny commented Mar 6, 2025

shohamc1 and others added 11 commits February 13, 2025 14:18
…tech#13774)

This PR updates the **QA sync workflow** to improve flexibility and
expand compatibility with different consensus clients and networks.

#### Key Changes:
1. **Renamed Job**:  
- Changed `prysm-minimal-node-sync-from-scratch-test` →
`sync-with-externalcl` for broader applicability.

2. **Matrix Strategy for Multiple Clients & Chains**:  
   - Introduced `matrix` configuration to run tests for:
     - Clients: **Lighthouse, Prysm**
     - Chains: **Mainnet, Gnosis**
   - **Exclusion**: Prysm + Gnosis is **not** tested.

3. **Client-Specific Setup**:  
- Lighthouse and Prysm are now dynamically installed based on the matrix
configuration.
   - JWT secret is generated per test run.

4. **Erigon Sync Execution Update**:  
- Uses `${{ matrix.chain }}` and `${{ matrix.client }}` dynamically in
test execution and result logging.

5. **Enhanced Logging & Artifacts**:  
   - Client-specific logs are now uploaded as artifacts.
   - Improved log handling for both Lighthouse and Prysm.

6. **Additional Cleanup Steps**:  
- Removed consensus layer data (`CL_DATA_DIR`) after tests to free up
disk space.
Diable test that are failing due to:
- Pectra changes (eth_feeHistory)
- totalDifficulty removal (eth_getBlockByHash, eth_getBlockByNumber)
@mininny mininny merged commit 5a0d097 into op-erigon Mar 7, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants