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

besu internal error when filling constantinople/eip1014_create2/test_recreate.py #882

Open
winsvega opened this issue Oct 8, 2024 · 2 comments
Labels
t8ntools Third party t8n tools support

Comments

@winsvega
Copy link
Collaborator

winsvega commented Oct 8, 2024

Command breaks besu t8n server:

uv run fill --evm-bin=besu-t8n tests/constantinople/eip1014_create2/test_recreate.py
tests/constantinople/eip1014_create2/test_recreate.py:113: in test_recreate
    blockchain_test(genesis_environment=env, pre=pre, post=post, blocks=blocks)
src/pytest_plugins/filler/filler.py:805: in __init__
    fixture = self.generate(
src/ethereum_test_specs/blockchain.py:750: in generate
    return self.make_fixture(t8n, fork, eips)
src/ethereum_test_specs/blockchain.py:568: in make_fixture
    header, txs, requests, new_alloc, new_env = self.generate_block_data(
src/ethereum_test_specs/blockchain.py:432: in generate_block_data
    transition_tool_output = t8n.evaluate(
src/evm_transition_tool/besu.py:166: in evaluate
    response.raise_for_status()  # exception visible in pytest failure output
.venv/lib/python3.12/site-packages/requests/models.py:1024: in raise_for_status
    raise HTTPError(http_error_msg, response=self)
E   requests.exceptions.HTTPError: 500 Server Error: Internal Server Error for url: http://localhost:40685/

@winsvega winsvega added the t8ntools Third party t8n tools support label Oct 8, 2024
@danceratopz
Copy link
Member

danceratopz commented Oct 8, 2024

@winsvega The timeout of 5 seconds is already quite generous (even for a gas limit type test). I don't see this in the stacktrace, but I assume we're hitting the timeout set here:

response = requests.post(self.server_url, json=post_data, timeout=5)

You could try experimenting locally to see if it helps?

@winsvega
Copy link
Collaborator Author

winsvega commented Oct 8, 2024

I see the issue is in this test:

tests/constantinople/eip1014_create2/test_recreate.py:113: in test_recreate
    blockchain_test(genesis_environment=env, pre=pre, post=post, blocks=blocks)
src/pytest_plugins/filler/filler.py:805: in __init__
    fixture = self.generate(
src/ethereum_test_specs/blockchain.py:750: in generate
    return self.make_fixture(t8n, fork, eips)
src/ethereum_test_specs/blockchain.py:568: in make_fixture
    header, txs, requests, new_alloc, new_env = self.generate_block_data(
src/ethereum_test_specs/blockchain.py:432: in generate_block_data
    transition_tool_output = t8n.evaluate(
src/evm_transition_tool/besu.py:166: in evaluate
    response.raise_for_status()  # exception visible in pytest failure output
.venv/lib/python3.12/site-packages/requests/models.py:1024: in raise_for_status
    raise HTTPError(http_error_msg, response=self)
E   requests.exceptions.HTTPError: 500 Server Error: Internal Server Error for url: http://localhost:40685/

uv run fill --evm-bin=besu-t8n tests/constantinople/eip1014_create2/test_recreate.py

it breaks the server so all following tests timeout

@winsvega winsvega changed the title besu timeout when filling the tests besu internal error when filling constantinople/eip1014_create2/test_recreate.py Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t8ntools Third party t8n tools support
Projects
None yet
Development

No branches or pull requests

2 participants