Migrate to OpenSearch with Elasticsearch v7 compatibility mode #130
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Docker compose v2.28.1 fails with mapping key "extends" already defined.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The first two commits are related to the fact that some things got broken since the last 2 years:
fix: docker-compose: remove duplicated extends in cacheDocker compose v2.28.1 fails with mapping key "extends" already defined.
fix: docker: pin pipenv version and use pip for setupUsing pip instead of pipenv since pipenv was trying to rebuild the Pipfile.lock while we just want to install the setup
The following commit might be breaking changes and should be tested:
fix: OpenSearch v2 with Elasticsearch v7 compatibility mode bulk URL with _docFor the Zenodo instance of the citation broker, the bulk URL called was
relations.../_doc/_bulkand it was failing. The change here makes the bulk URL calledrelations.../_bulkand fixed our issue.fix: do not enable flask-breadcrumbs in invenio_base.apps to avoid flask-menu errorFor the Zenodo instance of the citation broker, we were getting errors when starting the web API with uwsgi (the stack trace is in the code comment). Removing the entry point on the flask-breadcrumbs fixed our problem.
Release:
release: v1.2.0.dev1We needed a version number to deploy but this could be removed from this pull request if needed.