-
Notifications
You must be signed in to change notification settings - Fork 432
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
Add traceectl to tracee #4387
Add traceectl to tracee #4387
Conversation
There's a bunch of commented code inside, is this meant to be WIP or draft? |
You are right to note that out, traceectl is an in work in process stat. The commends you refer to are part of the easy to pick up method I use when I code. |
Hey @ShohamBit , congrats for the new tool :) I did a quick review and have some suggestions. If anything doesn’t align, please let me know! Frontend:
Functionality:
version metrics
stream
Note: I'll revisit this subcommand to test other options after successfully creating a stream. event
|
Hey @rscampos, thank you for your review! I appreciate the feedback and will work on implementing some of the suggested changes. I wanted to clarify a few points regarding the
I'll make sure to address these points and provide a clearer documentation. Thanks again for your valuable input! |
1. What does this PR do?
traceectl
as a new CLI tool for interacting with Tracee.2. How to test it
Compile and run Tracee with the gRPC service configuration:
Then start Tracee with the following command:
Run
traceectl
:This will display the help command. Other supported commands include:
version
metrics
stream
event
This version is more organized, with clearer instructions on testing and usage.