Skip to content

feat(webhook): Improve instrumentation for certificate rotation#1145

Merged
Techassi merged 5 commits intomainfrom
feat/webhook-improve-instrumentation
Feb 3, 2026
Merged

feat(webhook): Improve instrumentation for certificate rotation#1145
Techassi merged 5 commits intomainfrom
feat/webhook-improve-instrumentation

Conversation

@Techassi
Copy link
Member

@Techassi Techassi commented Jan 30, 2026

Based on #1144, needs rebase before merge.

This PR improves the instrumentation of the certificate rotation process/handlers.

INFO stackable_webhook: rotate certificate for registered webhooks x509.not_before="2026-01-30T14:37:03Z" x509.not_after="2026-01-31T14:37:03Z" x509.serial_number="00:D0:CA:F3:A3:DF:C3:79:FE" x509.subject="CN=Leaf Certificate for webhook" x509.issuer="CN=Stackable Data Platform Internal CA"

@Techassi Techassi force-pushed the feat/webhook-improve-instrumentation branch from ce234fc to fd256cf Compare February 3, 2026 09:34
@Techassi Techassi marked this pull request as ready for review February 3, 2026 09:50
@Techassi Techassi moved this from Development: In Progress to Development: Waiting for Review in Stackable Engineering Feb 3, 2026
@maltesander maltesander self-requested a review February 3, 2026 11:19
@maltesander maltesander moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Feb 3, 2026
Copy link
Member

@maltesander maltesander left a comment

Choose a reason for hiding this comment

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

LGTM!

@Techassi Techassi added this pull request to the merge queue Feb 3, 2026
@Techassi Techassi moved this from Development: In Review to Development: Done in Stackable Engineering Feb 3, 2026
Merged via the queue into main with commit caaeb08 Feb 3, 2026
5 checks passed
@Techassi Techassi deleted the feat/webhook-improve-instrumentation branch February 3, 2026 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Development: Done

Development

Successfully merging this pull request may close these issues.

2 participants