-
Notifications
You must be signed in to change notification settings - Fork 15
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
Issue# 412: Home Replication Long Running Setup with Real Drives #418
Conversation
@@ -177,7 +178,8 @@ class HSTestHelper { | |||
|
|||
static void start_homestore(const std::string& test_name, std::map< uint32_t, test_params >&& svc_params, | |||
hs_before_services_starting_cb_t cb = nullptr, bool fake_restart = false, | |||
bool init_device = true, uint32_t shutdown_delay_sec = 5) { | |||
bool init_device = true, uint32_t shutdown_delay_sec = 5, | |||
std::vector< std::string > cust_dev_names = {}) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is cust
stands for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"customized".
src/lib/device/device_manager.cpp
Outdated
@@ -36,16 +36,19 @@ static int determine_open_flags(io_flag oflags) { | |||
|
|||
switch (oflags) { | |||
case io_flag::BUFFERED_IO: | |||
open_flags = O_RDWR | O_CREAT; | |||
// open_flags = O_RDWR | O_CREAT; | |||
open_flags = O_RDWR; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change doesn't make much difference, but thought is the right thing to do.
PS, 1.3 we also always pass this O_CREAT flag.
@@ -113,7 +113,7 @@ if (${io_tests}) | |||
can_build_epoll_io_tests(epoll_tests) | |||
if(${epoll_tests}) | |||
add_test(NAME LogDev-Epoll COMMAND ${CMAKE_BINARY_DIR}/bin/test_log_dev) | |||
add_test(NAME LogStore-Epoll COMMAND ${CMAKE_BINARY_DIR}/bin/test_log_store) | |||
#add_test(NAME LogStore-Epoll COMMAND ${CMAKE_BINARY_DIR}/bin/test_log_store) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hanging in github CI, seen also in other PR's CI build: https://github.com/eBay/HomeStore/actions/runs/9107660378/job/25036931542?pr=426.
seems consistently reproducable
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#424 is tracking this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
@@ -113,7 +113,7 @@ if (${io_tests}) | |||
can_build_epoll_io_tests(epoll_tests) | |||
if(${epoll_tests}) | |||
add_test(NAME LogDev-Epoll COMMAND ${CMAKE_BINARY_DIR}/bin/test_log_dev) | |||
add_test(NAME LogStore-Epoll COMMAND ${CMAKE_BINARY_DIR}/bin/test_log_store) | |||
#add_test(NAME LogStore-Epoll COMMAND ${CMAKE_BINARY_DIR}/bin/test_log_store) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#424 is tracking this
Test Command:
./test_raft_repl_dev --gtest_filter=Snapshot_and_Compact --replica_dev_list /dev/data1,/dev/data2,/dev/data3 --num_io=999999 --snapshot_distance=200 --num_raft_logs_resv=20000 --res_mgr_audit_timer_ms=120000
from log, each replica can take the real device with 5TB size and I/Os can be run. Snapshot and Truncation is going through.