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

feat: added keploy to ebpf tools list #97

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Shashwat79802
Copy link

Added Keploy to eBPF tools list

Copy link
Collaborator

@qmonnet qmonnet left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! But I have some comments.

  • I'm not sure the website for the project is the best resource to link, as it's not apparent at all how eBPF is used in the project. It's not mentioned anywhere. Maybe the GitHub repo would make more sense here, even though it's not really intuitive to understand how eBPF is used exactly.

  • Related to the above - Where are the BPF sources in the repository? I couldn't find them, or any C code in the repo. I found the object files, generated from GPL code from what I understand, so I suppose you release the BPF sources somewhere?

  • The project is not really a tool for working with eBPF programs. The entry would maybe fit better under the Observability section. It's not exactly providing observability per se either, but as far as I understand this would be closer to how Keploy uses eBPF?

  • The description could be more concise. For example, “a backend testing tool” “makes backend tests” - this goes without saying. Another example is that “powerful” brings little useful info here.

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.

None yet

2 participants