Project

General

Profile

Bug #15741

librados get_last_version() doesn't return correct result after aio completion

Added by David Zafman almost 8 years ago. Updated over 6 years ago.

Status:
New
Priority:
Normal
Assignee:
David Zafman
Category:
Dev Interfaces
Target version:
-
% Done:

0%

Source:
Development
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(RADOS):
Objecter, librados
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

After noticing that nothing in the aio path set the last_objver I tried an ASSERT into a unit test.

diff --git a/src/test/librados/misc.cc b/src/test/librados/misc.cc
index 7ddf5a8..31df4b3 100644
--- a/src/test/librados/misc.cc
+++ b/src/test/librados/misc.cc
@@ -579,6 +579,7 @@ TEST_F(LibRadosMiscPP, AioOperatePP) {
   ASSERT_EQ(0, ioctx.aio_operate("foo", my_completion, &o));
   ASSERT_EQ(0, my_completion->wait_for_complete_and_cb());
   ASSERT_EQ(my_aio_complete, true);
+  ASSERT_NE(0, ioctx.get_last_version());

   uint64_t size;
   time_t mtime;

[ RUN ] LibRadosMiscPP.AioOperatePP
test/librados/misc.cc:582: Failure
Expected: (0) != (ioctx.get_last_version()), actual: 0 vs 0
[ FAILED ] LibRadosMiscPP.AioOperatePP (20 ms)

History

#1 Updated by David Zafman almost 8 years ago

  • Description updated (diff)

#2 Updated by Greg Farnum almost 7 years ago

Any update on this, David? :)

#3 Updated by Greg Farnum almost 7 years ago

  • Project changed from Ceph to RADOS
  • Category set to Dev Interfaces
  • Component(RADOS) Objecter, librados added

#4 Updated by David Zafman over 6 years ago

This bug still exists.

Also available in: Atom PDF