Project

General

Profile

Bug #61349

ObjectWriteOperation::mtime2() works with IoCtx::operate() but not aio_operate()

Added by Casey Bodley 9 months ago. Updated 3 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
-
% Done:

100%

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

Description

librados::IoCtxImpl::operate() takes an optional ceph::real_time* and uses it when given

but librados::IoCtxImpl::aio_operate() does not accept an mtime, and always uses ceph::real_clock::now() instead


Related issues

Copied to RADOS - Backport #61729: reef: ObjectWriteOperation::mtime2() works with IoCtx::operate() but not aio_operate() Resolved
Copied to RADOS - Backport #61730: pacific: ObjectWriteOperation::mtime2() works with IoCtx::operate() but not aio_operate() Resolved
Copied to RADOS - Backport #61731: quincy: ObjectWriteOperation::mtime2() works with IoCtx::operate() but not aio_operate() Resolved

History

#1 Updated by Casey Bodley 9 months ago

  • Status changed from New to Fix Under Review
  • Pull request ID set to 51681

#2 Updated by Casey Bodley 9 months ago

for background, https://github.com/ceph/ceph/pull/50206 changes some of rgw's librados operations to aio_operate(), and we found that the mtime2() call was no longer taking effect

#3 Updated by Casey Bodley 9 months ago

  • Status changed from Fix Under Review to Pending Backport

#4 Updated by Backport Bot 9 months ago

  • Copied to Backport #61729: reef: ObjectWriteOperation::mtime2() works with IoCtx::operate() but not aio_operate() added

#5 Updated by Backport Bot 9 months ago

  • Copied to Backport #61730: pacific: ObjectWriteOperation::mtime2() works with IoCtx::operate() but not aio_operate() added

#6 Updated by Backport Bot 9 months ago

  • Copied to Backport #61731: quincy: ObjectWriteOperation::mtime2() works with IoCtx::operate() but not aio_operate() added

#7 Updated by Backport Bot 9 months ago

  • Tags changed from librados to librados backport_processed

#8 Updated by Konstantin Shalygin 3 months ago

  • Status changed from Pending Backport to Resolved
  • % Done changed from 0 to 100
  • Source set to Development

Also available in: Atom PDF