Skip to content

[koptan]: add cert-manager and webhook resources to Helm chart - #9

Draft
MahmoudBeshay with Copilot wants to merge 2 commits into
mainfrom
copilot/koptan-add-cert-manager-resources
Draft

[koptan]: add cert-manager and webhook resources to Helm chart#9
MahmoudBeshay with Copilot wants to merge 2 commits into
mainfrom
copilot/koptan-add-cert-manager-resources

Conversation

Copilot AI commented Mar 22, 2026

Copy link
Copy Markdown

The koptan controller introduced validating/mutating webhooks, but the Helm chart lacked the corresponding Kubernetes resources to deploy them.

Changes

New values (values.yaml)

  • webhook.enabled (default: false) — gates all webhook-related resources
  • certManager.enabled (default: false) — gates cert-manager Certificate/Issuer and injects cert-manager.io/inject-ca-from annotations onto webhook configs

New templates

  • webhooks.yamlMutatingWebhookConfiguration + ValidatingWebhookConfiguration covering GoApp, JavaApp, Slipway, Voyage
  • webhook-service.yaml — Service routing port 443 → 9443 for the webhook server
  • certmanager.yaml — self-signed Issuer + Certificate (webhook-server-cert) for TLS termination

Updated deployment.yaml

  • Adds containerPort: 9443 (webhook-server) when webhook.enabled=true
  • Mounts webhook-server-cert Secret at /tmp/k8s-webhook-server/serving-certs

Example minimal values to enable webhooks with cert-manager:

webhook:
  enabled: true
certManager:
  enabled: true
Original prompt

This section details on the original issue you should resolve

<issue_title>[koptan]: add cert-manager resources in helmChart</issue_title>
<issue_description>As title says. As you made the validations and webhooks, that logic introduced new resources, that must be added to the helm chart.

Acceptance Criteria:

  1. Validating/Mutating resources
  2. Cert manager resources</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: MahmoudBeshay <197021227+MahmoudBeshay@users.noreply.github.com>
Agent-Logs-Url: https://github.com/felukka/charts/sessions/ebd1138f-ae5f-452b-8770-6524479f808d
Copilot AI changed the title [WIP] Add cert-manager resources to helm chart [koptan]: add cert-manager and webhook resources to Helm chart Mar 22, 2026
Copilot AI requested a review from MahmoudBeshay March 22, 2026 20:35
@MahmoudBeshay
MahmoudBeshay changed the base branch from main to feat/koptan-cert-manager-webhooks March 31, 2026 20:10
@MahmoudBeshay
MahmoudBeshay force-pushed the feat/koptan-cert-manager-webhooks branch 2 times, most recently from 0ea7eb3 to 8fa80a5 Compare April 1, 2026 06:26
Base automatically changed from feat/koptan-cert-manager-webhooks to main April 1, 2026 17:30
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.

[koptan]: add cert-manager resources in helmChart

2 participants