Skip to content
This repository was archived by the owner on May 9, 2023. It is now read-only.

Changes related to the next Meilisearch release (v1.0.0) #98

Merged
merged 5 commits into from
Feb 6, 2023

Conversation

meili-bot
Copy link
Contributor

This PR gathers the changes related to the next Meilisearch release (v1.0.0) so that this package is ready when the official release is out.

⚠️ This PR should NOT be merged until the next release of Meilisearch (v1.0.0) is out.

This PR is auto-generated for the pre-release week purpose.

Related to this issue: meilisearch/integration-guides#230

@alallema alallema force-pushed the bump-meilisearch-v1.0.0 branch from 2a410d5 to 5d5da0c Compare January 16, 2023 18:30
@alallema alallema marked this pull request as ready for review January 18, 2023 14:02
@alallema alallema requested a review from brunoocasali February 1, 2023 17:09
Environment="MEILI_SERVER_PROVIDER=aws"
Environment="MEILI_DUMP_DIR=/var/opt/meilisearch/dumps"
ExecStart=/usr/bin/meilisearch --db-path /var/lib/meilisearch/data.ms --env development --dump-dir $MEILI_DUMP_DIR
Copy link
Member

Choose a reason for hiding this comment

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

Is it correct to set as "development"?

Copy link
Contributor

@alallema alallema Feb 2, 2023

Choose a reason for hiding this comment

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

By default, it's set as development because we don't set any masterKey.
At the first connexion you will have to choose if you want or no a masterKey and choose the environment

Environment="MEILI_SERVER_PROVIDER=digitalocean"
Environment="MEILI_DUMP_DIR=/var/opt/meilisearch/dumps"
Copy link
Member

Choose a reason for hiding this comment

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

Same here.

Environment="MEILI_SERVER_PROVIDER=aws"
Environment="MEILI_DUMP_DIR=/var/opt/meilisearch/dumps"
Copy link
Member

Choose a reason for hiding this comment

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

Why you sometimes set Environment="MEILI_DUMP_DIR=/var/opt/meilisearch/dumps" and other places have Environment="MEILI_DUMP_DIR=$MEILI_DUMP_DIR"

Copy link
Contributor

@alallema alallema Feb 2, 2023

Choose a reason for hiding this comment

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

This is a really good question! systemd doesn't read the environment variable, so that is why there are defined directly on the file. It should probably be better to use EnvironmentFile= instead of Environment= option and create an env file I didn't take time to do it but I maybe will before v1 if I can.

@alallema alallema merged commit bad15f6 into main Feb 6, 2023
@alallema alallema deleted the bump-meilisearch-v1.0.0 branch February 6, 2023 11:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants