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

conductor: executes_soft_first_then_updates_firm test is flaky #2040

Open
ethanoroshiba opened this issue Mar 17, 2025 · 0 comments · May be fixed by #2042
Open

conductor: executes_soft_first_then_updates_firm test is flaky #2040

ethanoroshiba opened this issue Mar 17, 2025 · 0 comments · May be fixed by #2042
Assignees
Labels
conductor pertaining to the astria-conductor crate testing

Comments

@ethanoroshiba
Copy link
Contributor

The conductor::soft_and_firm::executes_soft_first_then_updates_firm test is flaky for me locally, with the following output:

thread 'soft_and_firm::executes_soft_first_then_updates_firm' panicked at crates/astria-conductor/tests/blackbox/soft_and_firm.rs:109:6:
Conductor should have executed the block and updated the soft commitment state within 500ms: Elapsed(())
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
thread 'tokio-runtime-worker' panicked at crates/astria-grpc-mock/src/mock_server.rs:194:17:
Verifications failed:
- Mock #2 for RPC get_filtered_sequencer_block.
        Expected range of matching incoming requests: 1 <= x
        Number of matched requests with valid mock responses: 0
        Number of matched requests with bad mock responses: 0

- Mock #3 for RPC execute_block.
        Expected range of matching incoming requests: == 1
        Number of matched requests with valid mock responses: 0
        Number of matched requests with bad mock responses: 0

- Mock #4 for RPC update_commitment_state.
        Expected range of matching incoming requests: == 1
        Number of matched requests with valid mock responses: 0
        Number of matched requests with bad mock responses: 0


Received requests:
- Request #0
        RPC name: get_commitment_state
        Requests Protobuf as JSON
        {}
- Request #1
        RPC name: get_genesis_info
        Requests Protobuf as JSON
        {}



failures:
    soft_and_firm::executes_soft_first_then_updates_firm
@ethanoroshiba ethanoroshiba added conductor pertaining to the astria-conductor crate testing labels Mar 17, 2025
@ethanoroshiba ethanoroshiba self-assigned this Mar 17, 2025
@ethanoroshiba ethanoroshiba linked a pull request Mar 17, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conductor pertaining to the astria-conductor crate testing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant