Skip to content

Commit

Permalink
Merge pull request ceph#35023 from yuvalif/fix_issue_45513
Browse files Browse the repository at this point in the history
cls/2pc_queue: fix timing issue in test

Reviewed-by: Casey Bodley <[email protected]>
  • Loading branch information
cbodley committed May 15, 2020
2 parents 7d6bff6 + 3689283 commit 04a430d
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/test/cls_2pc_queue/test_cls_2pc_queue.cc
Original file line number Diff line number Diff line change
Expand Up @@ -505,7 +505,7 @@ TEST_F(TestCls2PCQueue, Cleanup)
}
});
}

auto cleaned_reservations = 0U;
auto committed_reservations = 0U;
auto aborter = std::thread([this, &queue_name, &stale_time, &cleaned_reservations, &committed_reservations] {
Expand Down Expand Up @@ -544,7 +544,7 @@ TEST_F(TestCls2PCQueue, Cleanup)
aborter.join();

ASSERT_GT(cleaned_reservations, 0);
ASSERT_GT(committed_reservations, 0);
ASSERT_EQ(committed_reservations + cleaned_reservations, number_of_ops*max_workers);
cls_2pc_reservations reservations;
ASSERT_EQ(0, cls_2pc_queue_list_reservations(ioctx, queue_name, reservations));
ASSERT_EQ(reservations.size(), 0);
Expand Down

0 comments on commit 04a430d

Please sign in to comment.