Merge ~ubuntu-docker-images/ubuntu-docker-images/+git/postgresql:install-gosu-21.04 into ~ubuntu-docker-images/ubuntu-docker-images/+git/postgresql:13-21.04

Proposed by Sergio Durigan Junior
Status: Merged
Merged at revision: a9b1fa4e86cf2238edeb35508ead07036240cd95
Proposed branch: ~ubuntu-docker-images/ubuntu-docker-images/+git/postgresql:install-gosu-21.04
Merge into: ~ubuntu-docker-images/ubuntu-docker-images/+git/postgresql:13-21.04
Diff against target: 12 lines (+1/-0)
1 file modified
Dockerfile (+1/-0)
Reviewer Review Type Date Requested Status
Bryce Harrington Approve
Canonical Server Pending
Review via email: mp+400001@code.launchpad.net

Description of the change

We need gosu because the entrypoint script uses it to restart the postgres service if the container is being started as root and if the postgres service should actually run as its own user.

To post a comment you must log in.
Revision history for this message
Sergio Durigan Junior (sergiodj) wrote :

Ops, sorry, I created the branch using the wrong base; let me refresh it.

Revision history for this message
Sergio Durigan Junior (sergiodj) wrote :

Done.

Revision history for this message
Bryce Harrington (bryce) wrote :

LGTM, +1

Revision history for this message
Bryce Harrington (bryce) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/Dockerfile b/Dockerfile
2index d9ee782..b9133f5 100644
3--- a/Dockerfile
4+++ b/Dockerfile
5@@ -21,6 +21,7 @@ RUN set -eux; \
6 DEBIAN_FRONTEND=noninteractive apt-get full-upgrade -y; \
7 DEBIAN_FRONTEND=noninteractive apt-get install -y --no-install-recommends \
8 locales \
9+ gosu \
10 tzdata; \
11 localedef -i en_US -c -f UTF-8 -A /usr/share/locale/locale.alias en_US.UTF-8;
12 ENV LANG en_US.utf8

Subscribers

People subscribed via source and target branches

to all changes: