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

Don't fill up cluster if there is no manual intervention #67

Closed
fsteeg opened this issue Apr 5, 2016 · 1 comment
Closed

Don't fill up cluster if there is no manual intervention #67

fsteeg opened this issue Apr 5, 2016 · 1 comment
Assignees

Comments

@fsteeg
Copy link
Member

fsteeg commented Apr 5, 2016

New indexes should be switched to production automatically, one old index retained to switch back manually if something is wrong. I seem to remember this was our original setup. Currently it's reversed: new index is switched to production manually if everything is ok. This leads to the cluster filling up and failing if indexes are not switched manually in time (see hbz/nwbib#302, hbz/nwbib#303, hbz/lobid#299).

@dr0i
Copy link
Member

dr0i commented Sep 20, 2016

New indexes should be switched to production automatically

This can be easily done by setting the alias parameter to "resources" instead of setting it to "resources-staging". Although, I wouldn't recommend that for obvious reasons (at least there should be a test proving if the new index is plausible (i.e. more than 66M documents etc)).

This leads to the cluster filling up and failing if indexes are not switched manually in time

No. The underlying cause is a full HD. In https://github.com/hbz/lobid-resources/blob/master/src/main/java/org/lobid/resources/ElasticsearchIndexer.java#L297 it's specified that if there are 3 (or more) indices all old ones will be removed (as long as they have no alias set) so that only 2 indices are left. Thus, normally the HD wouldn't fill up.

A full HD also depends on other things (overflowing logs etc). This was probably the cause of hbz/nwbib#302.

So I would suggest to close this issue.

@dr0i dr0i added review and removed ready labels Sep 20, 2016
@fsteeg fsteeg closed this as completed Sep 22, 2016
@fsteeg fsteeg removed the review label Sep 22, 2016
dr0i added a commit that referenced this issue Jan 11, 2017
Fix "Standard", remove unused "isDescribedBy".
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

2 participants