Project

General

Profile

Bug #41585

mds: client evicted twice in one tick

Added by Erqi Chen 20 days ago. Updated 2 days ago.

Status:
Pending Backport
Priority:
Normal
Assignee:
Category:
-
Target version:
Start date:
Due date:
% 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:

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 New
Copied to fs - Backport #41886: nautilus: mds: client evicted twice in one tick New

History

#1 Updated by Patrick Donnelly 10 days ago

  • Status changed from New to Need 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 2 days ago

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

#3 Updated by Nathan Cutler 2 days ago

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

#4 Updated by Nathan Cutler 2 days ago

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

Also available in: Atom PDF