Skip to content

[v18] keystore: add active health checker (#61962)#62637

Merged
dboslee merged 2 commits intobranch/v18from
david/v18/kms-hc
Jan 14, 2026
Merged

[v18] keystore: add active health checker (#61962)#62637
dboslee merged 2 commits intobranch/v18from
david/v18/kms-hc

Conversation

@dboslee
Copy link
Copy Markdown
Contributor

@dboslee dboslee commented Jan 6, 2026

backport #61962 to v18

the active health checker performs signing requests against CAs.

failed signing requests are reported back to a callback which will trip teleports readiness state.

the health checker watches CA events to ensure that the keys being tested are kept up-to-date.

this can be enabled by configuring

keystore:
  health_check:
    active:
      enabled: true

this config format leaves room for adding alternative health checking methods and additional parameters for the active health checker

changelog: Added support for health checks to monitor cert authority availability and affect Teleport Auth readiness

* keystore: add active health checker

the active health checker performs signing requests against CAs.

failed signing requests are reported back to a callback which will
trip teleports readiness state.

the health checker watches CA events to ensure that the keys being
tested are kept up-to-date.

this can be enabled by configuring

```yaml
keystore:
  health_check:
    active:
      enabled: true

```

this config format leaves room for adding alternative health checking methods
and additional parameters for the active health checker

* improve godocs

* simplify logic

* fix outdated comment

* add file license header

* fix lint

* return curr signer on health failure

* refactor watch loop to improve readability

* fix comments

* add context to deletekey godocs

* move GetTLSSigner closer to similar func def

* use synctest and remove fake clock

* update lint comment

* use lexical sorting to select next key
@dboslee dboslee enabled auto-merge January 14, 2026 20:15
@dboslee dboslee added this pull request to the merge queue Jan 14, 2026
Merged via the queue into branch/v18 with commit ada01ff Jan 14, 2026
39 checks passed
@dboslee dboslee deleted the david/v18/kms-hc branch January 14, 2026 20:35
@doggydogworld doggydogworld mentioned this pull request Jan 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants