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

healthcheck bug fix and add podman compatibility #68

Merged
merged 2 commits into from
Nov 13, 2024

Conversation

mtcolman
Copy link
Contributor

@mtcolman mtcolman commented Nov 13, 2024

fix FATAL error messages:

2024-11-12 17:08:53.814 UTC [38] LOG:  database system was shut down at 2024-11-12 17:08:53 UTC
2024-11-12 17:08:53.818 UTC [1] LOG:  database system is ready to accept connections
2024-11-12 17:09:06.897 UTC [43] FATAL:  role "postgres" does not exist
2024-11-12 17:09:17.904 UTC [45] FATAL:  role "postgres" does not exist
2024-11-12 17:09:28.931 UTC [48] FATAL:  role "postgres" does not exist

postgres user isn't created because POSTGRES_USER=${POSTGRESQL_AUTH_USERNAME} and POSTGRESQL_AUTH_USERNAME=cbomkit. Therefore need to set user as cbomkit (-U cbomkit) and database as postgres (-d postgres) and then the errors don't occur.

===

Also, updated the makefile to allow use of podman-compose via a parameter.

Signed-off-by: Matt Colman <[email protected]>
Copy link
Member

@n1ckl0sk0rtge n1ckl0sk0rtge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mtcolman thanks for the changes! LGTM

@n1ckl0sk0rtge n1ckl0sk0rtge merged commit 1f59a06 into IBM:main Nov 13, 2024
3 checks passed
@mtcolman mtcolman deleted the healthcheck-and-podman branch November 13, 2024 09:38
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