Opened 8 months ago

Closed 6 months ago

#5534 closed enhancement (complete)

cache-remove, cache-flush commands

Reported by: tomek Owned by: fdupont
Priority: medium Milestone: Kea1.4
Component: host-reservations 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: 4
Total Hours: 0 Internal?: no

Description

The RadiusDesign calls for implementation of a caching mechanism. The hosts returned by other backends may be cached. However, this brings a topic of dealing with the information in the cache becoming stale (i.e. they were changed in the original backend, but cache contains old information about it).

To deal with this problem, at least two commands should be implemented:

  • cache-remove that removes a single host
  • cache-flush that removes all hosts

Additional commands may be added if desired, but only if their implementation does not significantly extend the time needed to implement this ticket.

Subtickets

Change History (8)

comment:1 Changed 7 months ago by fdupont

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

The cache-flush already exists and the code is ready for the cache-remove control as there is a cache-insert doing the opposite.
Requires #5532 (but not #5533).

comment:2 Changed 7 months ago by fdupont

  • Add Hours to Ticket changed from 0 to 4
  • Owner changed from fdupont to UnAssigned
  • Status changed from accepted to reviewing

Done. Interface looks like the reservation-del's one.
Ready for review (after #5532).

comment:3 Changed 7 months ago by fdupont

  • Owner changed from UnAssigned to fdupont

comment:4 Changed 7 months ago by fdupont

  • Owner changed from fdupont to UnAssigned

Please review premium trac5534a (note the 'a') branch.

comment:5 Changed 7 months ago by fdupont

Please review kea trac5533a and premium trac5533.
(#5533 is after #5534 in the host cache train).

comment:6 Changed 7 months ago by tomek

  • Owner changed from UnAssigned to tomek

Since 5528, 5531, 5532, 5433 and 5534 are on the same branch, reviewing all them at once.

comment:7 Changed 6 months ago by tomek

  • Owner changed from tomek to fdupont

The code on trac5533 (premium) is ready for merge. See comment ticket:5533#comment:14 for details.

comment:8 Changed 6 months ago by fdupont

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

5533 was merged. Closing.

Note: See TracTickets for help on using tickets.