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

Add external ip connections counters for each service #109

Merged
merged 3 commits into from
Nov 14, 2024

Conversation

lnarolski
Copy link
Collaborator

@lnarolski lnarolski commented Oct 30, 2024

For each service line in ebpf-discovery output added externalIPv4_16ClientNets, externalIPv4_24ClientNets and externalIPv6ClientsNets counters.

We want to adjust exposed for public traffic information to the new algorithm. The current implementation is wrong and need to be fixed. The first step is to publish missing data from ebpf to osagent.

@lnarolski lnarolski force-pushed the Add-external-ip-connections-counters-for-each-service branch 3 times, most recently from 96a5e5a to e5c7bea Compare November 5, 2024 15:31
@lnarolski lnarolski force-pushed the Add-external-ip-connections-counters-for-each-service branch 2 times, most recently from 4126294 to 86a5ec6 Compare November 8, 2024 08:58
@lnarolski lnarolski changed the title [WIP] Add external ip connections counters for each service Add external ip connections counters for each service Nov 8, 2024
@lnarolski lnarolski force-pushed the Add-external-ip-connections-counters-for-each-service branch from 86a5ec6 to d8fa245 Compare November 12, 2024 06:35
@lnarolski
Copy link
Collaborator Author

I've duplicated component tests for ebpfdiscoverysrv running with enabled and disabled network counters.

libservice/src/Aggregator.cpp Outdated Show resolved Hide resolved
libservice/headers/service/Service.h Show resolved Hide resolved
ebpfdiscoverysrv/src/main.cpp Show resolved Hide resolved
libservice/headers/service/Service.h Outdated Show resolved Hide resolved
libservice/headers/service/Service.h Outdated Show resolved Hide resolved
libservice/headers/service/Service.h Outdated Show resolved Hide resolved
libservice/headers/service/Service.h Outdated Show resolved Hide resolved
libservice/src/Aggregator.cpp Outdated Show resolved Hide resolved
libservice/src/Aggregator.cpp Outdated Show resolved Hide resolved
libservice/src/Aggregator.cpp Outdated Show resolved Hide resolved
pawsten
pawsten previously approved these changes Nov 14, 2024
@lnarolski lnarolski merged commit 57009a2 into main Nov 14, 2024
4 checks passed
@lnarolski lnarolski deleted the Add-external-ip-connections-counters-for-each-service branch November 14, 2024 13:43
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.

3 participants