-
Notifications
You must be signed in to change notification settings - Fork 70
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 backlog_wait_time_actual status field #93
Add backlog_wait_time_actual status field #93
Conversation
💚 CLA has been signed |
13fe9c9
to
925b819
Compare
❕ Build Aborted
Expand to view the summary
Build stats
|
That will be a nice metric to have. Thanks @maxenglander We need to ensure this continues to work with older kernels as well. @efd6 , can you look this over please. |
@andrewkroh happy to help out with testing. Let me know what kinds of tests you'd like to see. So far I've tested it on the Vagrant image provided in this repo:
|
After looking more closely at the implementation it should be safe to add the new field. Even the godoc on the GetStatus explains this. |
/test |
@maxenglander Will you please add an entry to the changlelog file in the root of the repo. |
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
|
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.
Apologies for the delay, there is probably a very confused @dan out there.
/test |
Failure appears to be due to renaming master => main (jenkins: |
@Mergifyio rebase |
It might not be watching this repo. In any case can you try rebasing this on |
925b819
to
fb767c5
Compare
The status field backlog_wait_time_actual is available since Linux 5.9. It reports the amount of time the kernel has spent waiting for the backlog to drain with schedule_timeout.
fb767c5
to
cfcd379
Compare
@andrewkroh I re-committed on top of latest |
/test |
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.
Thanks
The status field backlog_wait_time_actual is available since Linux 5.9. It reports the amount of time the kernel has spent waiting for the backlog to drain with schedule_timeout.
The status field backlog_wait_time_actual is available since
Linux 5.9. It reports the amount of time the kernel has spent
waiting for the backlog to drain with schedule_timeout.