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

Amend the logic of ebpf-plugin package suggestion for network-viewer plugin #17085

Merged
merged 2 commits into from Mar 2, 2024

Conversation

tkatsoulas
Copy link
Contributor

@tkatsoulas tkatsoulas commented Mar 2, 2024

Summary

We dont provide ebpf plugin for architectures other than x86_64 i386. We enhance network viewer plugin capabilities with the ebpf-plugin. The previous workflow assumed that in any case, during installation of network viewer plugin we suggest or require the ebpf-plugin. This doesn't work in archs other that (x86_64 i386)

Test Plan
Additional Information
For users: How does this change affect me?

Signed-off-by: Tasos Katsoulas <tasos@netdata.cloud>
@tkatsoulas tkatsoulas requested a review from Ferroin as a code owner March 2, 2024 16:19
@github-actions github-actions bot added the area/packaging Packaging and operating systems support label Mar 2, 2024
@tkatsoulas tkatsoulas changed the title Ammend the logic of ebpf-plugin suggestion for network-viewer plugin Amend the logic of ebpf-plugin suggestion for network-viewer plugin Mar 2, 2024
@tkatsoulas tkatsoulas changed the title Amend the logic of ebpf-plugin suggestion for network-viewer plugin Amend the logic of ebpf-plugin suggestion for network-viewer plugin Mar 2, 2024
@tkatsoulas tkatsoulas changed the title Amend the logic of ebpf-plugin suggestion for network-viewer plugin Amend the logic of ebpf-plugin package suggestion for network-viewer plugin Mar 2, 2024
Copy link
Member

@Ferroin Ferroin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need a similar change in the Debian packaging control file as well.

Signed-off-by: Tasos Katsoulas <tasos@netdata.cloud>
@tkatsoulas tkatsoulas merged commit 08189cc into master Mar 2, 2024
134 of 135 checks passed
@tkatsoulas tkatsoulas deleted the sug-ebf branch March 2, 2024 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/packaging Packaging and operating systems support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants