test/store_test: fix broken test case#42918
Conversation
aclamk
left a comment
There was a problem hiding this comment.
On the side note - do we imagine going back to having allocations in b?
If so, then maybe UT should base required value on bluestore_allocation_from_file ?
After merging NCB stuff missed shared blob detection doesn't reveal extent leak any more - which is expected. Hence we need to adjust the relevant UT. Fixes:https://tracker.ceph.com/issues/52398 Signed-off-by: Igor Fedotov <ifedotov@suse.com>
c1d97ee to
6a0ac27
Compare
good point, updated. |
|
https://pulpito.ceph.com/yuriw-2021-08-26_18:40:53-rados-wip-yuri7-testing-2021-08-26-0841-distro-basic-smithi/ failures unrelated, tracked in: https://tracker.ceph.com/issues/52399 Details: Bug #52399: src/os/bluestore/HybridAllocator.cc: FAILED ceph_assert(false) - bluestore - Ceph |
After merging NCB stuff missed shared blob detection doesn't reveal
extent leak any more - which is expected. Hence we need to adjust the
relevant UT.
Fixes: https://tracker.ceph.com/issues/52398
Signed-off-by: Igor Fedotov ifedotov@suse.com
Checklist
Show available Jenkins commands
jenkins retest this pleasejenkins test classic perfjenkins test crimson perfjenkins test signedjenkins test make checkjenkins test make check arm64jenkins test submodulesjenkins test dashboardjenkins test dashboard cephadmjenkins test apijenkins test docsjenkins render docsjenkins test ceph-volume alljenkins test ceph-volume tox