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

stats: track packet check results per ISD-AS and DRKey protocol number #56

Open
wants to merge 9 commits into
base: open-source
Choose a base branch
from

Conversation

fstreun
Copy link
Collaborator

@fstreun fstreun commented Jan 15, 2025

LF packet check results are now tracked for each ISD-AS and DRKey protocol that is listed as a peer in the configuration. The metrics are exposed on the telemetry path lf/ia/stats. Furthermore, lf/ia/stats/list provides a list of all combinations of ISD-AS and DRKey ptorocol numbers that are being tracked.


This change is Reviewable

Fabio Streun added 6 commits January 11, 2025 09:21
Each worker thread has its own counter struct and updates it independently. The telemetry thread can read the counter struct from any worker at any time because we assume that updates (to uint64_t) are atomic. Reading the counter struct should not cause an invalidation of the worker's cache.
Only account them once.
@fstreun fstreun force-pushed the fstreun/stats-per-ia branch from 3318d8a to fb55a99 Compare January 15, 2025 10:06
LF packet check results are now tracked for each ISD-AS and DRKey protocol that is listed as a peer in the configuration separately. The metrics are exposed on the telemetry path `lf/ia/stats`. Furthermore, `lf/ia/stats/list` provides a list of all combinations of ISD-AS and DRKey ptorocol numbers that are being tracked.
@fstreun fstreun force-pushed the fstreun/stats-per-ia branch from fb55a99 to 88b9c1f Compare January 15, 2025 10:13
@fstreun fstreun marked this pull request as ready for review January 18, 2025 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant