Skip to content

Ensure entrypoint waits until Moodle is fully installed#24

Open
derdennis1012 wants to merge 1 commit intomainfrom
fix-entrypoint-wait
Open

Ensure entrypoint waits until Moodle is fully installed#24
derdennis1012 wants to merge 1 commit intomainfrom
fix-entrypoint-wait

Conversation

@derdennis1012
Copy link
Owner

@derdennis1012 derdennis1012 commented Apr 2, 2025

Fixes #23

Add logic to wait for Moodle to be fully installed and healthy before proceeding with the entry point.

  • docker-compose.yml
    • Modify health check to include a check for the admin page to ensure Moodle is fully installed.
  • docker/moodle/scripts/setup.sh
    • Add function is_moodle_installed to check if Moodle is fully installed by querying the database.
    • Add function is_moodle_healthy to check the health status of the Moodle service.
    • Add a loop to wait for Moodle to be fully installed and healthy before proceeding with the setup script.
  • Makefile
    • Modify install-plugins command to wait for Moodle to be fully installed and healthy before proceeding.
    • Modify check-updates command to wait for Moodle to be fully installed and healthy before proceeding.

For more details, open the Copilot Workspace session.

Fixes #23

Add logic to wait for Moodle to be fully installed and healthy before proceeding with the entry point.

* **docker-compose.yml**
  - Modify health check to include a check for the admin page to ensure Moodle is fully installed.
* **docker/moodle/scripts/setup.sh**
  - Add function `is_moodle_installed` to check if Moodle is fully installed by querying the database.
  - Add function `is_moodle_healthy` to check the health status of the Moodle service.
  - Add a loop to wait for Moodle to be fully installed and healthy before proceeding with the setup script.
* **Makefile**
  - Modify `install-plugins` command to wait for Moodle to be fully installed and healthy before proceeding.
  - Modify `check-updates` command to wait for Moodle to be fully installed and healthy before proceeding.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/derdennis1012/moodle-cli/issues/23?shareId=XXXX-XXXX-XXXX-XXXX).
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.

entrypint wait

1 participant