Skip to content

Commit

Permalink
refactor!: remove cert-manager references from readme and values
Browse files Browse the repository at this point in the history
Signed-off-by: Fabrizio Sestito <[email protected]>
  • Loading branch information
fabriziosestito committed Jul 24, 2024
1 parent cd87987 commit cdac210
Show file tree
Hide file tree
Showing 5 changed files with 3 additions and 37 deletions.
6 changes: 1 addition & 5 deletions charts/kubewarden-controller/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,21 +11,17 @@ The kubewarden-controller can be deployed using a helm chart.

## Installing the charts

Make sure you have [`cert-manager` installed](https://cert-manager.io/docs/installation/)
and then install the kubewarden-controller chart.

If you want to enable telemetry, you also need to install [OpenTelemetry Operator](https://github.com/open-telemetry/opentelemetry-operator).

For example:
```console
$ kubectl apply -f https://github.com/jetstack/cert-manager/releases/download/v1.5.3/cert-manager.yaml
$ helm repo add kubewarden https://charts.kubewarden.io
$ helm install --create-namespace -n kubewarden kubewarden-crds kubewarden/kubewarden-crds
$ helm install --wait -n kubewarden kubewarden-controller kubewarden/kubewarden-controller
$ helm install --wait -n kubewarden kubewarden-defaults kubewarden/kubewarden-defaults
```

This will install cert-manager, kubewarden-crds, kubewarden-controller, and a
This will install kubewarden-crds, kubewarden-controller, and a
default PolicyServer on the Kubernetes cluster in the default configuration
(which includes self-signed TLS certs).

Expand Down
20 changes: 0 additions & 20 deletions charts/kubewarden-controller/questions.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -114,26 +114,6 @@ questions:
description: |
Number of replicas of the Controller Deployment
group: "Controller HA"
# Certificates:
- variable: tls.source
default: "cert-manager-self-signed"
description: "Source for TLS certificates"
label: TLS certificates source
type: enum
options:
- "cert-manager-self-signed"
- "cert-manager"
show_subquestion_if: "cert-manager"
group: "Certificates"
subquestions:
- variable: "tls.certManagerIssuerName"
type: string
default: ""
label: cert-manager Issuer name
description: |
Name of cert-manager Issuer
group: "Certificates"
show_if: "tls.source=cert-manager"
# Telemetry:
- variable: "telemetry.metrics.enabled"
type: boolean
Expand Down
2 changes: 1 addition & 1 deletion charts/kubewarden-controller/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ spec:
- name: cert
secret:
defaultMode: 420
secretName: webhook-server-cert
secretName: kubewarden-webhook-server-cert
{{- if .Values.podSecurityContext }}
securityContext:
{{ toYaml .Values.podSecurityContext | indent 8 }}
Expand Down
1 change: 1 addition & 0 deletions charts/kubewarden-controller/templates/rbac.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -122,6 +122,7 @@ rules:
- create
- delete
- list
- get
- patch
- watch
- apiGroups:
Expand Down
11 changes: 0 additions & 11 deletions charts/kubewarden-controller/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -113,17 +113,6 @@ podAnnotations: {}
nodeSelector: {}
tolerations: []
affinity: {}
tls:
# source options:
# - "cert-manager-self-signed": Scaffold cert-manager integration, and create
# a self-signed certificate with a cert-manager self-signed Issuer. Depends
# on cert-manager. (default)
# - "cert-manager": Scafffold cert-manager integration. User configures their
# own Issuer. Depends on cert-manager. Set tls.certManagerIssuerName to the
# desired Issuer.
source: cert-manager-self-signed
# "cert-manager"-only options:
certManagerIssuerName: ""
# Resource limits & requests
# Ref: https://kubernetes.io/docs/user-guide/compute-resources/
resources:
Expand Down

0 comments on commit cdac210

Please sign in to comment.