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

Assume sharded DB in replay-verify #16058

Merged
merged 1 commit into from
Mar 5, 2025
Merged

Conversation

msmouse
Copy link
Contributor

@msmouse msmouse commented Mar 5, 2025

Description

Resolves the error where ledger metadb can't be opened under db/ledger_db, because when sharded the db should be at db/ledger_db/metadata

How Has This Been Tested?

manual run

Key Areas to Review

Type of Change

  • Tests -- replay-verify

Which Components or Systems Does This Change Impact?

  • Developer Infrastructure -- replay-verify

Resolves the error where ledger metadb can't be opened under
db/ledger_db, because when sharded the db should be at
db/ledger_db/metadata
Copy link

trunk-io bot commented Mar 5, 2025

⏱️ 2h 31m total CI duration on this PR
Job Cumulative Duration Recent Runs
replay-mainnet / run-replay-verify 2h 10m 🟥
rust-targeted-unit-tests 16m 🟥
rust-cargo-deny 2m 🟩
check-dynamic-deps 1m 🟩
run-replay-verify 29s 🟥
determine-test-metadata 27s 🟩🟩🟩
general-lints 26s 🟩
semgrep/ci 23s 🟩
file_change_determinator 15s 🟩
permission-check 4s 🟩
permission-check 2s 🟩
replay-testnet / run-replay-verify 1s

🚨 1 job on the last run was significantly faster/slower than expected

Job Duration vs 7d avg Delta
check-dynamic-deps 1m 2m -46%

settingsfeedbackdocs ⋅ learn more about trunk.io

@msmouse msmouse requested review from sionescu and grao1991 March 5, 2025 19:46
@msmouse msmouse marked this pull request as ready for review March 5, 2025 19:49
@sherry-x sherry-x merged commit 4e7e43f into main Mar 5, 2025
51 of 57 checks passed
@sherry-x sherry-x deleted the 250305-alden-replay-sharded branch March 5, 2025 21:03
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.

3 participants