Project

General

Profile

Actions

Cleanup #47160

closed

qa/tasks/cephfs: Break up test_volumes.py

Added by Ramana Raja over 3 years ago. Updated over 3 years ago.

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

0%

Tags:
Backport:
Reviewed:
Affected Versions:
Component(FS):
qa-suite
Labels (FS):
qa
Pull request ID:

Description

The test_volumes has become unwieldy with growing number of non-trivial tests to test growing set of features in mgr/volumes. Adding/reviewing tests during code development and tracking down qa test run failures has become harder.

Solution as per Patrick Donnelly:
"To be clear: this is as simple as breaking the volumes tests into separate classes in teh same file. Then the yaml fragment [1] would be split into multiple yaml fragments that each refer to a specific class of tests."

[1] https://github.com/ceph/ceph/blob/d2a3fcc728c31b25166aeb55c99b9ccf8d4985bb/qa/suites/fs/basic_functional/tasks/volumes.yaml#L20

Actions #1

Updated by Patrick Donnelly over 3 years ago

To be clear: this is as simple as breaking the volumes tests into separate classes in teh same file. Then the yaml fragment [1] would be split into multiple yaml fragments that each refer to a specific class of tests.

[1] https://github.com/ceph/ceph/blob/d2a3fcc728c31b25166aeb55c99b9ccf8d4985bb/qa/suites/fs/basic_functional/tasks/volumes.yaml#L20

Actions #2

Updated by Ramana Raja over 3 years ago

  • Subject changed from qa/tasks/cephfs: break up test_volumes.py into smaller modules to qa/tasks/cephfs: reak up test_volumes.py
  • Description updated (diff)
  • Assignee deleted (Shyamsundar Ranganathan)
  • Backport set to octopus,nautilus
Actions #3

Updated by Ramana Raja over 3 years ago

  • Description updated (diff)
Actions #4

Updated by Ramana Raja over 3 years ago

  • Subject changed from qa/tasks/cephfs: reak up test_volumes.py to qa/tasks/cephfs: Break up test_volumes.py
Actions #5

Updated by Ramana Raja over 3 years ago

  • Status changed from New to In Progress
  • Assignee set to Ramana Raja
Actions #6

Updated by Rishabh Dave over 3 years ago

  • Pull request ID set to 37593
Actions #7

Updated by Patrick Donnelly over 3 years ago

  • Status changed from In Progress to Resolved
  • Backport deleted (octopus,nautilus)

I don't think this will be feasible to backport without significant effort. Ramana, do you think it's worth it?

Actions

Also available in: Atom PDF