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

dnsdist: Enable XSK in our Noble and Oracular Ubuntu packages #15112

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rgacogne
Copy link
Member

@rgacogne rgacogne commented Feb 3, 2025

Short description

It turns out that libbpf-dev and libxdp-dev are there in the end. Note that you need #15065 to test this PR correctly, otherwise it breaks.

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)

It turns out that libbpf-dev and libxdp-dev are there in the end.
@coveralls
Copy link

Pull Request Test Coverage Report for Build 13116425370

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 60 unchanged lines in 17 files lost coverage.
  • Overall coverage remained the same at 64.7%

Files with Coverage Reduction New Missed Lines %
modules/gpgsqlbackend/gpgsqlbackend.cc 1 88.62%
pdns/dnsdistdist/dnsdist.hh 1 84.18%
pdns/recursordist/sortlist.cc 2 72.94%
pdns/distributor.hh 2 51.86%
pdns/backends/gsql/gsqlbackend.hh 2 97.71%
modules/lmdbbackend/lmdbbackend.cc 2 72.71%
pdns/rcpgenerator.cc 2 90.46%
pdns/ws-auth.cc 2 80.8%
pdns/tcpreceiver.cc 2 66.82%
pdns/recursordist/syncres.cc 3 80.32%
Totals Coverage Status
Change from base Build 13115942971: 0.0%
Covered Lines: 127857
Relevant Lines: 166534

💛 - Coveralls

@rgacogne
Copy link
Member Author

rgacogne commented Feb 4, 2025

Successful test run at https://github.com/rgacogne/pdns/actions/runs/13116131772 (technically it reports a failure because we don't generate hashes for ocular since I added it manually).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants