Project

General

Profile

Actions

Backport #24296

closed

mimic: repeated eviction of idle client until some IO happens

Added by Nathan Cutler almost 6 years ago. Updated almost 6 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Target version:
Release:
mimic
Pull request ID:
Crash signature (v1):
Crash signature (v2):


Related issues 1 (0 open1 closed)

Copied from CephFS - Bug #24052: repeated eviction of idle client until some IO happensResolvedZheng Yan05/08/2018

Actions
Actions #1

Updated by Nathan Cutler almost 6 years ago

  • Copied from Bug #24052: repeated eviction of idle client until some IO happens added
Actions #2

Updated by Prashant D almost 6 years ago

  • Status changed from New to Need More Info

While backporting changes related to tracker 24052, getting cbegin not found compilation error :
/home/pdvian/backport/ceph3_mimic/src/mds/Migrator.cc: In member function ‘void Migrator::handle_export_caps_ack(MExportCapsAck*)’:
/home/pdvian/backport/ceph3_mimic/src/mds/Migrator.cc:3326:28: error: ‘using bufferlist = class ceph::buffer::list {aka class ceph::buffer::list}’ has no member named ‘cbegin’; did you mean ‘begin’?
auto blp = ack->cap_bl.cbegin();
^~~~
begin
make3: * [src/mds/CMakeFiles/mds.dir/build.make:399: src/mds/CMakeFiles/mds.dir/Migrator.cc.o] Error 1
make3: *
Waiting for unfinished jobs....
[ 52%] Building CXX object src/librbd/CMakeFiles/rbd_internal.dir/Operations.cc.o
Scanning dependencies of target mgr-dashboard-frontend-build

It will be good to backport PR#22015 completely or if not then we need to backport commit-id 12f76357ac9e9d5f6c50074c365bef04cb5d25b8 related changes to get this backport tracker to logical end.

Actions #3

Updated by Zheng Yan almost 6 years ago

just replace 'cbegin()' with begin()

Actions #4

Updated by Nathan Cutler almost 6 years ago

  • Description updated (diff)
  • Status changed from Need More Info to In Progress
  • Assignee set to Nathan Cutler
Actions #5

Updated by Nathan Cutler almost 6 years ago

Zheng Yan wrote:

just replace 'cbegin()' with begin()

Thanks, Zheng. Did just that.

Actions #6

Updated by Yuri Weinstein almost 6 years ago

Nathan Cutler wrote:

https://github.com/ceph/ceph/pull/22550

merged

Actions #7

Updated by Nathan Cutler almost 6 years ago

  • Status changed from In Progress to Resolved
  • Target version set to v13.2.1
Actions

Also available in: Atom PDF