manageiq / container-postgresql Goto Github PK

See

I don't believe we use this at all, and this would slim up the image a bit.

@bdunne Thoughts?

cc @jrafanie

From what I can tell, this can't be done in a pure Dockerfile, so we'll have to do that in a bin/build script. As such, I don't know if this can be done automatically with dockerhub builds.

Needed for infrastructure management

The base image used in the Dockerfile centos/postgresql-10-centos7 is not multi-arch and seems to be built from https://github.com/sclorg/postgresql-container/blob/generated/10/Dockerfile which again relies on centos/s2i-core-centos7 which is not multi-arch either.

Looking at alternatives to get this working

Thanks for providing this useful container.

I have a question about two postgre config: It seems you keep the default setting of checkpoint_segments to 5min and checkpoint_completion_target to 0.5.

However, The official document recommends to set checkpoint_segments to at least 10, which improves the performance: "Unless you're running on a very small configuration, you'll almost certainly be better setting this to at least 10, which also allows usefully increasing the completion target."

The document also recommends to set checkpoint_completion_target to 0.9, which is the more useful value and improves the performance: "You can spread those writes out further, lowering the average write overhead, by increasing the checkpoint_completion_target parameter to its useful maximum of 0.9"

Shall we change the default of checkpoint_segments to 10 and checkpoint_completion_target to 0.9 for better performance?
Thanks.

Specifically these:
https://github.com/ManageIQ/manageiq-appliance/blob/master/TEMPLATE/var/opt/rh/rh-postgresql95/lib/pgsql/data/postgresql.conf
https://github.com/ManageIQ/manageiq-appliance/blob/master/COPY/etc/manageiq/postgresql.conf.d/01_miq_overrides.conf

When scaling manageiq past 4 pods we run into the default connection limit fairly quickly, so that at least needs to be updated.

We should move forward to a ubi8 based container, also to align with the other containers. This may unlock #18 as well. When we switch should we switch to a vanilla postgres, or to something like the crunchydata image?

This issue lists Renovate updates and detected dependencies. Read the Dependency Dashboard docs to learn more.

This repository currently has no open or pending branches.

Detected dependencies

• Check this box to trigger a request for Renovate to run again on this repository

We should version this container in some sane way so that the released versions of ManageIQ can reference the correct version of the postgres packages.