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

recursor: Add pdnslog to Lua configuration scripts #6848

Merged
merged 1 commit into from Sep 3, 2018

Conversation

Projects
None yet
3 participants
@zeha
Collaborator

zeha commented Aug 10, 2018

Short description

As a distro maintainer, I'd like to ship a default Lua config file that also logs something, so users can know that they are not running the default empty configuration.

I intend to carry this as a 4.1 patch in Debian.

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)

@zeha zeha changed the title from Add pdnslog to Lua configuration scripts to [WIP] Add pdnslog to Lua configuration scripts Aug 10, 2018

@zeha zeha changed the title from [WIP] Add pdnslog to Lua configuration scripts to Add pdnslog to Lua configuration scripts Aug 10, 2018

@zeha zeha added this to the rec-4.2.0 milestone Aug 14, 2018

ploxiln added a commit to ploxiln/pdns that referenced this pull request Aug 30, 2018

ploxiln added a commit to ploxiln/pdns that referenced this pull request Aug 30, 2018

changelog: 4.1.4 back-ported PowerDNS#6648 not PowerDNS#6848
pull-request PowerDNS#6648 is mysql transaction-isolation syntax change
pull-request PowerDNS#6848 is lua logging stuff

@ploxiln ploxiln referenced this pull request Aug 30, 2018

Merged

changelog: 4.1.4 back-ported #6648 not #6848 #6898

1 of 7 tasks complete

pieterlexis added a commit that referenced this pull request Aug 30, 2018

Merge pull request #6898 from ploxiln/changelog_414_fix
changelog: 4.1.4 back-ported #6648 not #6848

@rgacogne rgacogne referenced this pull request Sep 3, 2018

Merged

recursor 4.1: Add pdnslog to Lua configuration scripts #6919

3 of 8 tasks complete

@zeha zeha changed the title from Add pdnslog to Lua configuration scripts to recursor: Add pdnslog to Lua configuration scripts Sep 3, 2018

@rgacogne

LGTM!

@Habbie Habbie merged commit ad62915 into PowerDNS:master Sep 3, 2018

3 of 4 checks passed

LGTM analysis: C/C++ Running analyses for revisions
Details
LGTM analysis: JavaScript No alert changes
Details
LGTM analysis: Python No alert changes
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment