-
Notifications
You must be signed in to change notification settings - Fork 14
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
NETOBSERV-1287: rtt blog updates to reflect last changes in ebpf agent #63
Conversation
1c04b32
to
1e1f1bd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few comments and suggestions :) looking really good, team!
blogs/rtt/index.md
Outdated
an initial RTT of `10usec` is assigned. | ||
This initial value for RTT may be considered quite low. | ||
|
||
Upon triggering the eBPF (Extended Berkeley Packet Filter) socket, the flow's RTT |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Upon triggering the eBPF (Extended Berkeley Packet Filter) socket, the flow's RTT | |
Upon triggering the eBPF (Extended Berkeley Packet Filter) socket, the RTT flow |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
instead of making flow possessive, I wonder if we can flip it to "RTT flow value"?
1e1f1bd
to
799d887
Compare
799d887
to
8827e99
Compare
Signed-off-by: Mohamed Mahmoud <mmahmoud@redhat.com>
8827e99
to
5146e5b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, feel free to merge !
Thanks @msherif1234
additionl updates to RTT blog follow up for #54