Project

General

Profile

Actions

Bug #54283

closed

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

Added by Rishabh Dave about 4 years ago. Updated 8 months ago.

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

0%

Source:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
Labels (FS):
Pull request ID:
Tags (freeform):
Fixed In:
v17.0.0-14392-gcef4c8d832b
Released In:
v18.2.0~1535
Upkeep Timestamp:
2025-07-14T07:41:12+00:00

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 open1 closed)

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

Actions
Actions #1

Updated by Venky Shankar about 4 years ago

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

Updated by Rishabh Dave over 3 years ago

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

Updated by Rishabh Dave over 3 years ago

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

Actions #4

Updated by Rishabh Dave over 3 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 #5

Updated by Rishabh Dave over 3 years ago

  • Status changed from Fix Under Review to Resolved
Actions #6

Updated by Upkeep Bot 8 months ago

  • Merge Commit set to cef4c8d832b1475d955b1e9df7a11f2cdaf430b9
  • Fixed In set to v17.0.0-14392-gcef4c8d832b
  • Released In set to v18.2.0~1535
  • Upkeep Timestamp set to 2025-07-14T07:41:12+00:00
Actions

Also available in: Atom PDF