#5447 closed task (complete)

Create stub libdhcp_ha (high availability) hooks library

Reported by: marcin Owned by: marcin
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

The high availability feature will be implemented within a hooks library. This ticket is about creating skeleton for this library.

Subtickets

Change History (12)

comment:1 Changed 14 months ago by marcin

  • Owner set to UnAssigned
  • Status changed from new to reviewing

Added skeleton libdhcp_ha.so library. Ready for review.

Proposed ChangeLog entry:

XX.	[func]		marcin
	Skeleton libdhcp_ha hooks library added for implementation of High Availability.
	(Trac #5447, git cafe)

comment:2 follow-up: Changed 14 months ago by fdupont

If nobody wants it I'll take the review. By its a premium tier2 stuff so at least there should be a plain Kea branch updating the doc...

comment:3 Changed 14 months ago by fdupont

  • Owner changed from UnAssigned to fdupont

comment:4 follow-up: Changed 14 months ago by fdupont

  • Owner changed from fdupont to marcin

Git ignore files are text files so should terminate by a linefeed character (\n). I added them so please pull.
I took the occasion to fix the ./configure warnings by defining CPPP. IMHO if the ticket is not merged quickly it should be fixed in the premium master branch.
There is nothing in the plain Kea documentation: when the hooks library code is really a skeleton IMHO we should at least reverse the name...

comment:5 in reply to: ↑ 2 Changed 13 months ago by marcin

Replying to fdupont:

If nobody wants it I'll take the review. By its a premium tier2 stuff so at least there should be a plain Kea branch updating the doc...

There will be a dedicated ticket updating the doc. However, the library doesn't do anything useful at the moment, so I don't see why we'd need to create doc for it.

comment:6 in reply to: ↑ 4 Changed 13 months ago by marcin

Replying to fdupont:

Git ignore files are text files so should terminate by a linefeed character (\n). I added them so please pull.
I took the occasion to fix the ./configure warnings by defining CPPP. IMHO if the ticket is not merged quickly it should be fixed in the premium master branch.

Thanks for your fixes.

There is nothing in the plain Kea documentation: when the hooks library code is really a skeleton IMHO we should at least reverse the name...

What do you mean by "reverse the name" ?

comment:7 Changed 13 months ago by marcin

  • Owner changed from marcin to fdupont

comment:8 Changed 13 months ago by fdupont

reverse -> reserve

comment:9 Changed 13 months ago by fdupont

  • Owner changed from fdupont to marcin

comment:10 Changed 13 months ago by marcin

  • Owner changed from marcin to fdupont

I created a branch in the main Kea repository and updated the docs with placeholder section for libdhcp_ha hook library.

comment:11 Changed 13 months ago by fdupont

  • Owner changed from fdupont to marcin

Fine. Please merge!

comment:12 Changed 13 months ago by marcin

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

Merged with commits d939b5b8bc4befb24daf863f2408d97493e4bfbf and dafc7066bf9a7700ef8e109453f27b071a577307 into core and premium respectively.

Note: See TracTickets for help on using tickets.