Skip to content

issue: 60 solo repl dev recovery#65

Merged
yamingk merged 1 commit intoeBay:mainfrom
yamingk:yk_vol_fix
Mar 19, 2025
Merged

issue: 60 solo repl dev recovery#65
yamingk merged 1 commit intoeBay:mainfrom
yamingk:yk_vol_fix

Conversation

@yamingk
Copy link
Collaborator

@yamingk yamingk commented Mar 18, 2025

  1. Volume index recovery fix
  2. Turn on volume recovery test case.
  3. Enable volume superblock handler in on_init_complete()

This PR has to wait for HomeStore PR that enables solo repl dev to send on_repl_devs_init_completed() back to HomeBlocks layer after solo repl dev service is started. The HomeStore PR will have to be merged before this PR.

Testing:

Tested with private homestore fix and passed.

[03/18/25 16:44:42-07:00] [I] [test_volume] [287573] [iomgr_timer.cpp:179:setup_timer_fd] Creating non-recurring per-thread timer fd 36 and adding it into fd poll list
[03/18/25 16:44:42-07:00] [I] [test_volume] [287520] [homeblks_impl.hpp:132:on_index_table_found] Recovered index table to index service
[03/18/25 16:44:42-07:00] [I] [test_volume] [287520] [volume_mgr.cpp:52:recover_index_table] Recovering index table for index_uuid: 60351b63-9eea-40e5-b699-77478ee64616, parent_uuid: 2b3eba4c-b855-4fb2-9f98-4d458c020621
[03/18/25 16:44:42-07:00] [I] [test_volume] [287520] [homeblks_impl.hpp:132:on_index_table_found] Recovered index table to index service
[03/18/25 16:44:42-07:00] [I] [test_volume] [287520] [volume_mgr.cpp:52:recover_index_table] Recovering index table for index_uuid: 6bee109c-0a97-4c28-ae5e-33aa55fa2f6a, parent_uuid: ee20ebfe-18f7-486a-a34a-8581efa1ad6e
[03/18/25 16:44:42-07:00] [I] [test_volume] [287520] [generic_repl_svc.cpp:89:start] Repl devs load completed, calling upper layer on_repl_devs_init_completed
[03/18/25 16:44:42-07:00] [I] [test_volume] [287520] [meta_blk_service.cpp:383:register_handler] [type=Volume2] being registered after scanned from disk.
[03/18/25 16:44:42-07:00] [I] [test_volume] [287520] [volume.cpp:48:Volume] Volume superblock loaded from disk, vol_info : VolumeInfo: id=2b3eba4c-b855-4fb2-9f98-4d458c020621 size_bytes=1073741824, page_size=4096, name=vol_1
[03/18/25 16:44:42-07:00] [I] [test_volume] [287520] [volume.cpp:73:init] Getting repl dev for volume: vol_1, uuid: 2b3eba4c-b855-4fb2-9f98-4d458c020621
[03/18/25 16:44:42-07:00] [I] [test_volume] [287520] [volume.cpp:48:Volume] Volume superblock loaded from disk, vol_info : VolumeInfo: id=ee20ebfe-18f7-486a-a34a-8581efa1ad6e size_bytes=1073741824, page_size=4096, name=vol_0
[03/18/25 16:44:42-07:00] [I] [test_volume] [287520] [volume.cpp:73:init] Getting repl dev for volume: vol_0, uuid: ee20ebfe-18f7-486a-a34a-8581efa1ad6e
[03/18/25 16:44:42-07:00] [I] [test_volume] [287574] [reactor.hpp:50:IOThreadMetrics] Registring metrics group name = IOThreadMetrics, thread_name = log_flush_thread
[03/18/25 16:44:42-07:00] [I] [test_volume] [287574] [iomgr_timer.cpp:179:setup_timer_fd] Creating non-recurring per-thread timer fd 41 and adding it into fd poll list
[03/18/25 16:44:42-07:00] [I] [test_volume] [287520] [cp_mgr.cpp:58:start_timer] cp timer is set to 60000000 usec
[03/18/25 16:44:42-07:00] [I] [test_volume] [287520] [iomgr_timer.cpp:179:setup_timer_fd] Creating recurring global timer fd 45 and adding it into fd poll list
[03/18/25 16:44:42-07:00] [I] [test_volume] [287520] [resource_mgr.cpp:70:start_timer] resource audit timer is set to 120000 usec
[03/18/25 16:44:42-07:00] [I] [test_volume] [287520] [iomgr_timer.cpp:179:setup_timer_fd] Creating recurring global timer fd 46 and adding it into fd poll list
[03/18/25 16:44:42-07:00] [I] [test_volume] [287520] [homeblks_impl.cpp:207:init_homestore] Initialize and start HomeStore is successfully
[ OK ] VolumeTest.CreateVolumeThenRecover (5269 ms)
[----------] 1 test from VolumeTest (5269 ms total)

[----------] Global test environment tear-down
[==========] 1 test from 1 test suite ran. (5269 ms total)
[ PASSED ] 1 test.

@yamingk yamingk requested a review from raakella1 March 18, 2025 23:52
@szmyd szmyd added this to the HomeBlocks2.1 milestone Mar 19, 2025
Copy link
Contributor

@raakella1 raakella1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LG

@yamingk yamingk merged commit 12d0498 into eBay:main Mar 19, 2025
27 checks passed
@yamingk yamingk deleted the yk_vol_fix branch March 19, 2025 21:30
@yamingk yamingk linked an issue Mar 19, 2025 that may be closed by this pull request
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.

Solo Repl Dev Recovery

3 participants