Project

General

Profile

Actions

Bug #15741

open

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)

Actions #1

Updated by David Zafman almost 8 years ago

  • Description updated (diff)
Actions #2

Updated by Greg Farnum almost 7 years ago

Any update on this, David? :)

Actions #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
Actions #4

Updated by David Zafman over 6 years ago

This bug still exists.

Actions

Also available in: Atom PDF