Project

General

Profile

Actions

Bug #56476

closed

qa/suites: evicted client unhandled in 4-compat_client.yaml

Added by Rishabh Dave almost 2 years ago. Updated over 1 year ago.

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

0%

Source:
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
qa-suite
Labels (FS):
qa, qa-failure
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

In 4-compat_client.yaml creates two clients, evicts one of them but the evicted client is left unhandled which leads to failure for final teardown -

- fs.clients_evicted:
    clients:
      client.0: False
      client.1: True

The evicted client is not cleaned up later, that is, it is left unmounted and the mount point is left undeleted. This doesn't cause any failure during final teardown for main branch but, with PR number 45036, it does lead to teuthology job failures every time.

With PR 45036, the "qa/cephfs" code does not depend (anymore) on the value of the attribute "is_mounted" to check if a CephFS has been unmounted or not. Instead, delete the attribute "is_mounted" it and runs the command "findmnt <mntpt>" to check if the client was actually unmounted from the system's FS.

Operating on a CephFS mount point after the client has been evicted causes the operation to hang. Thus, with PR 45036, the final teardown for this teuthology job fails every time.

The fix for this is too run umount -f for the mountpoint of evicted client and delete the mountpoint. Then, final teardown will run fine with PR 45036 too.


Related issues 1 (0 open1 closed)

Related to CephFS - Bug #54283: qa/cephfs: is_mounted() depends on a mutable variableResolvedRishabh Dave

Actions
Actions #1

Updated by Rishabh Dave almost 2 years ago

  • Description updated (diff)
Actions #2

Updated by Rishabh Dave almost 2 years ago

  • Description updated (diff)
Actions #3

Updated by Rishabh Dave almost 2 years ago

  • Assignee set to Rishabh Dave
Actions #4

Updated by Rishabh Dave almost 2 years ago

  • Pull request ID set to 46988
Actions #5

Updated by Rishabh Dave almost 2 years ago

  • Related to Bug #54283: qa/cephfs: is_mounted() depends on a mutable variable added
Actions #6

Updated by Xiubo Li almost 2 years ago

  • Status changed from In Progress to Fix Under Review
Actions #7

Updated by Rishabh Dave almost 2 years ago

  • Description updated (diff)
Actions #8

Updated by Rishabh Dave almost 2 years ago

  • Description updated (diff)
Actions #9

Updated by Rishabh Dave almost 2 years ago

  • Description updated (diff)
Actions #10

Updated by Rishabh Dave almost 2 years ago

  • Description updated (diff)
Actions #11

Updated by Rishabh Dave almost 2 years ago

  • Description updated (diff)
Actions #12

Updated by Rishabh Dave over 1 year ago

  • Status changed from Fix Under Review to Resolved
Actions

Also available in: Atom PDF