Skip to content

SQL exporter kopf handling - #934

Draft
goat-ssh wants to merge 4 commits into
masterfrom
s/sql-exporter-kopf-handling
Draft

goat-ssh wants to merge 4 commits into
masterfrom
s/sql-exporter-kopf-handling

Conversation

@goat-ssh

@goat-ssh goat-ssh commented Nov 13, 2026 •

Copy link
Copy Markdown
Contributor

Summary of changes

On top of #931

SQL exporter image and configuration were only written when a cluster was created, so version upgrades were manual action.

With new env var SQL_EXPORTER_AUTO_UPDATE enabled, a resume handler updates the
ConfigMap, restarts only the sql-exporter container one pod at a time, and then patches the StatefulSets.
An image that does not start is reverted on the first pod and not retried until the operator restarts.

🤖

Checklist

  • Link to issue this PR refers to:
  • Relevant changes are reflected in CHANGES.rst
  • Added or changed code is covered by tests
  • Documentation has been updated if necessary
  • Changed code does not contain any breaking changes (or this is a major version change)

goat-ssh and others added 4 commits November 12, 2026 18:24
Their only caller, the ConfigMap resume handler, was removed in 0c7478e.

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
The exporter image and configuration were only written when a cluster was
created, so sql_exporter bumps never reached existing clusters without
manual kubectl work.

With SQL_EXPORTER_AUTO_UPDATE enabled, a resume handler updates the
ConfigMap, restarts only the sql-exporter container one pod at a time, and
then patches the StatefulSets. An image that does not start is reverted on
the first pod and not retried until the operator restarts.

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
@goat-ssh
goat-ssh marked this pull request as draft November 13, 2026 11:28

This branch has not been deployed

No deployments
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.

1 participant