Project

General

Profile

Feature #10509

mon: wishlist tracking ticket

Added by Joao Eduardo Luis over 6 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Monitor
Target version:
-
% Done:

33%

Source:
other
Tags:
Backport:
Reviewed:
Affected Versions:
Pull request ID:

Description

This is a tracking ticket for the monitor wishlist in src/doc/mon-wishlist.txt

Low-hanging fruit

  • audit helpers that put() messages but do not get() them.
    where possible, get rid of those put(). No one expects helpers to
    put() messages and that may lead to double frees. (issue #9378)

Medium complexity

  • get rid of QuorumServices. It seemed like a neat idea, but we only have
    one or two and they just add complexity and noise. (issue #10506)

Time consuming / complex

  • Split the OSDMonitor.cc file into auxiliary files. This will mean:

    1. Logically split subsystems (osd crush, osd pool, ...)
    2. Split the big badass functions, especially prepare/process_command()

  • Have Tracked Ops on the monitor, similarly to the OSDs. (issue #10507)
  • Move to Ref'erenced messages instead of pointers all around. This would
    also help with the Tracked Ops thing, as we'd be able to simply ignore all
    the get() and put() stuff behind it. (issue #3500)

Delicate / complex

  • Finer-grained Paxos::is_readable() and/or PaxosService::is_readable() (issue #10508)

    Rationale: a given service S should be able to read its committed state
    even though a Paxos proposal is happening, as long as the on-going
    proposal is not a value of service S.


Subtasks

Subtask #9378: mon: audit helpers that put() messages but do not get() themClosed

RADOS - Fix #3500: mon: use new MessageRef and intrusive_ptr's throughout the Monitor's message handlingNew

Subtask #10508: mon: finer-grain Paxos::is_readable() / PaxosService::is_readable()New


Related issues

Related to Ceph - Feature #10507: mon: have an OpTracker Resolved 01/10/2015

History

#1 Updated by Joao Eduardo Luis over 6 years ago

  • Description updated (diff)

#2 Updated by Joao Eduardo Luis over 6 years ago

  • Target version set to v0.94

#3 Updated by Joao Eduardo Luis over 6 years ago

  • Assignee set to Joao Eduardo Luis

#4 Updated by Joao Eduardo Luis over 6 years ago

  • Target version deleted (v0.94)

#5 Updated by Joao Eduardo Luis over 6 years ago

  • Assignee deleted (Joao Eduardo Luis)

#6 Updated by Joao Eduardo Luis about 1 year ago

  • Status changed from New to Closed

Also available in: Atom PDF