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

Feat/es mapping updates #296

Open
wants to merge 54 commits into
base: stg
Choose a base branch
from
Open

Feat/es mapping updates #296

wants to merge 54 commits into from

Conversation

alexandre-snr
Copy link
Member

This PR refactors some ES related code: mainly moving everything related to mappings scattered in the code base into a new package.

It also adds versioning of ES index mappings:
a new MySQL keeps track of the template's version used by every index in the ES.

ES updates are only done when the task "update-es-mappings" is run.
This task tries to discover indexes which are not tracked in the Mysql table yet. It also check if there is a new version for existing index' mappings and updates them using the following technique: Changing maping with zero downtime.

This is for review only for now.

giubil and others added 30 commits November 28, 2019 15:19
Merge stg branch into master branch
Created task to ingest with an upper limit on manifest dates
Removed month before for limit ingestion
Merge error logs & lambda mapping fix to master
Merge tagging task and routes
Fixed regex to ignore aws:* in most used tags
Added a README in tagging folder
Fixed Circle CI build status in READMEs
Merge tagging suggestions and onboard tagbot
Merge unused accounts into master
Merge resources route changes to master
Merge new entitlement checks
Merge Elasticsearch URL fix & should-popup route
@alexandre-snr
Copy link
Member Author

This branch needs to be rebased once all other pull requests are merged.

@GabrielRavier
Copy link
Contributor

GabrielRavier commented Oct 5, 2021

Updated it to be based on the latest version of stg (although I did a bunch of merges to make the merging process easier rather than a rebase, but I would hope that's fine).

@GabrielRavier GabrielRavier requested review from cruffinoni and J3suis and removed request for SchneiderNicolas October 5, 2021 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review only Only for review, do not merge yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants