Project

General

Profile

Actions

Bug #56416

closed

qa/cephfs: delete path from cmd args after use

Added by Rishabh Dave over 3 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-13672-g4c24755126c
Released In:
v18.2.0~1775
Upkeep Timestamp:
2025-07-13T10:41:38+00:00

Description

Method conduct_neg_test_for_write_caps() in qa/tasks/cephfs/caps_helper.py appends path to command arguments but doesn't delete it after execution. This will lead to 2 paths in second iteration which will lead to a bug in tests.

    def conduct_neg_test_for_write_caps(self, filepaths, mounts):
        cmdargs = ['echo', 'some random data', Raw('|'), 'tee']

        for mount in mounts:
            for path in filepaths:
                if path.find(mount.hostfs_mntpt) != -1:
                    cmdargs.append(path)
                    mount.negtestcmd(args=cmdargs, retval=1,
                                     errmsg='permission denied')

Removing path after the command has bee executed will fix the issue.

Actions #1

Updated by Rishabh Dave over 3 years ago

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

Updated by Rishabh Dave over 3 years ago

  • Pull request ID changed from 41778 to 46991
Actions #3

Updated by Upkeep Bot 8 months ago

  • Merge Commit set to 4c24755126cecd564a27a97342efdd2b356fedeb
  • Fixed In set to v17.0.0-13672-g4c24755126c
  • Released In set to v18.2.0~1775
  • Upkeep Timestamp set to 2025-07-13T10:41:38+00:00
Actions

Also available in: Atom PDF