add verbosity to pg connections for the backend app

kumahq / kuma-demo

A demo app to illustrate the capabilities and advantages of Kuma. 🐻

https://kuma.io/

Apache License 2.0

57 stars 54 forks source link

add verbosity to pg connections for the backend app #165

Open bcollard opened 5 months ago

bcollard commented 5 months ago

Signed-off-by: Baptiste Collard 9626829+bcollard@users.noreply.github.com

Add DNS lookup for the PostgreSQL DB and improve logging for the backend nodejs app.

bcollard commented 5 months ago

Thanks for the review. All suggestions were applied.

slonka commented 3 months ago

@bcollard - is this PR ready to be merged?

bcollard commented 3 months ago

Hey! Sorry about the delay! Yes, it's ready for another round of review.

Summary:

pinned the version of docker base images
log DNS details about the Postgres Host
switched to full async CRUD methods
adopted PINO as the underlying logging system
use GHA to create AMD and ARM docker images

use GHA to push images to a docker hub account that should be configured with GHA Secrets:

  - name: Login to Docker Hub
    uses: docker/login-action@v3
    with:
      username: ${{ secrets.DOCKERHUB_USERNAME }}
      password: ${{ secrets.DOCKERHUB_TOKEN }}

slonka commented 2 months ago

If everything works and it's complete ping me and I'll merge it

bcollard commented 2 months ago

The requested changes were applied. Sorry about the messy commit history! Will do more granular PRs next time 🙏 Don't hesitate to squash the commits, if possible on merge action.