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

BUG/Enhancement - Elastic-search init container #220

Closed
RegisHubelia opened this issue Oct 16, 2023 · 4 comments
Closed

BUG/Enhancement - Elastic-search init container #220

RegisHubelia opened this issue Oct 16, 2023 · 4 comments
Labels
question Further information is requested

Comments

@RegisHubelia
Copy link

Bug/Enhancement request

  • Currently, when a zammad container start, elasticsearch-init can take a long time to index if you have a lot of data.
  • On top of that, if you want 2 replicas of zammad, the second replica also re-index all the data.
  • it then takes quite some time when upgrading/changing configuration for the app to be available. For us it's about 20 minutes per pod - and we have 2 - so about 40 minutes until app/search works as expected.

What you expected to happen:

  • The init container should check if elastic search indexes is already initialised and contains data. If so, ignore re-indexing.
  • Using redis pub/sub to add/remove items from the indexes - my guess is that it's probably already the case as new tickets gets added in the indexes.
  • Scheduled full re-index on a specified/configured cron schedule?

This way, indexing wouldn't block the app availability, and we'd still be quite comfortable with the accuracy of the indexes.

@MrGeneration MrGeneration added the question Further information is requested label Oct 16, 2023
@MrGeneration
Copy link
Member

This is not a bug but expected behavior.
You may want to check the configuration options you have and adjust your configuration accordingly.

https://github.com/zammad/zammad-helm/blob/main/zammad/values.yaml#L57-L68

@MrGeneration MrGeneration closed this as not planned Won't fix, can't repro, duplicate, stale Oct 16, 2023
@RegisHubelia
Copy link
Author

Ah - thanks, I appreciate the quick response. Should of check the settings, so glad it's already an option.

@alexkorotysh
Copy link
Contributor

@RegisHubelia
https://github.com/zammad/zammad-helm/blob/main/zammad/values.yaml#L66
set reindex = false to skip reindex each restart pod
If you need you can easily reindex from zammad pod

@RegisHubelia
Copy link
Author

Thank you. This worked fine.

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

No branches or pull requests

3 participants