Project

General

Profile

Bug #41585

mds: client evicted twice in one tick

Added by Erqi Chen about 1 year ago. Updated 10 months ago.

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

0%

Source:
Community (dev)
Tags:
Backport:
nautilus,mimic
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
MDS
Labels (FS):
Pull request ID:
Crash signature:

Description

2019-08-09 14:41:39.292140 7fd33eba7700 0 log_channel(cluster) log [WRN] : client id 2646901 has not responded to cap revoke by MDS for over 30 seconds, evicting
2019-08-09 14:41:42.431933 7fd33eba7700 0 log_channel(cluster) log [WRN] : client id 2646901 has not responded to cap revoke by MDS for over 30 seconds, evicting

std::vector<client_t> Locker::get_late_revoking_clients(double timeout) const
{
  std::vector<client_t> result;

  if (any_late_revoking_caps(revoking_caps, timeout)) {
    // Slow path: execute in O(N_clients)
    for (auto &p : revoking_caps_by_client) {
      if (any_late_revoking_caps(p.second, timeout)) {
        result.push_back(p.first);
      }
    }
  } else {
    // Fast path: no misbehaving clients, execute in O(1)
  }
  return result;
}

client is put to to_evict vector twice in the situation client does't response two caps revoke.


Related issues

Copied to fs - Backport #41885: mimic: mds: client evicted twice in one tick Resolved
Copied to fs - Backport #41886: nautilus: mds: client evicted twice in one tick Resolved

History

#1 Updated by Patrick Donnelly about 1 year ago

  • Status changed from New to Fix Under Review
  • Assignee set to Erqi Chen
  • Target version set to v15.0.0
  • Source set to Community (dev)
  • Backport set to nautilus,mimic
  • Pull request ID set to 30029
  • Component(FS) MDS added

#2 Updated by Patrick Donnelly about 1 year ago

  • Status changed from Fix Under Review to Pending Backport
  • Start date deleted (08/30/2019)

#3 Updated by Nathan Cutler about 1 year ago

  • Copied to Backport #41885: mimic: mds: client evicted twice in one tick added

#4 Updated by Nathan Cutler about 1 year ago

  • Copied to Backport #41886: nautilus: mds: client evicted twice in one tick added

#5 Updated by Nathan Cutler 11 months ago

  • Description updated (diff)

#6 Updated by Nathan Cutler 10 months ago

  • Status changed from Pending Backport to Resolved

While running with --resolve-parent, the script "backport-create-issue" noticed that all backports of this issue are in status "Resolved" or "Rejected".

Also available in: Atom PDF