Opened 2 years ago

Closed 21 months ago

#5463 closed enhancement (complete)

HA: Heartbeat

Reported by: tomek Owned by: UnAssigned
Priority: medium Milestone: Kea1.4
Component: high-availability Version: git
Keywords: Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity: N/A
Sub-Project: DHCP Feature Depending on Ticket:
Estimated Difficulty: 0 Add Hours to Ticket: 0
Total Hours: 0 Internal?: no

Description

Once #5461 (and possibly #5462) is implemented, the next step is to implement the actual heartbeat mechanism. One partner will send a command and the second one will respond.

It seems reasonable for each partner to include its failover state. One complication with this approach is that the heartbeat is not expected to be exchanged during heavy traffic. On the other hand, if there are many lease updates being exchanged, it is clear that both partners are healthy.

Subtickets

Change History (5)

comment:1 Changed 23 months ago by fdupont

IHMO the heartbeat message should contain a timestamp (same usage than a counter and in addition detects desynchronization of respective clocks).

comment:2 Changed 22 months ago by marcin

  • Owner set to marcin
  • Status changed from new to accepted

comment:3 Changed 22 months ago by marcin

  • Owner changed from marcin to UnAssigned
  • Status changed from accepted to reviewing

IMPORTANT NOTES:

This ticket contains much more than in the description. It is stacked on the trac5459 because #5459 hasn't been merged. Also this ticket contains the implementation of the entire heartbeat mechanism for v4, so it includes #5461. I suggest to simply review branch trac5463 of the premium repository since the following commit:

commit 69d95cbf70cf5ce0f9ffee061fc46ec500210fdc
Author: Marcin Siodelski <marcin@isc.org>
Date:   Thu Jan 18 19:48:54 2018 +0100

    [master] Added ChangeLog entry for #5454.

This review will cover tickets #5459, #5461 and #5463.

Also, the trac5463 branch has to be compiled with branch trac5457 of the Kea open source repository because trac5457 is still awaiting merge to master.

comment:4 Changed 22 months ago by marcin

Proposed ChangeLog entry:

XX.	[func]		marcin
	Implemented heartbeat mechanism in the libkea-ha hooks library.
	(Trac #5463, git cafe)

comment:5 Changed 21 months ago by marcin

  • Resolution set to complete
  • Status changed from reviewing to closed

Reviewed and merged as part of the #5476.

Note: See TracTickets for help on using tickets.