Skip to content

PostgreSQL 18-minimal PoC#653

Open
phracek wants to merge 5 commits intomasterfrom
minimalization
Open

PostgreSQL 18-minimal PoC#653
phracek wants to merge 5 commits intomasterfrom
minimalization

Conversation

@phracek
Copy link
Copy Markdown
Member

@phracek phracek commented Apr 16, 2026 •
edited by github-actions Bot
Loading

This pull request adds PostgreSQL 18-minimal image.

  • CentOS Stream 11 image use as based image centos-stream-10 base image
  • RHEL11 image use as based image ubi10-minimal image

No s2i-core image is used.

  • Added script cgroup_limits
  • Added fix_permissions script
  • Added rpm package for envsubst command, It can be remove in case we find a way in common.sh script
  • Added test_ocp_container_size.py test for comparing compressed and uncompressed images.

This pull request is blocked by #640 , sclorg/container-common-scripts#424, and sclorg/container-common-scripts#423

The fixing of tests were not covered by this pull request yet.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 16, 2026

Pull Request validation

Failed

🔴 Review - Missing review from a member (1 required)

Success

🟢 CI - All checks have passed

@phracek phracek mentioned this pull request Apr 16, 2026
Closed
@phracek
Copy link
Copy Markdown
Member Author

phracek commented Apr 21, 2026

The built images have this sizes

$ podman images
REPOSITORY                                     TAG               IMAGE ID      CREATED        SIZE
localhost/rhel11/postgresql-18-minimal         18-minimal        481709a1f188  4 minutes ago  368 MB
registry.stage.redhat.io/ubi10-minimal         latest            b040a758792c  33 hours ago   87.4 MB
registry.stage.redhat.io/rhel10/postgresql-18  latest            cceeb123e158  4 days ago     423 MB

As you can see postgresql-18 image has size 423 MB based on s2i-core image, whereas postgresql-18-minimal based on ubi10-minimal image has size 368MB.
The question is we need to have in postgresql-18-minimal image postgresql18-upgrade package that adds around 100MB

@phracek
Copy link
Copy Markdown
Member Author

phracek commented Apr 21, 2026 •
edited
Loading

Here is a reference from Dockerhub https://github.com/docker-library/postgres/blob/master/Dockerfile-debian.template

The question is if debian provides upgrade as well. FYI @fila43

@phracek
Copy link
Copy Markdown
Member Author

phracek commented Apr 21, 2026

As this is a minimal image, then postgresql18-upgrade does not make sence, though.

@phracek phracek requested a review from fila43 April 21, 2026 11:02
phracek added 5 commits April 21, 2026 13:04
@phracek
Add dist-gen source for support C11S and RHEL11 minimal
156e16a 
18-minimal version is added.

Base images are ubi10-minimal and centos-stream-10 till
base images for C11S and RHEL11 are not available

Added also files `cgroup-limits` and `fix-permission`
that are missing from s2i-core base image

Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@phracek
Add dist-gen generated sources when minimal image
2d4ba52 
is not based on s2i-core

Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@phracek
Add missing gettext for now to use envsubst.
f078f2f 
Also do not test upgrades and migrations on 18-minimal image

Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@phracek
First PoC of PostgreSQL image minimalization.
3e95345 
Building of PostgreSQL 18-minimal image is based on centos-stream-10 image.

Building of PostgreSQL 18-minimal image is based on ubi10-minimal image.

No s2i-core image is used for building 18-minimal image.

Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@phracek
Minimal image does not need any 'upgrade' package.
2a2b20b 
This can be done by other tooling or by customer image

Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@phracek
Copy link
Copy Markdown
Member Author

phracek commented Apr 23, 2026

As the minimal package should be really minimal I have more improvements.

In case we remove package postgresql18-upgrade then the sizes are:

$ podman images
REPOSITORY                               TAG               IMAGE ID      CREATED        SIZE
localhost/rhel11/postgresql-18-minimal   20260421-2a2b20b  a38818742bc4  5 minutes ago  314 MB
localhost/rhel11/postgresql-18-minimal   latest            a38818742bc4  5 minutes ago  314 MB
localhost/rhel11/postgresql-18-minimal   18-minimal        a38818742bc4  5 minutes ago  314 MB
localhost/rhel11/postgresql-18-minimal   rhel11            a38818742bc4  5 minutes ago  314 MB
localhost/rhel10/postgresql-18           20260421-2a2b20b  0c47144137a1  7 minutes ago  439 MB
localhost/rhel10/postgresql-18           latest            0c47144137a1  7 minutes ago  439 MB
localhost/rhel10/postgresql-18           18                0c47144137a1  7 minutes ago  439 MB
localhost/rhel10/postgresql-18           rhel10            0c47144137a1  7 minutes ago  439 MB
registry.stage.redhat.io/ubi10-minimal   latest            6c064da8b274  4 hours ago    87.9 MB
registry.stage.redhat.io/ubi10/s2i-core  latest            0fcf48f38479  20 hours ago   253 MB

In case we remove postgresql18-upgrade and postgresql18-contrib package then sizes are:

$  podman images
REPOSITORY                               TAG               IMAGE ID      CREATED         SIZE
localhost/rhel11/postgresql-18-minimal   20260421-2a2b20b  0ba767a31ace  5 seconds ago   231 MB
localhost/rhel11/postgresql-18-minimal   latest            0ba767a31ace  5 seconds ago   231 MB
localhost/rhel11/postgresql-18-minimal   18-minimal        0ba767a31ace  5 seconds ago   231 MB
localhost/rhel11/postgresql-18-minimal   rhel11            0ba767a31ace  5 seconds ago   231 MB
<none>                                   <none>            a38818742bc4  13 minutes ago  314 MB
localhost/rhel10/postgresql-18           20260421-2a2b20b  0c47144137a1  15 minutes ago  439 MB
localhost/rhel10/postgresql-18           latest            0c47144137a1  15 minutes ago  439 MB
localhost/rhel10/postgresql-18           18                0c47144137a1  15 minutes ago  439 MB
localhost/rhel10/postgresql-18           rhel10            0c47144137a1  15 minutes ago  439 MB
registry.stage.redhat.io/ubi10-minimal   latest            6c064da8b274  4 hours ago     87.9 MB
registry.stage.redhat.io/ubi10/s2i-core  latest            0fcf48f38479  20 hours ago    253 MB

@fila43
Copy link
Copy Markdown
Member

fila43 commented Apr 28, 2026

From my perspective it doesn't make sense to remove upgrade functionality from the image. Because it was one form the most required feature to fix. Even our CUs contributed on it. It would also change the behavior and feature set of the image. So I disagree with functionality removal.

But I understand the effort to reduce the size. So I adjusted RPM and prepare specific server subpackage postgresql-server-container for container images and separated perl dependencies to the separate package postgresql-upgrade-plperl. It allewd me to reduce the size of container to 312 MB.

For me it seems to be a good trade of. Feel free to experiment with https://src.fedoraproject.org/rpms/postgresql18/pull-request/14

If it makes sense we can get started with reviews of RPM changes and prepare the container build.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fila43 fila43 Awaiting requested review from fila43

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

pr/missing-review ready for review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@phracek @fila43