Skip to content

Comments

Reintroduce eBPF compatibility check script for manual checking before installation#85

Closed
PetrHeinz wants to merge 1 commit intomainfrom
ph/add-ebpf-script
Closed

Reintroduce eBPF compatibility check script for manual checking before installation#85
PetrHeinz wants to merge 1 commit intomainfrom
ph/add-ebpf-script

Conversation

@PetrHeinz
Copy link
Member

This script was previously part of the project, removed in #75, but we're still linking the file from https://betterstack.com/docs/logs/collector/#required-kernel-features

I'm proposing to keep the file in main for the purpose of manual checking. Alternatively, we could change the docs link to the v1.0.40 version of the script:

https://raw.githubusercontent.com/BetterStackHQ/collector/refs/tags/v1.0.40/ebpf.sh

WDYT? Is there any better way to check for the support before running collector?

This script checks system support for eBPF features required by Beyla, including BTF + CO-RE support and eBPF ring buffer.
@PetrHeinz PetrHeinz requested a review from paweljw December 4, 2025 10:00
@paweljw
Copy link
Member

paweljw commented Dec 4, 2025

Thanks @PetrHeinz , I would update the link in the docs. The file is unlikely to change, but I'd like to keep it out of the 1.1.0+ tags. Linking to main is kinda risky in the first place, as seen here :)

@PetrHeinz
Copy link
Member Author

Thanks for mentioning main can be risky - added #86 to make sure this doesn't happen for other scripts in the future.

I've changed to links to mention v1.0.40 directly, closing this one 🙌

@PetrHeinz PetrHeinz closed this Dec 4, 2025
@PetrHeinz PetrHeinz deleted the ph/add-ebpf-script branch December 4, 2025 13:16
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.

2 participants