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

Do yarn migrate is need for first install after uwazi v1.4? #41

Closed
fititnt opened this issue Dec 1, 2018 · 3 comments
Closed

Do yarn migrate is need for first install after uwazi v1.4? #41

fititnt opened this issue Dec 1, 2018 · 3 comments

Comments

@fititnt
Copy link
Owner

fititnt commented Dec 1, 2018

Pertinent change huridocs/uwazi@fcfc25e

The v1.4 now runs yarn migrate after ./database/blank_state.sh and before yarn reindex.

If this is one yes, a new change will need to be done at the https://github.com/fititnt/uwazi-docker/blob/master/docker-entrypoint.sh. This issue is just to confirm the need.

Edit: also impacts the #40

@fititnt fititnt changed the title Do yarn migrate is need for first install? Do yarn migrate is need for first install after uwazi v1.4? Dec 1, 2018
@txau
Copy link

txau commented Dec 1, 2018

@fititnt in principle the blank state comes already with the latest data structure so there is not need to run 'yarn migrate'. That said, its always safe to run migrate since it keeps track of the data version and won't alter anything if its already up to date.

So its safer to run it, in case a dev forgets to update the blank state after a breaking change, but the odds of that are super low.

@fititnt
Copy link
Owner Author

fititnt commented Dec 1, 2018

Ah ok ok. It's not super critical, but will do it later.

@fititnt
Copy link
Owner Author

fititnt commented Jul 10, 2023

Documented this strategy to do it on dockerized version. It is necessary because the normal run up we do not automate yarn migrate / yarn reindex, and in some cases (no major MongoDB/ElasticSearch database upgrade) this migth be sufficient.

docker compose run -e RUN_YARN_MIGRATE_REINDEX=true --rm uwazi

@fititnt fititnt closed this as completed Jul 10, 2023
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

No branches or pull requests

2 participants