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.

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

Remove Dnstap in favor of (ShoDoHFlo) dnstap_agent JSON UDP datagram telemetry #10

Open
m3047 opened this issue Apr 18, 2024 · 1 comment
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@m3047
Copy link
Owner

m3047 commented Apr 18, 2024

It will be a couple of months before I do this. This will be a breaking change if you presently consume Dnstap telemetry directly.

It's worth noting at the outset that consuming Dnstap telemetry depends on the ShoDoH framestreams consumer already. This is tightly coupled, whereas the dnstap_agent consumes Dnstap telemetry and emits JSON as UDP datagrams. At the present time Rear View RPZ supports both methods.

This will remove the tight coupling to ShoDoHFlo.

  • Rear View RPZ UDP telemetry consumption is (already) compatible with the output of ShoDoHFlo dnstap_agent.py.
  • This decouples the need for Rear View RPZ to be on the same host as BIND and dnstap_agent.py. (If you have more than one dns server then Rear View RPZ necessarily will be on a different host than at least one of them.)
  • Provides the mechanism for Rear View RPZ to consume the telemetry from all of your caching / recursive servers.
  • Removes the need for cloning ShoDoHFlo in order to run Rear View RPZ when BIND is running on a different host. (Presently required even if unused.)
  • Removes the need for a symlink to ShoDoHFlo or another method to find the shodohflo library module (obviating any related motivation for writing an installer).
  • Will simplify support and package management going forward.

I will write a Telemetry Quickstart prior to removing direct support for Dnstap.

You can run the ShoDoHFlo dnstap_agent today and consume its telemetry; this is the recommended configuration for new deployments. If somebody feels like writing the quickstart for me I will graciously accept the contribution!

@m3047 m3047 pinned this issue Apr 18, 2024
@m3047 m3047 added enhancement New feature or request help wanted Extra attention is needed labels Apr 18, 2024
@m3047 m3047 changed the title Remove Dnstap in favor of (ShoDoHFlo) dns_agent JSON UDP datagram telemetry Remove Dnstap in favor of (ShoDoHFlo) dnstap_agent JSON UDP datagram telemetry Apr 18, 2024
@m3047
Copy link
Owner Author

m3047 commented Dec 9, 2024

This is in smoketest on fwm. So far no smoke, lookin' good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant