Skip to content

fix(l2): flamegraph and benchmark workflows #3652

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

Merged
merged 4 commits into from
Jul 16, 2025
Merged

Conversation

avilagaston9
Copy link
Contributor

@avilagaston9 avilagaston9 commented Jul 15, 2025

Motivation

Install solc to flamegraph-reth job. Successful run here.
Closes None

@avilagaston9 avilagaston9 marked this pull request as ready for review July 15, 2025 22:42
@Copilot Copilot AI review requested due to automatic review settings July 15, 2025 22:42
@avilagaston9 avilagaston9 requested a review from a team as a code owner July 15, 2025 22:42
@avilagaston9 avilagaston9 moved this to In Review in ethrex_l2 Jul 15, 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 fixes the reth flamegraph workflow by adding a missing dependency. The workflow was failing because it required the Solidity compiler (solc) but didn't have it installed.

  • Adds solc installation step to the flamegraph-reth job
  • Uses pontem-network/get-solc action to install solc version v0.8.29

Copy link
Collaborator

@mpaulucci mpaulucci left a comment

Choose a reason for hiding this comment

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

we're discussing if we want to enforce solc as a dependency

@github-project-automation github-project-automation bot moved this from In Review to Requires Changes in ethrex_l2 Jul 16, 2025
@avilagaston9 avilagaston9 changed the title fix(l2): reth flamegraph workflow fix(l2): flamegraph and benchmark workflows Jul 16, 2025
@Arkenan Arkenan added this pull request to the merge queue Jul 16, 2025
Merged via the queue into main with commit 77b2819 Jul 16, 2025
41 of 47 checks passed
@Arkenan Arkenan deleted the fix/flamegraph_reth_workflow branch July 16, 2025 17:42
@github-project-automation github-project-automation bot moved this from Requires Changes to Done in ethrex_l2 Jul 16, 2025
@github-project-automation github-project-automation bot moved this from Todo to Done in ethrex_performance Jul 16, 2025
avilagaston9 added a commit that referenced this pull request Jul 16, 2025
d-roak pushed a commit to 1sixtech/ethrex that referenced this pull request Jul 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L2 Rollup client
Projects
Status: Done
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants