-
Notifications
You must be signed in to change notification settings - Fork 3
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
Regression Issue: Server Crashing with semtech/mu-search:0.10.0-beta.5
in docker-compose.yml
#72
Comments
UPDATE: when testing this on the docker-dev server the mu-search does not crash. It does output some UPDATE ERRORS regarding max yaml points in the logs 2024-11-06T10:21:19.391924044Z INFO [#1] UPDATE HANDLER -- Persisting update queue to disk (length: 0)
2024-11-06T10:21:19.638663437Z ERROR [#1] UPDATE HANDLER -- Failed to persist update queue to disk
2024-11-06T10:21:19.639079436Z ERROR [#1] UPDATE HANDLER -- org.snakeyaml.engine.v2.scanner.ScannerImpl.fetchMoreTokens(ScannerImpl.java:289): The incoming YAML document exceeds the limit: 3145728 code points. (Java::OrgSnakeyamlEngineV2Exceptions::YamlEngineException) |
Hi on the last comment: On the first issue: we've not been able to reproduce this |
Thanks for the follow-up and for confirming the potential issue with large update queues and the default 3MB YAML storage limit. |
Hi @cedricdcc, Your report triggered some further investigation and we found an issue with beta.5 due to an underlying upgrade. Previously we had no size limit, but the yaml parsing library we use introduced one as a safety measure. We've just created a PR that bumps the default size limit to 20MB on mu-search. I expect this branch to be merged later this week |
@cedricdcc this should be resolved by bumping mu-search to 0.10.0 |
If the issue is resolved, please close the issue to let us know :) |
We are currently running a big test on our docker-dev server. In a couple of days we can be sure if this specific issue has been resolved :) |
This zip file includes the following tools to aid in analyzing the logs and identifying the root cause of the issue:
Out of these logs I have found that the YAML storage size issue has not been resolved. |
@cedricdcc can you confirm you bumped search to |
I can indeed confirm that in our docker-dev stack the mu-search component was updated to 0.10.0 |
This is also confirmed by the logs you provided since they mention exceeding 20MB size. I see it does actually manage to index everything aside from the warnings. So while not ideal, I guess it is manageable at the moment? Obviously we will see how to best resolve this issue, at the very least we can be smarter when trying to write the queue (it now errors because we try to read it before writing it). |
After upgrading the
semtech/mu-search
image from0.10.0-beta.3
to0.10.0-beta.5
in thedocker-compose.yml
file, our previously functioning server setup is now crashing. Specifically, the mu-search container in thebeta.5
version restarts continuously after completing indexing, while thebeta.3
version does not exhibit this issue.Context and Setup:
Problem Details:
beta.5
: Themu-search
container restarts after indexing is complete.beta.3
: Themu-search
container does not restart and remains stable after indexing.mu-search
image upgrade, all other environment variables, volumes, configurations, and settings remain unchanged between the two setups.Expected Behavior:
The server setup should maintain stability post-indexing in
beta.5
, consistent with its behavior inbeta.3
.Steps to Reproduce:
docker-compose.yml
with thesemtech/mu-search:0.10.0-beta.5
image.mu-search
container behavior after indexing.The text was updated successfully, but these errors were encountered: