Project

General

Profile

Actions

Cleanup #10506

open

mon: get rid of QuorumServices

Added by Joao Eduardo Luis over 9 years ago. Updated almost 7 years ago.

Status:
New
Priority:
Normal
Assignee:
Joao Eduardo Luis
Category:
Code Hygiene
Target version:
% Done:

0%

Tags:
Backport:
Reviewed:
Affected Versions:
Component(RADOS):
Monitor
Pull request ID:

Description

A QuorumService is an abstraction allowing for easier management of services requiring a quorum to work correctly.

While it seemed like a good idea at the time, hopeful of being useful for the future expansion of the monitor in terms of in-memory-only services (say, for health purposes), it turned out to be a slight annoyance code-wise.

At the moment, we have 3 services defined in mon/QuorumService.h: HEALTH, TIMECHECK, CONFIG_KEY. Of these, only two are built using the QuorumService class: HealthService and ConfigKeyService. Timechecks are embedded in the Monitor class code.

HealthService is another mind boggling piece of code: defines a HEALTH_DATA service and is implemented by mon/HealthMonitor.cc, which in turn will manage mon/DataHealthService.cc, the latter being responsible for tracking monitor disk usage across the quorum.

Given the small usefulness of this infrastructure compared to its large footprint, I would very much enjoy getting rid of it all or implement it in a way that's so trivial that doesn't make my head hurt.

Actions #1

Updated by Joao Eduardo Luis over 9 years ago

  • Parent task set to #10509
Actions #2

Updated by Joao Eduardo Luis almost 7 years ago

  • Tracker changed from Subtask to Cleanup
  • Project changed from Ceph to RADOS
  • Subject changed from mon: get rid/refactor of QuorumServices to mon: get rid of QuorumServices
  • Category changed from Monitor to Code Hygiene
  • Assignee set to Joao Eduardo Luis

I don't think the QuorumService interface is bringing enough to the table to keep it around.

What we are achieving with it easily be mimicked with the PaxosService, but keeping it readable and, in case of the DataHealth stuff, also always writeable. On the other hand, moving the ConfigKeyService to a PaxosService makes a lot of sense now.

Actions #3

Updated by Joao Eduardo Luis almost 7 years ago

  • Parent task deleted (#10509)
  • Component(RADOS) Monitor added
Actions #4

Updated by Joao Eduardo Luis almost 7 years ago

  • Target version set to 540
Actions #5

Updated by Joao Eduardo Luis almost 7 years ago

  • Target version changed from 540 to v13.0.0
Actions

Also available in: Atom PDF