Skip to content
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

Race condition loading initial data from ElasticSearch #2415

Open
adrianlzt opened this issue Jun 21, 2022 · 0 comments
Open

Race condition loading initial data from ElasticSearch #2415

adrianlzt opened this issue Jun 21, 2022 · 0 comments

Comments

@adrianlzt
Copy link
Contributor

Today we found a race condition with an ElasticSearch starting at the same time of Skydive.

Skydive was able to start the ES client but then failed to get the data from the live index.

The query function does not allow to return an error, so in case of error only an error log is generated.

I'm not sure how this could be handled. Maybe Skydive should refuse to start if it fails to load the data?
In this case, some work would be needed to handle those errors in the Query function and the callers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant