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

Upgrade ES to 8.x #10771

Merged
merged 5 commits into from Oct 9, 2023
Merged

Upgrade ES to 8.x #10771

merged 5 commits into from Oct 9, 2023

Conversation

stsewd
Copy link
Member

@stsewd stsewd commented Sep 26, 2023

We need to wait for django-es/django-elasticsearch-dsl#458 or #10730.

There are no breaking changes in the API,
so this can be deployed without any downtime.

The only change is that it now has authentication enabled by default, so our docker image now uses a password.

How to deploy:

  • Deploy web extra
  • The new packages will continue working with the old ES, but we can make sure by searching from web-extra.
  • Finish the deploy
  • Upgrade the ES cluster from cloud.elastic.co

Closes #10496

We need to wait for django-es/django-elasticsearch-dsl#458
or #10730.

There are no breaking changes in the API,
so this can be deployed without any downtime.

The only change is that it now has authentication enabled by default,
so our docker image now uses a password.

How to deploy:

- Deploy web extra
- The new packages will continue working with the old ES,
  but we can make sure by searching from web-extra.
- Finish the deploy
- Upgrade the ES cluster from cloud.elastic.co
@stsewd
Copy link
Member Author

stsewd commented Sep 26, 2023

Hmm, there are some SSL errors, but I don't see those locally, probably something with circle...

@stsewd stsewd marked this pull request as ready for review October 9, 2023 19:27
@stsewd stsewd requested a review from a team as a code owner October 9, 2023 19:27
Copy link
Member

@ericholscher ericholscher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like mostly just a dependency upgrade? Simple enough 👍

I think removing the DSL package is a good path forward, if that's on our roadmap anyway.

@stsewd stsewd merged commit e32e707 into main Oct 9, 2023
7 checks passed
@stsewd stsewd deleted the upgrade-es-8.x branch October 9, 2023 20:44
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

Successfully merging this pull request may close these issues.

Elasticsearch: upgrade to 8.x
2 participants