Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve container image #60

Merged
merged 11 commits into from
Jul 25, 2023
Merged

Improve container image #60

merged 11 commits into from
Jul 25, 2023

Conversation

bjoernricks
Copy link
Contributor

What

Make the database setup more flexible and less error prone.

Why

Allow to run the postgres cluster with TCP access and avoid printing errors to console.

The postgres hba config file contains the access rules to the postgres
cluster. It will be adjusted in the entry point script in future.
This allows for easier adjustments in future and for easier quoting.
Allow for more flexible configuration of the database parameters.
Always allow access via the unix domain socket and allow to configure
the authentication for the access via tcp via a
POSTGRES_HOST_AUTH_METHOD environment variable. By default users must be
authenticated via password when using tcp and the passwords are created
using the md5 authentication method.
We need to initialize the db without influence from the outside.
Therefore don't allow access to the database via TCP connections for the
moment.
Listen on all IP addresses of the container when starting the final db
cluster. This will allow to access the database via TCP connections from
the host for example when exposing the port 5432.
Avoid errors in the log when trying the create the db user twice.
For TCP access from the outside allow to set a password for the created
database user.
Avoid errors when starting a container with an existing database.
Avoid errors when starting an already existing and set up database.
@bjoernricks bjoernricks requested a review from a team as a code owner July 25, 2023 09:10
@github-actions
Copy link

Conventional Commits Report

Type Number
Changed 8

🚀 Conventional commits found.

@bjoernricks bjoernricks enabled auto-merge (rebase) July 25, 2023 10:27
@bjoernricks bjoernricks merged commit 4f7cb02 into main Jul 25, 2023
7 checks passed
@bjoernricks bjoernricks deleted the improve-container-image branch July 25, 2023 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants