Opened 4 years ago

Closed 4 years ago

#4316 closed enhancement (fixed)

change of the logging levels in KEA

Reported by: acoon Owned by: tmark
Priority: high Milestone: Kea1.1
Component: logging Version: 1.0.0
Keywords: logging level 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: 1 Internal?: no

Description

The default logging level of INFO is rather low.
Especially i dint see why we need to set logging to DEBUG level 99 to get a lease log, which is needed currently to backtrace users. I would expect a default logging facility with more useful information.

After a discussion on kea mailling list I was asked by Tomek Mrugalski to create this ticket.

Subtickets

Change History (7)

comment:1 Changed 4 years ago by acoon

  • Type changed from defect to enhancement

comment:2 Changed 4 years ago by stephen

Suggest that the code be changed to

  • Change the logger for lease-related activity to something like "leases"
  • Change the logging level for lease acquisitions, renewals, declines etc. to INFO.

comment:3 Changed 4 years ago by hschempf

  • Milestone changed from Kea-proposed to Kea1.1

Per team meeting Mar 17, accept 1.1, est .5 days

comment:4 Changed 4 years ago by tmark

  • Owner set to tmark
  • Status changed from new to assigned

comment:5 Changed 4 years ago by tmark

  • Owner changed from tmark to Unassigned
  • Status changed from assigned to reviewing
  • Total Hours changed from 0 to 3

The primary lease events (grants, renewals, releases, declines) are now being logged to lease4_logger and lease6_logger at the INFO logging level. Two logging messages were added to kea-dhcp6 to log renewals as the server was not logging them at any level. This at least makes the information being logged more or less symetrical between the v4 and v6 servers. There is some question as to whether renewals might produce too much log output. It may be more prudent to log renewals at the DEBUG level.

The suggested ChangeLog entry:

10xx.   [func]      tmark
    Both kea-dhcp4 and kea-dhcp6, now log the primary lease events (e.g.
    grants, renewals, releases, declines) at the INFO log level to the
    lease4_logger and lease6_logger repsectively.  Prior to this these
    events were logged at the DEBUG log level.
    (Trac #4316, git TBD)

comment:6 Changed 4 years ago by fdupont

  • Owner changed from Unassigned to tmark

Looks OK!

comment:7 Changed 4 years ago by tmark

  • Resolution set to fixed
  • Status changed from reviewing to closed
  • Total Hours changed from 3 to 1

Changes merged with git 9beca27e3d76d0ccec925125f23074227db08869
Added ChangeLog? entry 1096.

Ticket is complete.

Note: See TracTickets for help on using tickets.