-
Notifications
You must be signed in to change notification settings - Fork 2
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.
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
(dnscache role) change default bind logging to notice
#659
Conversation
a5df41b
to
d6ad85c
Compare
@@ -8,7 +8,7 @@ dns::sysconfig_startup_options: "-4" | |||
dns::additional_directives: | |||
- "logging {" | |||
- " channel queries_syslog {" | |||
- " severity info;" | |||
- " severity notice;" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to make sure logs queries entries are still getting to graylog when changed to 'notice', which host is in this branch already?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dns2.ls.lsst.org
, dns3.ls.lsst.org
, and dns1.dev.lsst.org
are on a env that includes these commits.
d6ad85c
to
21264e8
Compare
Rebased on current master. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
a7c7b82
to
d818521
Compare
On EL8, severity level of `info` was causing all queries to be logged.
d818521
to
18065b7
Compare
Rebased on current master. |
On EL8, severity level of
info
was causing all queries to be logged.