Project

General

Profile

Actions

Bug #55316

open

qa: add client asok support to get the options

Added by Xiubo Li about 2 years ago. Updated almost 2 years ago.

Status:
New
Priority:
Normal
Category:
Administration/Usability
Target version:
-
% Done:

0%

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

Description

Currently for the vstart_runner.py it only support mon/mds/osd:

 839     def get_config(self, key, service_type=None):                                                                                                 
 840         if service_type is None:
 841             service_type = 'mon'
 842   
 843         # FIXME hardcoded vstart service IDs
 844         service_id = {
 845             'mon': 'a',
 846             'mds': 'a',
 847             'osd': '0'
 848         }[service_type]
 849   
 850         return self.json_asok(['config', 'get', key], service_type, service_id)[key]

We need to support the client service type, because some options only exist in the client daemons.

For more context see: https://github.com/ceph/ceph/pull/45869#pullrequestreview-946808926

Actions #1

Updated by Venky Shankar almost 2 years ago

  • Category set to Administration/Usability
  • Assignee set to Neeraj Pratap Singh
  • Backport set to quincy

Neeraj, guessing this is probably required for writing test to be run by vstart_runner for https://github.com/ceph/ceph/pull/45869?

Actions #2

Updated by Rishabh Dave almost 2 years ago

  • Description updated (diff)
Actions #3

Updated by Rishabh Dave almost 2 years ago

  • Description updated (diff)
Actions

Also available in: Atom PDF