Project

General

Profile

Bug #39159

qa: Fix ambiguous store_thrash thrash_store in mon_thrash.py

Added by Jos Collin 4 months ago. Updated 12 days ago.

Status:
Resolved
Priority:
Low
Assignee:
Category:
Correctness/Safety
Target version:
Start date:
Due date:
% Done:

0%

Source:
Development
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(RADOS):
Monitor
Pull request ID:

Description

Both store_thrash and thrash_store names are used for the same thing in mon_thrash.py. 'thrash_store' is used here: https://github.com/ceph/ceph/blob/master/qa/suites/rados/monthrash/thrashers/force-sync-many.yaml#L11 and 'store_thrash' is read in mon_thrash.py.

History

#1 Updated by Patrick Donnelly 4 months ago

  • Project changed from fs to RADOS
  • Subject changed from qa/tasks: Fix ambiguous store_thrash thrash_store in mon_thrash.py to qa: Fix ambiguous store_thrash thrash_store in mon_thrash.py
  • Category changed from Correctness/Safety to Correctness/Safety
  • Assignee set to Jos Collin
  • Target version set to v15.0.0
  • Start date deleted (04/09/2019)
  • Source set to Development
  • Component(RADOS) Monitor added

#2 Updated by Jos Collin 4 months ago

  • Status changed from New to Need Review
  • Pull request ID set to 39159

#3 Updated by Jos Collin 4 months ago

  • Pull request ID changed from 39159 to 27542

#4 Updated by Jos Collin 4 months ago

  • Description updated (diff)

#5 Updated by Jos Collin 4 months ago

  • Backport set to nautilus, luminous

#6 Updated by Nathan Cutler 4 months ago

  • Backport changed from nautilus, luminous to nautilus, mimic, luminous

#7 Updated by Sage Weil 12 days ago

  • Status changed from Need Review to Resolved
  • Backport deleted (nautilus, mimic, luminous)

I don't think this needs a backport... ?

Also available in: Atom PDF