Skip to content

EPMDEDP-16582: feat: replace self-signed cert generation with cert-manager#279

Merged
SergK merged 1 commit into
masterfrom
migrate-webhooks
Apr 15, 2026
Merged

EPMDEDP-16582: feat: replace self-signed cert generation with cert-manager#279
SergK merged 1 commit into
masterfrom
migrate-webhooks

Conversation

@zmotso
Copy link
Copy Markdown
Member

@zmotso zmotso commented Apr 15, 2026

Pull Request Template

Description

Remove the custom TLS certificate lifecycle logic (CertService, PopulateCertificates) that regenerated self-signed certs on every restart. Delegate certificate issuance and CA injection to cert-manager via a Helm-managed Issuer and Certificate, and the cert-manager.io/inject-ca-from annotation on ValidatingWebhookConfiguration.

Drop cluster-scoped RBAC rules that were required solely for cert generation (ValidatingWebhookConfiguration patch, Secrets create/update). Gate all webhook resources behind the new enableWebhooks Helm value.

@zmotso zmotso requested a review from a team as a code owner April 15, 2026 13:55
@zmotso zmotso self-assigned this Apr 15, 2026
…nager

Remove the custom TLS certificate lifecycle logic (CertService,
PopulateCertificates) that regenerated self-signed certs on every
restart. Delegate certificate issuance and CA injection to cert-manager
via a Helm-managed Issuer and Certificate, and the
cert-manager.io/inject-ca-from annotation on ValidatingWebhookConfiguration.

Drop cluster-scoped RBAC rules that were required solely for cert
generation (ValidatingWebhookConfiguration patch, Secrets create/update).
Gate all webhook resources behind the new enableWebhooks Helm value.
@zmotso zmotso force-pushed the migrate-webhooks branch from ad712ca to bb88210 Compare April 15, 2026 14:00
@SergK SergK merged commit 8fed7d9 into master Apr 15, 2026
6 checks passed
@SergK SergK deleted the migrate-webhooks branch April 15, 2026 14:54
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