[PR #3150] [MERGED] fix: Add healthcheck for hoppscotch-db in docker-compose, and more #4254

Closed
opened 2026-03-17 01:48:22 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/hoppscotch/hoppscotch/pull/3150
Author: @NicklasWallgren
Created: 6/18/2023
Status: Merged
Merged: 7/17/2023
Merged by: @AndrewBastin

Base: release/2023.4.8Head: run-migration-on-boot


📝 Commits (5)

  • 0f851b2 Run database migration on docker-compose up.
  • 3eacfb3 Re-add the hoppscotch-backend volume and add comment.
  • d39d4b5 Decrease the healthcheck internal
  • cc1a666 Do not run migration when starting hoppscotch-backend
  • 3f6b691 Add additional options to pg_isready

📊 Changes

1 file changed (+13 additions, -3 deletions)

View changed files

📝 docker-compose.yml (+13 -3)

📄 Description

Run database migration on docker-compose up.
Add docker-compose healthcheck for hoppscotch-backend.
Remove volume to fix MODULE_NOT_FOUND error.

Closes #2999

Description

  1. Applies the database migration after the database is ready, and before the hoppscotch-backend application starts.
  2. Removes unnecessary volumes which results in MODULE_NOT_FOUND and/or permission errors.

Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/hoppscotch/hoppscotch/pull/3150 **Author:** [@NicklasWallgren](https://github.com/NicklasWallgren) **Created:** 6/18/2023 **Status:** ✅ Merged **Merged:** 7/17/2023 **Merged by:** [@AndrewBastin](https://github.com/AndrewBastin) **Base:** `release/2023.4.8` ← **Head:** `run-migration-on-boot` --- ### 📝 Commits (5) - [`0f851b2`](https://github.com/hoppscotch/hoppscotch/commit/0f851b21ddea3b88a50d9c02d22268511774e903) Run database migration on docker-compose up. - [`3eacfb3`](https://github.com/hoppscotch/hoppscotch/commit/3eacfb3e9c98dbe5ca7ed96b0f3a765bdf82af1b) Re-add the hoppscotch-backend volume and add comment. - [`d39d4b5`](https://github.com/hoppscotch/hoppscotch/commit/d39d4b532878c5ce3aae2974adca72bc6783c45f) Decrease the healthcheck internal - [`cc1a666`](https://github.com/hoppscotch/hoppscotch/commit/cc1a6666424406aa057f25630ead902fd99271cf) Do not run migration when starting hoppscotch-backend - [`3f6b691`](https://github.com/hoppscotch/hoppscotch/commit/3f6b691cb5ddbcfeeaab519b8e0b6b76c663bb60) Add additional options to pg_isready ### 📊 Changes **1 file changed** (+13 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `docker-compose.yml` (+13 -3) </details> ### 📄 Description Run database migration on docker-compose up. Add docker-compose healthcheck for hoppscotch-backend. Remove volume to fix `MODULE_NOT_FOUND` error. Closes #2999 ### Description 1. Applies the database migration after the database is ready, and before the `hoppscotch-backend` application starts. 2. Removes unnecessary volumes which results in `MODULE_NOT_FOUND` and/or permission errors. ### Checks <!-- Make sure your pull request passes the CI checks and do check the following fields as needed - --> - [X] My pull request adheres to the code style of this project - [ ] My code requires changes to the documentation - [ ] I have updated the documentation as required - [ ] All the tests have passed --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-17 01:48:22 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/hoppscotch#4254
No description provided.