-
-
Notifications
You must be signed in to change notification settings - Fork 529
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
[log_files] Major fixes for log download #2234
Conversation
be852e2
to
999fba9
Compare
999fba9
to
b257191
Compare
Initially I had updated |
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.
Wow quite some effort here, thanks for that!
It looks good to me, and your graphs are quite convincing 😊.
Signed-off-by: Julian Oes <[email protected]>
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 for the rewrite! I remember adding this one in a hurry.
I've added one commit with a few nitpicks, and things clang-tidy wanted.
I'll merge it as soon as CI sanity is restored.
|
Description
Fixes the
log_files
plugin. It's currently really broken and just barely works. I made a few posts in Discord if you want more information in rambling format https://discord.com/channels/1022170275984457759/1022185801615933460/1209702398650749030Basically the current implementation continuously spams LOG_REQUEST_LIST and LOG_REQUEST_DATA and the logic for handling the incoming data is broken.
Before
This wireshark screenshot should give you a pretty good idea of the problem

After
I rewrote most of the code using the QGC implementation as a reference. It works reliably and much faster now.

Example
I'm working on an open source automatic log upload service for @AlexKlimaj and we will be regularly exercising this code. Check it out if you're interested! https://github.com/ARK-Electronics/logloader