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

extend leap-util logs for smoke test #1704

Merged
merged 1 commit into from
Oct 3, 2023
Merged

extend leap-util logs for smoke test #1704

merged 1 commit into from
Oct 3, 2023

Conversation

dimas1185
Copy link
Contributor

@dimas1185 dimas1185 commented Oct 2, 2023

resolves #1348

here is my test logs:

leap-util block-log --blocks-dir /home/dima/work/leap/build/unittests/snapshots/prod_sched  smoke-test

Smoke test of blocks.log and blocks.index in directory "/home/dima/work/leap/build/unittests/snapshots/prod_sched"
info  2023-10-02T23:37:54.770 leap-util block_log.cpp:1583            smoke_test           ] block log version= 2
info  2023-10-02T23:37:54.770 leap-util block_log.cpp:1584            smoke_test           ] first block= 1
info  2023-10-02T23:37:54.770 leap-util block_log.cpp:1585            smoke_test           ] last block= 5
error 2023-10-02T23:37:54.771 leap-util subcommand.cpp:10             print_exception      ] 3190000 block_log_exception: Block log exception
/home/dima/work/leap/build/unittests/snapshots/prod_sched/blocks.log says it has 5 blocks which disagrees with 4 indicated by /home/dima/work/leap/build/unittests/snapshots/prod_sched/blocks.index
    {"block_file_name":"/home/dima/work/leap/build/unittests/snapshots/prod_sched/blocks.log","log_num_blocks":5,"index_num_blocks":4,"index_file_name":"/home/dima/work/leap/build/unittests/snapshots/prod_sched/blocks.index"}
    leap-util  block_log.cpp:350 validate_index

@dimas1185 dimas1185 changed the title extended leap-util logs for smoke test extend leap-util logs for smoke test Oct 2, 2023
@dimas1185 dimas1185 requested review from heifner and linh2931 October 3, 2023 01:29
@dimas1185 dimas1185 requested a review from greg7mdp October 3, 2023 19:17
@dimas1185 dimas1185 merged commit 19f78f9 into main Oct 3, 2023
22 checks passed
@dimas1185 dimas1185 deleted the block_log_fix branch October 3, 2023 19:40
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.

leap-util is unable to successfully run a detailed smoke test and trim the blocklog.
3 participants