-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Upgrade Elasticsearch to 6.8.3 #6309
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
Conversation
There are several steps involved in upgrading the ES. Here is the full path with steps for Rolling Upgrades for upgrading to 7.4: https://www.elastic.co/guide/en/elasticsearch/reference/7.4/setup-upgrade.html |
I think in Elastic Cloud -- Rolling Upgrades for a cluster are just one click. I am not sure though. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm always 👍 on upgrading our dependencies.
Although, considering the effort that upgrading ES involves, this may not be the case. I would ask some questions before continuing with this path:
- are we talking about Python
elasticsearch
module or the ES server for the rolling updates? - are we using some new nice features from ES 7.x?
- what kind of improvements are we getting in comparison with 6.x?
- what is the version we are currently using in production?
- do we know if 7.x is supported in AWS/Azure/ES or whatever we use in production?
- would the changes done in this PR be compatible with the current version we have running in production?
Reading a bit the upgrading docs, it seems that what is not supported is to use the index from an old version in a newer one. So, can we just spin up an ES 7.x instance and do a full re-index while keep using the previous instance until the full re-index finishes?
I think the original issue was marked as
Accepted
just to stop the stale bot, but it should have been marked asNeeded: design decision
IMO.
We are talking about both. Upgrading the python module as well as the ES server. Rolling upgrades will update the ES server.
No, we are not using any "new" features from ES 7.x. But there are always performance improvement of which few are listed here: https://www.elastic.co/guide/en/elasticsearch/reference/current/release-highlights-7.0.0.html
According to our docs, we are using
This PR upgrades ES to 6.8 and the current latest version is 7.4. So I believe that version 6.8 is available in AWS/Azure/ES. I am not sure though. I believe we are actually using Elastic Cloud.
No. We need to update the ES on the server also.
Upgrading from version 6.8 to 7.4 does not require a reindex. Just a simple rolling upgrade. |
# from elasticsearch_dsl import Document as DSLDocument | ||
# ImportError: cannot import name 'Document' | ||
django-elasticsearch-dsl==0.5.1 # pyup: ignore | ||
elasticsearch>=6.0.0,<7.0.0 # pyup: ignore |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Better to explicit the version. otherwise it will break between version
Related #5620
As per the documentation for upgrading, we first need to upgrade elasticsearch to 6.8 before upgrading to 7.4.
https://www.elastic.co/guide/en/elastic-stack/7.4/upgrading-elastic-stack.html#upgrading-elastic-stack
Related PR in readthedocs-sphinx-search: readthedocs/readthedocs-sphinx-search#44