6
0
mirror of https://github.com/ChristianLight/tutor.git synced 2025-02-11 13:48:30 +00:00

Switch to docker-compose stop for stopping containers

I do not understand the reason why we should use "rm" instead of "stop".

This fixes podman compatibility:
https://discuss.overhang.io/t/tutor-with-podman-not-working/905
This commit is contained in:
Régis Behmo 2020-09-04 12:07:44 +02:00
parent fd50f3c384
commit e9585bdc80
2 changed files with 2 additions and 1 deletions

View File

@ -4,6 +4,7 @@ Note: Breaking changes between versions are indicated by "💥".
## Unreleased
- [Bugfix] Fix podman compatibility by replacing `docker-compose rm` command by `docker-compose stop` when stopping containers
- [Improvement] Improve plugin data deletion
- [Improvement] Introduce the `OPENEDX_COMMON_VERSION` setting
- [Bugfix] Make it possible to run init jobs without starting the entire platform

View File

@ -96,7 +96,7 @@ def start(context, detach, services):
@click.pass_obj
def stop(context, services):
config = tutor_config.load(context.root)
context.docker_compose(context.root, config, "rm", "--stop", "--force", *services)
context.docker_compose(context.root, config, "stop", *services)
@click.command(