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) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Sergio Durigan Junior | Needs Fixing | ||
Bryce Harrington | Pending | ||
Canonical Server | Pending | ||
Review via email:
|
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.
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.