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

Misc documentation enhancements #3819

Merged
merged 10 commits into from
Aug 1, 2024
Merged

Misc documentation enhancements #3819

merged 10 commits into from
Aug 1, 2024

Conversation

tillprochaska
Copy link
Contributor

This PR contains a few small documentation enhancements as follow-ups to #3064. The changes are mostly informed by bug reports/questions from users and admins. For details, see the commit messages.

…t-file containers

It seems to be a common issue that admins set config options only for some of the containers which leads to problems.
…G` env variable

`ALEPH_TAG` is different from other config options (it cannot be specified in `aleph.env`). Also, changing the Aleph version will often require more than just updating this env variable. I think it’s safer for most users to just set the Aleph version/tag in `docker-compose.yml`, because they don’t have to remember to set `ALEPH_TAG` whenever they run a `docker compose` command.
Starting Aleph before running migrations/creating ES indices can cause errors.
Without specifying any services, Docker Compose will start all of them. The default `docker-compose.yml` includes a `shell` service (which is just a bash session) that will exit immediately. This isn’t a problem per se, but it was confusing some users.
@tillprochaska
Copy link
Contributor Author

tillprochaska commented Aug 1, 2024

Also fix these two errors:

@tillprochaska tillprochaska merged commit aca1d03 into develop Aug 1, 2024
2 checks passed
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.

1 participant