Skip to content

[bitnami/moodle] modify readme by non-containerized dbs #80989

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

crvargasm
Copy link

Detail

This PR adds a new section to the documentation titled "Connecting to a Non-Containerized Database", which provides detailed guidance on:

  • The required MySQL user permissions for Moodle to function properly.
  • A technical explanation of potential compatibility issues when using external or cloud-managed MySQL-compatible databases (e.g., Amazon Aurora).
  • Specific problems related to the ROW_FORMAT=DYNAMIC directive included in the default docker-compose.yml used by the Bitnami Moodle container.

🧩 Problem Addressed

The current Bitnami documentation does not mention:

  • The exact privileges the Moodle MySQL user must have.
  • Compatibility concerns when using external databases that are not fully MySQL-compliant.

This omission can result in hard-to-debug issues like schema creation failures, permission errors, or "Row size too large" errors during initial setup or upgrades.

✅ Proposed Solution

  • Add a Markdown section with SQL examples for creating the Moodle database and user with proper permissions.
  • Add warnings about unsupported or partially supported database engines (e.g., Amazon Aurora).
  • Include a technical explanation of how ROW_FORMAT=DYNAMIC interacts with different database engines and why this can lead to installation or runtime issues.

📎 References

🛠️ Notes

  • This does not change any application logic or container configuration.
  • It is intended solely to improve the developer/operator onboarding experience when using non-containerized databases with Bitnami's Moodle image.

@github-actions github-actions bot added moodle triage Triage is needed labels May 10, 2025
@github-actions github-actions bot requested a review from javsalgar May 10, 2025 17:02
@javsalgar javsalgar added verify Execute verification workflow for these changes in-progress labels May 12, 2025
@github-actions github-actions bot removed the triage Triage is needed label May 12, 2025
@github-actions github-actions bot removed the request for review from javsalgar May 12, 2025 06:16
@github-actions github-actions bot requested a review from migruiz4 May 12, 2025 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in-progress moodle verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants