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
rbd: switched from legacy to new-style configuration options #16737
Conversation
Will move documentation over to schema once PR #16735 merges |
2d995d0
to
c131722
Compare
@@ -164,7 +164,6 @@ TEST_F(TestJournalEntries, AioDiscard) { | |||
REQUIRE_FEATURE(RBD_FEATURE_JOURNALING); | |||
|
|||
CephContext* cct = reinterpret_cast<CephContext*>(_rados.cct()); | |||
REQUIRE(!cct->_conf->rbd_skip_partial_discard); |
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.
@dillaman cct
variable is unused now. But I think we still need to skip the test when partial discard is enabled, due to journal events are optimized out for empty objects. See our discussion here: #10060 (comment)
% CEPH_ARGS='--rbd_skip_partial_discard=true' RBD_FEATURES=109 CEPH_LIB=./lib ./bin/unittest_librbd --gtest_filter=TestJournalEntries.AioDiscard
Note: Google Test filter = TestJournalEntries.AioDiscard
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from TestJournalEntries
[ RUN ] TestJournalEntries.AioDiscard
/home/mgolub/ceph.ci/src/test/librbd/journal/test_Entries.cc:181: Failure
Value of: wait_for_entries_available(ictx)
Actual: false
Expected: true
[ FAILED ] TestJournalEntries.AioDiscard (10018 ms)
[----------] 1 test from TestJournalEntries (10018 ms total)
[----------] Global test environment tear-down
[==========] 1 test from 1 test case ran. (10047 ms total)
[ PASSED ] 0 tests.
[ FAILED ] 1 test, listed below:
[ FAILED ] TestJournalEntries.AioDiscard
1 FAILED TEST
c131722
to
dd55d1f
Compare
@trociny update pushed |
@dillaman It needs update after #15579 merge.
|
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.
Fixes: http://tracker.ceph.com/issues/20737
?
please link with tracker if appropriate |
Fixes: http://tracker.ceph.com/issues/20737 Signed-off-by: Jason Dillaman <dillaman@redhat.com>
dd55d1f
to
7dc82bc
Compare
rebase pushed |
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
Signed-off-by: Jason Dillaman dillaman@redhat.com