Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 0 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -117,9 +117,6 @@ CRDs are located in `chart/crds/` and are installed automatically. The API group
|-----------|-------------|---------|
| `webhooks.enabled` | Enable or disable all Kubernetes admission webhook resources (ValidatingWebhookConfiguration, webhook Service, TLS Certificate, Issuer, and NetworkPolicy). Also sets the `ENABLE_WEBHOOKS` env var on the controller. | `true` |
| `webhookService.type` | Service type for the webhook endpoint | `ClusterIP` |
| `webhookService.ports[0].port` | Service port | `443` |
| `webhookService.ports[0].targetPort` | Container target port | `9443` |
| `webhookService.ports[0].protocol` | Protocol | `TCP` |

### TLS Certificates (cert-manager)

Expand Down
4 changes: 2 additions & 2 deletions mise.toml
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,6 @@ helm = "4.0.4"
[tasks.setup-helm-plugins]
description = "Install all needed helm plugins needed for development"
run = [
"helm plugin install https://github.com/losisin/helm-values-schema-json.git --verify=false",
"helm plugin install https://github.com/helm-unittest/helm-unittest.git --verify=false",
"helm plugin install https://github.com/losisin/helm-values-schema-json.git --verify=false 2>/dev/null || helm plugin update schema",
"helm plugin install https://github.com/helm-unittest/helm-unittest.git --verify=false 2>/dev/null || helm plugin update unittest",
]
4 changes: 3 additions & 1 deletion templates/webhook-service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,5 +11,7 @@ spec:
control-plane: controller-manager
{{- include "git-hubby.selectorLabels" . | nindent 4 }}
ports:
{{- .Values.webhookService.ports | toYaml | nindent 2 }}
- port: 443
protocol: TCP
targetPort: webhook-server
{{- end }}
2 changes: 1 addition & 1 deletion tests/services_test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ tests:
value: 443
- equal:
path: spec.ports[0].targetPort
value: 9443
value: webhook-server
- equal:
path: spec.selector.control-plane
value: controller-manager
Expand Down
17 changes: 0 additions & 17 deletions values.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -232,23 +232,6 @@
"webhookService": {
"type": "object",
"properties": {
"ports": {
"type": "array",
"items": {
"type": "object",
"properties": {
"port": {
"type": "integer"
},
"protocol": {
"type": "string"
},
"targetPort": {
"type": "integer"
}
}
}
},
"type": {
"type": "string"
}
Expand Down
4 changes: 0 additions & 4 deletions values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -63,10 +63,6 @@ serviceAccount:
create: true
name: ""
webhookService:
ports:
- port: 443
protocol: TCP
targetPort: 9443
type: ClusterIP
servingCert:
# duration: 2160h0m0s
Expand Down