Project

General

Profile

Actions
Copy link

Bug #54283

closed

qa/cephfs: is_mounted() depends on a mutable variable

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

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

0%

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

Description

This can lead to bugs when variable value is stale or incorrect. It's safer to actually check if the file system is mounted.

Related issues 1 (0 open — 1 closed)

Related to CephFS - Bug #56476: qa/suites: evicted client unhandled in 4-compat_client.yamlResolvedRishabh Dave

Actions
Actions
Copy link
 #1

Updated by Venky Shankar about 2 years ago

  • Status changed from In Progress to Fix Under Review
  • Pull request ID set to 45036
Actions
Copy link
 #2

Updated by Rishabh Dave almost 2 years ago

  • Related to Bug #56476: qa/suites: evicted client unhandled in 4-compat_client.yaml added
Actions
Copy link
 #3

Updated by Rishabh Dave almost 2 years ago

The PR for this ticket needed fix for ticket 56476 in order to pass QA runs.

Actions
Copy link
 #4

Updated by Rishabh Dave almost 2 years ago

Rishabh Dave wrote:

The PR for this ticket needed fix for ticket 56476 in order to pass QA runs.

Not anymore. See: https://github.com/ceph/ceph/pull/45036#issuecomment-1191310804.

Actions
Copy link
 #5

Updated by Rishabh Dave over 1 year ago

  • Status changed from Fix Under Review to Resolved
Actions
Copy link

Also available in: Atom PDF