Skip to content

Small Docker container-related fixes#40

Merged
evanebb merged 7 commits intomainfrom
fixes
Sep 21, 2023
Merged

Small Docker container-related fixes#40
evanebb merged 7 commits intomainfrom
fixes

Conversation

@evanebb
Copy link
Copy Markdown
Owner

@evanebb evanebb commented Sep 20, 2023

  • Add more .dockerignore entries to shrink the build context; previously it would send anything in the bin/ folder too, which can be large.
  • Fix the database name configuration directive being overwritten by the database host value when environment variables are used instead of command-line flags. Oops.
  • Add a 30 second time-out to the database connection check that continuously checks whether the database is reachable instead of just immediately exiting if the initial check fails.
  • Print the configured listening address in the logs when starting the API.
  • In the Docker Compose file, bump the image tag to the latest release, make the Gobble container dependent on the database container and add some friendlier container names.

@evanebb evanebb self-assigned this Sep 20, 2023
@evanebb evanebb marked this pull request as ready for review September 21, 2023 19:43
@evanebb evanebb merged commit f36bb20 into main Sep 21, 2023
@evanebb evanebb deleted the fixes branch September 21, 2023 19:44
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.

1 participant