Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Commit 3d45adc

Browse files
author
Mathieu Velten
committed
Dockerfile for tests: align comments with current behavior
Signed-off-by: Mathieu Velten <[email protected]>
1 parent 1a1abdd commit 3d45adc

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

docker/complement/Dockerfile

+4-4
Original file line numberDiff line numberDiff line change
@@ -8,16 +8,16 @@
88

99
ARG SYNAPSE_VERSION=latest
1010

11-
# first of all, we create a base image with a postgres server and database,
12-
# which we can copy into the target image. For repeated rebuilds, this is
13-
# much faster than apt installing postgres each time.
11+
# First of all, we copy postgres server from the official postgres image,
12+
# since for repeated rebuilds, this is much faster than apt installing
13+
# postgres each time.
1414
#
1515
# This trick only works because (a) the Synapse image happens to have all the
1616
# shared libraries that postgres wants, (b) we use a postgres image based on
1717
# the same debian version as Synapse's docker image (so the versions of the
1818
# shared libraries match).
1919

20-
# now build the final image, based on the Synapse image.
20+
# We also initialize the database itself at build and not runtime for speed reason.
2121

2222
FROM matrixdotorg/synapse-workers:$SYNAPSE_VERSION
2323
# copy the postgres installation over from the image we built above

0 commit comments

Comments
 (0)