-
Notifications
You must be signed in to change notification settings - Fork 19
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
mock_uss interaction logging endpoint requires an excessive amount of time #677
Comments
Taking a look at this, as I need to get better acquainted with the NotesWe seem to be saving the interactions using the current time in the file name. When looking up interactions, we iterate over every logged interaction file A simple change would be to save the time of the interaction in the filename, using the timestamp of the request from the ObservationsWe seem to log interactions via a query hook and a |
A first proposal for improving things is suggested in #750 This is not guaranteed to fix things as, depending on the requested time from which we might still need to iterate over many files. If the performance is still not giving satisfaction, we could consider returning the interactions without re-parsing them from the files. |
@BenjaminPelletier did you get a chance to observe the performance of the new version of the mock_uss (containing #750?) If performance has been improved we can close this issue. |
Describe the bug
When querying mock_uss's /interuss_logging/logs, mock_uss takes an excessive amount of time to return results (more than 5 seconds). Because uss_qualifier's timeout is 5 seconds, this causes scenarios relying on this reporting to fail.
To reproduce
Difference from expected behavior
The mock_uss logs endpoint should respond quickly; seems like sub-1s should be easily achievable.
Possible solution
Include filter criteria in file names so that content doesn't need to be loaded. Consider using tracer to obtain this information instead.
Screenshots
System on which behavior was encountered
Google Kubernetes Engine with one e2-standard-4 node
Codebase information
/status
output:Notes from the 2024-08-13 Community Meeting
We want to offer endpoints that are robust and precise: the ideal fix should allow to precisely retrieve events that occured at, or later, than a requested time.
At the same time, we assume that clients will properly deal with synchronizing their local clocks and we should not need to include mitigations for clock skew. (If clock skew should be an issue, this can be handled on the qualifier side)
The text was updated successfully, but these errors were encountered: