Skip to content

[v18] disable mysql health check#58331

Merged
GavinFrazar merged 2 commits intobranch/v18from
gavinfrazar/v18-disable-mysql-health-check
Aug 26, 2025
Merged

[v18] disable mysql health check#58331
GavinFrazar merged 2 commits intobranch/v18from
gavinfrazar/v18-disable-mysql-health-check

Conversation

@GavinFrazar
Copy link
Copy Markdown
Contributor

Backports #58277 to branch/v18.

Changelog: Disabled MySQL database health checks to avoid MySQL blocking the Teleport Database Service for too many connection errors. MySQL health checks can be re-enabled by setting max_connect_errors on MySQL to its maximum value and setting the environment variable TELEPORT_ENABLE_MYSQL_DB_HEALTH_CHECKS=1 on the Teleport Database Service instance.

MySQL automatically blocks remote clients that fail to handshake.
Every TCP health check is counted as a failed handshake, eventually
leading to the database blocking the Teleport database agent.
MySQL's automatic host blocking feature can be effectively disabled by
setting max_connect_errors to a very large value, but to avoid
triggering service outages we should disable MySQL health checks by
default.
For now, an env var TELEPORT_ENABLE_MYSQL_DB_HEALTH_CHECKS can be set on
the Teleport database service to enable MySQL health checks again.
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Aug 25, 2025

Amplify deployment status

Branch Commit Job ID Status Preview Updated (UTC)
gavinfrazar/v18-disable-mysql-health-check HEAD 1 ✅SUCCEED gavinfrazar-v18-disable-mysql-health-check 2025-08-25 23:18:53

@GavinFrazar GavinFrazar marked this pull request as ready for review August 25, 2025 23:13
@github-actions github-actions bot added database-access Database access related issues and PRs documentation size/sm labels Aug 25, 2025
@GavinFrazar GavinFrazar enabled auto-merge August 25, 2025 23:36
Copy link
Copy Markdown
Contributor

@Tener Tener left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@GavinFrazar GavinFrazar added this pull request to the merge queue Aug 26, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 26, 2025
@GavinFrazar GavinFrazar added this pull request to the merge queue Aug 26, 2025
Merged via the queue into branch/v18 with commit 2959f05 Aug 26, 2025
49 checks passed
@GavinFrazar GavinFrazar deleted the gavinfrazar/v18-disable-mysql-health-check branch August 26, 2025 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport database-access Database access related issues and PRs documentation size/sm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants