Skip to content

docs: Add MariaDB connection pool configuration guide#133

Closed
jinnabaalu wants to merge 1 commit intolouislam:masterfrom
jinnabaalu:master
Closed

docs: Add MariaDB connection pool configuration guide#133
jinnabaalu wants to merge 1 commit intolouislam:masterfrom
jinnabaalu:master

Conversation

@jinnabaalu
Copy link
Copy Markdown

  • Added workaround for Knex connection pool timeout errors
  • Documented database.js modifications for dynamic pool settings
  • Included Docker Compose setup with environment variables
  • Added performance tuning and troubleshooting sections

- Added workaround for Knex connection pool timeout errors
- Documented database.js modifications for dynamic pool settings
- Included Docker Compose setup with environment variables
- Added performance tuning and troubleshooting sections
@CommanderStorm
Copy link
Copy Markdown
Collaborator

I am not revealing this as it is fairly obviously vibe coded and +1.2k additions are not reviewable

Also suggesting a code modification in the docs is not going to make it past me as that is a very crude bodge.
Documenting crude bodies is not something that I support.

@louislam
Copy link
Copy Markdown
Owner

louislam commented Nov 28, 2025

As mentioned in louislam/uptime-kuma#6426 (comment)

Feel free to add as an unofficial method here, link it to your repo.
https://github.com/louislam/uptime-kuma/wiki/%F0%9F%94%A7-How-to-Install

We don't have time to maintain it officially.

Edit: Oh, wait, vide code is not good. We don't need vide code.

@jinnabaalu
Copy link
Copy Markdown
Author

This is not fair, I am currently using it in my current environment for 450+ monitors. Document is vibe coded, not the docker-compose. I created many days back with mariadb, I was running this in production environment.

I thought many people are asking for the Mysql Support with Max connections. I resolved it, I attached screenshor for your info. This was my prior commit for the uptime-kuma https://github.com/VibhuviOiO/infinite-containers/tree/main/uptime-kuma.

I came with PR for the max connections code modifications. You didn't accept. louislam/uptime-kuma#6426

I came with the README.md for how to use workaround for the mysql / mariad db with max connections this is also got rejected. After you push max connection issues this document can be deleted.

FYI @CommanderStorm @louislam I fixed it in my fork created my own image and deployed.

@louislam
Copy link
Copy Markdown
Owner

Clarify: I also meant the doc only. I believe the compose file is not vibe coded.

As I said before, feel free to add as an unofficial method, link it to your repo, but not add the tutorial/file here, because we don't have time to maintain them and test them.

For louislam/uptime-kuma#6426, I have no choice, because it was partial duplicate of louislam/uptime-kuma#6386

@JulianBorghuis
Copy link
Copy Markdown

@jinnabaalu can you help us changing this for our setup using docker on an QNAP Nas?

Please contact me.

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.

4 participants