Skip to content

Conversation

@abg
Copy link
Member

@abg abg commented Dec 11, 2025

Feature or Bug Description

This backports the BOSH DNS proxy healthcheck from #85 to the v1.1 support branch.

- Validate proxy dns healthcheck script influences bosh dns behavior and
  that connections to the proxy via the bosh-dns proxy link address
  continue to work.
- Cycle that test across all proxies (presumes 2 proxy nodes)
  to ensure we observe DNS route new connections away from an
  unhealthy proxy.

[TNZ-66259](https://vmw-jira.broadcom.net/browse/TNZ-66259)

Authored-by: Kim Bassett <kim.bassett@broadcom.com>
Runs a new "pingdb" command provided by the switchboard module to
validate the proxy is routing connections to a valid MySQL backend.

[TNZ-66259](https://vmw-jira.broadcom.net/browse/TNZ-66259)

Authored-by: Kim Bassett <kim.bassett@broadcom.com>
pingdb is expected to be colocated with the proxy and connects over
localhost to the local proxy instance on the proxy port and establishes
a MySQL connection without credentials.

If a MySQL server response is detected, the proxy is routing connections
to a valid backend and pingdb succeeds.  Otherwise pingdb fails on any
other error.

Notes:
- Move the switchboard proxy main entrypoint to ./cmd/proxy/
- Adds new ./cmd/pingdb to aid in DNS healthchecks for the proxy
- Adds new internal/testing/docker package to assist in integration
  tests for the new pingdb command

[TNZ-66259](https://vmw-jira.broadcom.net/browse/TNZ-66259)

Authored-by: Kim Bassett <kim.bassett@broadcom.com>
Copy link
Member

@kimago kimago left a comment

Choose a reason for hiding this comment

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

Looks good / same as 1.2.

@abg abg merged commit 6e5c4fa into support/1.1.x Dec 11, 2025
@abg abg deleted the backport/support/1.1.x/feature/dns-proxy-healthcheck-TNZ-66259 branch December 11, 2025 22:48
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.

2 participants