Update requirements.txt - #120
Merged
Merged
Conversation
mysql-connector-python defaults connect_timeout to None (no timeout at all). CONN_MAX_AGE=300 means most requests reuse an existing connection, but the *first* one -- opened by a freshly forked gunicorn worker, or by the Cron thread's first tick -- has nothing to reuse. If that connection attempt stalls for any reason, the calling thread hangs forever with nothing logged, which matches the current 499s: the container logs show "Cron run: starting" (the first line of Cron.run(), before it touches the DB) and then nothing else at all, ever. This doesn't tell us *why* the connection is stalling, but it stops it from hanging silently -- a stuck attempt will now fail loudly and fast, get logged, and let the worker (or Cron's next tick) recover instead of being wedged permanently. Prime suspect for the underlying stall: mysql-connector-python has no upper-bound version constraint in pyproject.toml (unlike Django's <6.0), and this branch's requirements.txt regen jumped it from 9.7.0 to 26.7.0 in one shot -- by far the largest version delta of anything in that diff, and the one piece that actually talks to the real MariaDB server on every request. Worth pinning down/back if the timeout reveals a connection failure rather than just occasional slowness.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.