Merge ~ubuntu-docker-images/ubuntu-docker-images/+git/postgresql:13.3-21.10-wip into ~ubuntu-docker-images/ubuntu-docker-images/+git/postgresql:13.1-21.04

Proposed by Athos Ribeiro
Status: Rejected
Rejected by: Athos Ribeiro
Proposed branch: ~ubuntu-docker-images/ubuntu-docker-images/+git/postgresql:13.3-21.10-wip
Merge into: ~ubuntu-docker-images/ubuntu-docker-images/+git/postgresql:13.1-21.04
Diff against target: 79 lines (+8/-8)
4 files modified
Dockerfile (+1/-1)
HACKING.md (+1/-1)
README.md (+4/-4)
data/postgres.yaml (+2/-2)
Reviewer Review Type Date Requested Status
Sergio Durigan Junior Needs Fixing
Bryce Harrington Pending
Canonical Server Pending
Review via email: mp+403515@code.launchpad.net

Description of the change

This change creates the postgresql OCI image for impish.

Just the trivial changes were needed and no regressions were detected when running the unit tests.

Note that this MP is targeting the 13.1-21.04 branch solely for comparing purposes. When accepted, these changes should be pushed into a new 13.3-21.10 branch.

Moreover, although a 13.2-21.04 branch exists, the 13.1 branch is the one set in the current recipe (and that is why this new one is based on the 13.1 branch). Finally, no significant changes were performed in the postgresql upstream Dockerfile from which this one was initially forked in the past, as per the AUTHORS file.

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

Thanks for the MP, Athos.

The changes to the Dockerfile and the doc regeneration are fine, but the branch name should not contain the minor version. It should be "13-21.10" (notice that there are the "12-20.10" and the "13-21.04" branches). Arguably, we should delete the other branches from the repo because they cause confusion. Also notice that you will have to use the 13-21.04 branch as the base for the new 13-21.10 branch.

Having said that, you actually caught a problem in the postgres-21.04 recipe: it's using the wrong branch to build the image. I went ahead and updated the recipe to use the 13-21.04 branch, and also triggered a full rebuild of the image.

review: Needs Fixing
Revision history for this message
Athos Ribeiro (athos-ribeiro) wrote :

Hi Sergio, I am closing this MR then and will file a new one from the proper branch when we re-visit the impish images:)

Thanks for the review!

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

Athos,

Now that we're working on the impish OCI images, would you like to reopen/refile this MP?

Revision history for this message
Athos Ribeiro (athos-ribeiro) wrote :

Sounds good. I will address your comments, check if any further changes are needed and re-open this one

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

On Monday, August 30 2021, Athos Ribeiro wrote:

> Sounds good. I will address your comments, check if any further changes are needed and re-open this one

+1, thanks Athos!

--
Sergio
GPG key ID: E92F D0B3 6B14 F1F4 D8E0 EB2F 106D A1C8 C3CB BF14

Unmerged commits

8365baa... by Athos Ribeiro

Regenerate documentation

Signed-off-by: Athos Ribeiro <email address hidden>

2858808... by Athos Ribeiro

Update to impish

Signed-off-by: Athos Ribeiro <email address hidden>

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/Dockerfile b/Dockerfile
2index 3e0a282..13b1d7a 100644
3--- a/Dockerfile
4+++ b/Dockerfile
5@@ -1,5 +1,5 @@
6 # vim:set ft=dockerfile:
7-FROM ubuntu:hirsute
8+FROM ubuntu:impish
9
10 ENV TZ UTC
11
12diff --git a/HACKING.md b/HACKING.md
13index cdcb3d0..1efa043 100644
14--- a/HACKING.md
15+++ b/HACKING.md
16@@ -1,7 +1,7 @@
17 # Contributing
18
19 In Ubuntu, Postgres is available as a `.deb` package. For this reason,
20-this image was built by installing the Postgres Ubuntu Focal package
21+this image was built by installing the Postgres Ubuntu 21.10 LTS package
22 inside a docker container.
23
24 In order to contribute to the Postgres OCI image, do the following:
25diff --git a/README.md b/README.md
26index a3a9642..231afef 100644
27--- a/README.md
28+++ b/README.md
29@@ -19,7 +19,7 @@ _Tags in italics are not available in ubuntu/postgres but are shown here for com
30
31 | Channel Tag | | | Currently | Architectures |
32 |---|---|---|---|---|
33-| **`12.4-20.04_beta`** &nbsp;&nbsp; | | | Postgres 12.4 on Ubuntu 20.04 LTS | `amd64`, `arm64`, `s390x` |
34+| **`13.3-21.10_beta`** &nbsp;&nbsp; | | | Postgres 13.3 on Ubuntu 21.10 LTS | `amd64`, `arm64`, `s390x` |
35 | _`track_risk`_ |
36
37 Channel tag shows the most stable channel for that track ordered `stable`, `candidate`, `beta`, `edge`. More risky channels are always implicitly available. So if `beta` is listed, you can also pull `edge`. If `candidate` is listed, you can pull `beta` and `edge`. When `stable` is listed, all four are available. Images are guaranteed to progress through the sequence `edge`, `beta`, `candidate` before `stable`.
38@@ -30,7 +30,7 @@ Channel tag shows the most stable channel for that track ordered `stable`, `cand
39 Launch this image locally:
40
41 ```sh
42-docker run -d --name postgres-container -e TZ=UTC -p 30432:5432 -e POSTGRES_PASSWORD=My:s3Cr3t/ ubuntu/postgres:12.4-20.04_beta
43+docker run -d --name postgres-container -e TZ=UTC -p 30432:5432 -e POSTGRES_PASSWORD=My:s3Cr3t/ ubuntu/postgres:13.3-21.10_beta
44 ```
45 Access your PostgreSQL server at `localhost:30432`.
46
47@@ -80,7 +80,7 @@ This image also includes the `psql` client for interactive container use:
48 ```sh
49 $ docker network create postgres-network
50 $ docker network connect postgres-network postgres-container
51-$ docker run -it --rm --network postgres-network ubuntu/postgres:12.4-20.04_beta psql -h postgres-container -U postgres
52+$ docker run -it --rm --network postgres-network ubuntu/postgres:13.3-21.10_beta psql -h postgres-container -U postgres
53 ```
54 The password will be asked and you can enter `My:s3Cr3t/`. Now, you are logged in and can enjoy your new instance.
55
56@@ -90,7 +90,7 @@ Works with any Kubernetes; if you don't have one, we recommend you [install Micr
57
58 Download
59 [postgresql.conf](https://git.launchpad.net/~canonical-server/ubuntu-docker-images/+git/postgresql/plain/examples/config/postgresql.conf) and
60-[postgres-deployment.yml](https://git.launchpad.net/~canonical-server/ubuntu-docker-images/+git/postgresql/plain/examples/postgres-deployment.yml) and set `containers.postgres.image` in `postgres-deployment.yml` to your chosen channel tag (e.g. `ubuntu/postgres:12.4-20.04_beta`), then:
61+[postgres-deployment.yml](https://git.launchpad.net/~canonical-server/ubuntu-docker-images/+git/postgresql/plain/examples/postgres-deployment.yml) and set `containers.postgres.image` in `postgres-deployment.yml` to your chosen channel tag (e.g. `ubuntu/postgres:13.3-21.10_beta`), then:
62
63 ```sh
64 kubectl create configmap postgres-config --from-file=main-config=postgresql.conf
65diff --git a/data/postgres.yaml b/data/postgres.yaml
66index aa807f6..187577d 100644
67--- a/data/postgres.yaml
68+++ b/data/postgres.yaml
69@@ -3,8 +3,8 @@ main: true
70 repo: postgres
71 description: >
72 PostgreSQL is a powerful, open source object-relational database system. It is fully ACID compliant, has full support for foreign keys, joins, views, triggers, and stored procedures (in multiple languages). It includes most SQL:2008 data types, including INTEGER, NUMERIC, BOOLEAN, CHAR, VARCHAR, DATE, INTERVAL, and TIMESTAMP. It also supports storage of binary large objects, including pictures, sounds, or video. It has native programming interfaces for C/C++, Java, .Net, Perl, Python, Ruby, Tcl, ODBC, among others, and exceptional documentation. Read more on the [PostgreSQL website](https://www.postgresql.org/docs/).
73-version: 13.1
74-base: Ubuntu 21.04 LTS
75+version: "13.3"
76+base: Ubuntu 21.10 LTS
77 architectures:
78 - amd64
79 - arm64

Subscribers

People subscribed via source and target branches

to all changes: