Opened 7 years ago

Closed 5 years ago

Last modified 4 years ago

#2146 closed enhancement (fixed)

Investigate dhcp{4,6} start-up merge possiblity

Reported by: tomek Owned by: tomek
Priority: medium Milestone: Outstanding Tasks
Component: dhcp Version: git
Keywords: Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity: Low
Sub-Project: DHCP Feature Depending on Ticket:
Estimated Difficulty: 7 Add Hours to Ticket: 0
Total Hours: 0 Internal?: no

Description

DHCPv4 and DHCPv6 servers have many similarities:

  • main() functions in src/bin/dhcp{4,6}/main.cc
  • ControlledDhcpv?{4,6}Srv classes.
  • TestDhcpv6Daemon from srv/bin/dhcp{4,6}/tests/dhcp{4,6}_test.py

Some form of merge should be investigated. A common base class, or perhaps template would be helpful.

It should be noted that in some cases the code is similar now, but is expected to grow differently in the future, so unification should not be the automatic answer and some thought should be given.

Subtickets

Change History (6)

comment:1 Changed 7 years ago by shane

  • Milestone changed from New Tasks to DHCP 2012

comment:2 Changed 7 years ago by stephen

  • Milestone changed from DHCP Outstanding Tasks to Sprint-DHCP-20130328

comment:3 Changed 7 years ago by tomek

  • Milestone changed from Sprint-DHCP-20130328 to DHCP Outstanding Tasks

comment:4 Changed 5 years ago by tomek

  • Owner set to tomek
  • Status changed from new to assigned
  • Version set to git

We do have isc::dhcp::Daemon class now that is shared by DHCPv4, DHCPv6 and DDNS.

comment:5 Changed 5 years ago by tomek

  • Resolution set to fixed
  • Status changed from assigned to closed

comment:6 Changed 4 years ago by tomek

  • Milestone changed from DHCP Outstanding Tasks to Outstanding Tasks

Milestone renamed

Note: See TracTickets for help on using tickets.