Project

General

Profile

Actions

Feature #8605

closed

mon: add sanity checks on configurables

Added by Sage Weil almost 10 years ago. Updated almost 10 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Joao Eduardo Luis
Category:
Monitor
Target version:
% Done:

0%

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

Description

add a check_config() type method to the config observer and init to verify that important config tunables (like timeouts and intervals for elections, paxos, leases, clock skew) obey the necessary constraints for correctness.

checks should include these tunables, at a minimum:

- mon_accept_timeout
- mon_lease_ack_timeout
- mon_leas_renew_interval

See similar pull request for checks in the osd: https://github.com/ceph/ceph/pull/1964

Actions #1

Updated by Sage Weil almost 10 years ago

  • Target version changed from 0.83 to 0.83 cont.
Actions #2

Updated by Samuel Just almost 10 years ago

  • Target version deleted (0.83 cont.)
Actions #3

Updated by Joao Eduardo Luis almost 10 years ago

  • Status changed from New to In Progress
  • Assignee set to Joao Eduardo Luis

also, making this twofold:
- monitors spit out warnings if in-memory config options do not pass the sanity checks.
- monitor will not start if options don't pass the sanity checks

Actions #4

Updated by Sage Weil almost 10 years ago

  • Target version set to 0.83
Actions #5

Updated by Sage Weil almost 10 years ago

  • Status changed from In Progress to Resolved
Actions

Also available in: Atom PDF