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

Fedora not supported by header install script #261

Open
tsloughter opened this issue Apr 15, 2024 · 0 comments · May be fixed by #262
Open

Fedora not supported by header install script #261

tsloughter opened this issue Apr 15, 2024 · 0 comments · May be fixed by #262
Labels
enhancement New feature or request

Comments

@tsloughter
Copy link
Member

Is your feature request related to a problem? Please describe.

Sort of. Attempting to try out the kernel collector resulted in a failure because I'm running on Fedora and that isn't supported by https://github.com/open-telemetry/opentelemetry-network/blob/main/collector/kernel/kernel_headers.sh

Describe the solution you'd like

Support for fedora in https://github.com/open-telemetry/opentelemetry-network/blob/main/collector/kernel/kernel_headers.sh. I think this can be done fairly easily since I see it supports dnf based distros already, so I can plan on doing a PR.

Describe alternatives you've considered

Manually installing the kernel headers on the node and configuring the container to use those.

Additional context

Issue is also that I was working off the helm chart (https://github.com/open-telemetry/opentelemetry-helm-charts/blob/main/charts/opentelemetry-ebpf). The docs in https://github.com/open-telemetry/opentelemetry-network/blob/main/docs/kernel-collector.md are clear but I didn't see anything in the chart docs so had asked in Slack.

@tsloughter tsloughter added the enhancement New feature or request label Apr 15, 2024
@tsloughter tsloughter linked a pull request Apr 15, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant