From 7b974ff581a743f6a2d18d2ed2143eaa3bda7890 Mon Sep 17 00:00:00 2001 From: Felix Claessen <30658763+Flix6x@users.noreply.github.com> Date: Thu, 10 Nov 2022 19:31:59 +0100 Subject: [PATCH] Issue 527 some database operations (dump and restore) fail in docker containers (#530) Fix ``flexmeasures db-ops dump`` and ``flexmeasures db-ops restore`` not working in docker containers. * Install postgresql-client so db-ops commands (dump and restore) work in docker container Signed-off-by: F.N. Claessen * changelog entry Signed-off-by: F.N. Claessen Signed-off-by: F.N. Claessen --- Dockerfile | 2 +- documentation/changelog.rst | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Dockerfile b/Dockerfile index a53e0fe80..403b5dcb4 100644 --- a/Dockerfile +++ b/Dockerfile @@ -5,7 +5,7 @@ ENV LC_ALL C.UTF-8 ENV LANG C.UTF-8 # pre-requisites -RUN apt-get update && apt-get install --no-install-recommends -y --upgrade python3 python3-pip git curl gunicorn coinor-cbc && apt-get clean +RUN apt-get update && apt-get install --no-install-recommends -y --upgrade python3 python3-pip git curl gunicorn coinor-cbc postgresql-client && apt-get clean WORKDIR /app # requirements - doing this earlier, so we don't install them each time. Use --no-cache to refresh them. diff --git a/documentation/changelog.rst b/documentation/changelog.rst index f1ee264a5..534a32eb9 100644 --- a/documentation/changelog.rst +++ b/documentation/changelog.rst @@ -25,7 +25,7 @@ Infrastructure / Support * Reduce size of Docker image (from 2GB to 1.4GB) [see `PR #512 `_] * Remove bokeh dependency and obsolete UI views [see `PR #476 `_] -* Fix ``flexmeasures db-ops dump`` and ``flexmeasures db-ops restore`` incorrectly reporting a success when `pg_dump` and `pg_restore` are not installed [see `PR #526 `_] +* Fix ``flexmeasures db-ops dump`` and ``flexmeasures db-ops restore`` not working in docker containers [see `PR #530 `_] and incorrectly reporting a success when `pg_dump` and `pg_restore` are not installed [see `PR #526 `_] * Plugins can save BeliefsSeries, too, instead of just BeliefsDataFrames [see `PR #523 `_]