This patch series adds DNS query statistics tracking to ovn-controller
using OVS coverage counters.
Coverage counters provide a standardized mechanism for tracking events
in OVS/OVN and can be queried using standard coverage commands:
- coverage/show: Display all counters
- coverage/read-counter <name>: Read specific counter value
The implementation tracks:
- Total queries and query types (A, AAAA, PTR, ANY, Other)
- Cache performance (hits and misses)
- DNS responses sent
- Error conditions (truncated packets, parse failures, etc.)
This approach leverages the existing OVS coverage infrastructure rather
than implementing a custom statistics mechanism.
v5:
- New implementation using coverage counters instead of custom
dns_statistics struct and dns/show-stats command
- Comprehensive test coverage with OVS_WAIT_UNTIL for reliable
coverage counter verification
- Tests focus solely on coverage counter verification
- Removed dns/show-stats command and related code
- Uses standard OVS coverage infrastructure (coverage/show,
coverage/read-counter)
- Shortened subject lines (patch 1 has minor 4-char overage due to
descriptive clarity)
v4:
- Fixed trailing whitespace in tests/ovn-dns-stats.at line 402
v3:
- Fixed checkpatch warnings: added periods to subject lines
- Fixed trailing whitespace in pinctrl.c
- Fixed line length violation (split long line to fit 79 char limit)
v2:
- Rebased on current upstream main
- Removed unrelated changes from v1
- Fixed test output format expectations
Ketan Supanekar (2):
controller: Add DNS statistics with coverage counters.
tests: Add tests for DNS statistics with coverage.
controller/pinctrl.c | 42 ++++
tests/automake.mk | 3 +-
tests/ovn-dns-stats.at | 497 +++++++++++++++++++++++++++++++++++++++++
tests/testsuite.at | 1 +
4 files changed, 542 insertions(+), 1 deletion(-)
create mode 100644 tests/ovn-dns-stats.at
--
2.52.0
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
