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

Prep work for Elasticsearch 8 upgrade #81

Merged
merged 29 commits into from
Aug 4, 2022
Merged

Prep work for Elasticsearch 8 upgrade #81

merged 29 commits into from
Aug 4, 2022

Conversation

mrsweaters
Copy link
Contributor

@mrsweaters mrsweaters commented Jul 29, 2022

Jira issue link: MOFONETNAV-167

Overview

Updating test suite and removing outdated specs in preparation for Elasticsearch 8 support.

  • Use refresh instead of flush where search is expected to be updated
  • Drop support for Ruby 2.5 and 2.6
  • Add support for Ruby 2.7 and 3.0, build against Ruby 3.1
  • Set search_type to query_then_fetch as we no longer support Elasticsearch version < 7.0
  • Set include_type_name when fetching the mapping. Will be removed in the next version to support ES version 8.
  • Index names automatically use the new timestamp format when creating an index.

QA Instructions

See https://github.com/doximity/doximity/pull/39531

@mrsweaters mrsweaters marked this pull request as ready for review August 4, 2022 19:44
@mrsweaters mrsweaters changed the title Test against updated versions of Ruby. Prep work for Elasticsearch 8 upgrade Aug 4, 2022
@mrsweaters mrsweaters requested a review from a team August 4, 2022 19:52
@mrsweaters mrsweaters merged commit bf02d96 into master Aug 4, 2022
@mrsweaters mrsweaters deleted the MOFONETNAV-167 branch August 4, 2022 20:42
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.

4 participants