Skip to content

Conversation

arzzon
Copy link
Contributor

@arzzon arzzon commented Aug 18, 2025

Description: support dynamic TLS certificate reload on update without restart

Changes Proposed in PR:

  1. Introduced a cert file watcher to watch the certificates files for updates.
  2. Ensured latest updated cert is used whenever certificates are rotated.
  3. Enhanced logs specific to webhook.
  4. Added UTs for Webhook.

Fixes: resolves #Github issue id

General Checklist

  • Updated Added functionality/ bug fix in release notes
  • Added examples for new feature
  • Updated the documentation
  • Smoke testing completed

CRD Checklist

  • Updated required CR schema

@arzzon
Copy link
Contributor Author

arzzon commented Aug 18, 2025

/azp run

@arzzon arzzon requested a review from vklohiya August 18, 2025 05:17
Copy link

Azure Pipelines successfully started running 1 pipeline(s), but failed to run 1 pipeline(s).

@arzzon arzzon force-pushed the webhook-cert-sync-no-restart branch 2 times, most recently from ca74201 to a99852f Compare August 18, 2025 05:27
@arzzon
Copy link
Contributor Author

arzzon commented Aug 18, 2025

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s), but failed to run 1 pipeline(s).

Copy link
Contributor

@vklohiya vklohiya left a comment

Choose a reason for hiding this comment

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

LGTM

@arzzon arzzon force-pushed the webhook-cert-sync-no-restart branch from a99852f to 0decf51 Compare August 19, 2025 05:11
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