Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Logging tweaks #12495

Merged
merged 4 commits into from Jan 31, 2023
Merged

Logging tweaks #12495

merged 4 commits into from Jan 31, 2023

Conversation

jsoref
Copy link
Contributor

@jsoref jsoref commented Jan 31, 2023

Short description

Adjust logging levels for a couple of messages that don't appear to be particularly fatal-error and fix a typo in another.

Checklist

I have:

  • read the CONTRIBUTING.md document
  • compiled this code
  • tested this code
  • included documentation (including possible behaviour changes)
  • documented the code
  • added or modified regression test(s)
  • added or modified unit test(s)
  • checked that this code was merged to master

@omoerbeek omoerbeek added the rec label Jan 31, 2023
@omoerbeek omoerbeek added this to the rec-4.9.0 milestone Jan 31, 2023
@jsoref
Adjust level for parsing to Notice
871f656 
This confirms to users that the zone has loaded.
@jsoref jsoref requested a review from omoerbeek January 31, 2023 14:03
@omoerbeek omoerbeek merged commit 25855ac into PowerDNS:master Jan 31, 2023
@jsoref jsoref deleted the logging-tweaks branch January 31, 2023 16:11
omoerbeek added a commit to omoerbeek/pdns that referenced this pull request Mar 3, 2023
@omoerbeek
rec: Backport 12495 to rex-4.8.x: Change a few logging urgency levels
0c2ec61 
Backport of PowerDNS#12495
@omoerbeek omoerbeek mentioned this pull request Mar 3, 2023
Merged
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@omoerbeek omoerbeek omoerbeek approved these changes

Assignees
No one assigned
Labels
rec
Projects
None yet
Milestone
rec-4.9.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@jsoref @omoerbeek