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

Slow anvil chain start + contract deploys in CI testing with recent release #7631

Open
1 of 2 tasks
Tracked by #8269
clemire opened this issue Apr 10, 2024 · 7 comments
Open
1 of 2 tasks
Tracked by #8269
Labels
C-anvil Command: anvil Cmd-forge-script Command: forge script P-low Priority: low T-bug Type: bug T-to-reproduce Type: requires reproduction

Comments

@clemire
Copy link

clemire commented Apr 10, 2024

Component

Forge, Anvil

Have you ensured that all of these are up to date?

  • Foundry
  • Foundryup

What version of Foundry are you on?

nightly, pulled from CI

What command(s) is the bug in?

No response

Operating System

Linux

Describe the bug

We run CI tests that stand up two mock chains with anvil and then use forge script to deploy smart contracts to them in our repo on every PR and also regularly on our main branch, pulling a linux image from the nightly foundry version. Yesterday we found that our tests were timing out because our contract deployments, which usually take about 2-2.5 minutes to complete, were taking as long as 9 minutes. Reverting to version nightly-de33b6af53005037b463318d2628b5cfcaf39916 fixes the issue for us.

@clemire clemire added the T-bug Type: bug label Apr 10, 2024
@klkvr
Copy link
Member

klkvr commented Apr 11, 2024

@clemire do you have a repro for this you could share? can you confirm if the issue first appeared during last two days or it was the case earlier?

@klkvr
Copy link
Member

klkvr commented Apr 11, 2024

Might be alloy-rs/alloy#389 if scripts are hanging after broadcasting start

@clemire
Copy link
Author

clemire commented Apr 17, 2024

It was last two days since bug report. I'm going to put nightly back in and see if that fixes it.

@DeRain
Copy link

DeRain commented May 22, 2024

@clemire thanks! I've reverted to the nightly-de33b6af53005037b463318d2628b5cfcaf39916 and performance was increased significantly! Looks like latest builds causing random slowness on specific transactions.

@zerosnacks zerosnacks added T-to-reproduce Type: requires reproduction C-anvil Command: anvil Cmd-forge-script Command: forge script labels Jun 27, 2024
@klkvr
Copy link
Member

klkvr commented Jul 12, 2024

@clemire is this still the case on latest nightly?

@zerosnacks zerosnacks added this to the v1.0.0 milestone Jul 26, 2024
@jenpaff
Copy link
Collaborator

jenpaff commented Sep 26, 2024

@clemire is this still an issue?

@zerosnacks
Copy link
Member

Hi @clemire / @DeRain is this still an issue on the latest nightly?

@zerosnacks zerosnacks added the P-low Priority: low label Oct 22, 2024
@zerosnacks zerosnacks removed this from the v1.0.0 milestone Oct 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-anvil Command: anvil Cmd-forge-script Command: forge script P-low Priority: low T-bug Type: bug T-to-reproduce Type: requires reproduction
Projects
Status: Todo
Development

No branches or pull requests

5 participants