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

docs: add overview for package features #1164

Merged
merged 2 commits into from
Oct 30, 2023
Merged

Conversation

rgo3
Copy link
Contributor

@rgo3 rgo3 commented Oct 10, 2023

This PR adds an overview page for the concept of feature detection in package features. It explains the differences of the provided API to bpftool, describes the returned error semantics and highlights a few of its shortcomings.

@rgo3 rgo3 requested a review from ti-mo October 10, 2023 13:58
@rgo3 rgo3 force-pushed the docs-features branch 2 times, most recently from 987ddaa to 349cbb9 Compare October 11, 2023 14:55
Copy link
Collaborator

@ti-mo ti-mo left a comment

Choose a reason for hiding this comment

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

I think we should have a short paragraph on each major type of probe, e.g. HaveMapType, HaveMapFlag, HaveProgramType, HaveProgramHelper and the misc helpers.

docs/ebpf/concepts/features.md Outdated Show resolved Hide resolved
docs/ebpf/concepts/features.md Show resolved Hide resolved
docs/ebpf/concepts/features.md Outdated Show resolved Hide resolved
docs/ebpf/concepts/features.md Outdated Show resolved Hide resolved
docs/ebpf/concepts/features.md Outdated Show resolved Hide resolved
docs/ebpf/concepts/features.md Outdated Show resolved Hide resolved
@rgo3
Copy link
Contributor Author

rgo3 commented Oct 18, 2023

What should be included in the additional paragraphs per major type of probe vs what we have in the godocs? I am unsure what information a user would need per API and an example for each is a bit overkill IMO.

rgo3 and others added 2 commits October 30, 2023 13:49
Signed-off-by: Robin Gögge <r.goegge@isovalent.com>
This commit adds an overview page for the concept of feature detection
in package `features`. It explains the differences of the provided API
to `bpftool`, describes the returned error semantics and highlights a
few of its limitations.

Signed-off-by: Robin Gögge <r.goegge@isovalent.com>
Co-authored-by: Timo Beckers <timo@isovalent.com>
@ti-mo ti-mo marked this pull request as ready for review October 30, 2023 13:30
@ti-mo ti-mo merged commit 572c585 into cilium:main Oct 30, 2023
12 of 13 checks passed
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