Network Viewer integrated with eBPF #17073
Conversation
thiagoftsm
requested review from
tkatsoulas,
Ancairon,
Ferroin and
vkalintiris
as code owners
github-actions
bot
added
area/packaging
thiagoftsm
force-pushed
the
nv_ebpf
branch
3 times, most recently
from
89d88eb
to
d1028fc
Compare
|
Mentioning this here as discussed in the sync: Because the changes to the network viewer plugin involve new dependencies on code from the kernel-collector and ebpf CO-RE repos, changes in the packaging code will probably be required to ensure that the required eBPF programs are installed independent of whether the user as asked for the eBPF plugin or not. In the interest of simplifying that, it would be appreciated if we could ensure that either the programs used by the network-viewer all have a consistent pattern to their name that is different from what is used for the programs used for the existing eBPF plugin, or better yet that they be handled in a separate sub-directory. |
|
@Ferroin as an initial step, I suggest confirming that this ebpf integration works and is worth using. And only after get busy with packaging. Because I doubt it (at least this initial version (this PR) not tested by anybody) - prev ebpf network function in my opinion is 100% useless. |
thiagoftsm
force-pushed
the
nv_ebpf
branch
7 times, most recently
from
b220c32
to
831c6e6
Compare
thiagoftsm
force-pushed
the
nv_ebpf
branch
2 times, most recently
from
8bd0205
to
cb0968e
Compare
…pointer for LS_STATE
Summary
Fixes #16986 (partially)
More details when PR is ready
Test Plan
Compile branch and run netdata. The plugin
network-viwershould run normally using/procfiles.Stop netdata;
Edit
netdata.confand append to collect using eBPF and managing the collection level:[plugin:network-viewer] # update every = 1 command options = ebpf apps-level 1eBPFsoftware.It is also possible to collect with eBPF showing all task ids, but this could create a chart unreadable and a break of pattern with our
proccollection. To visualize this, change step3for:[plugin:network-viewer] # update every = 1 command options = ebpf apps-level 1 use-pidAdditional Information
This PR was tested on:
For users: How does this change affect me?