From 86738b73e468940024a49a7f45a6353522e3fe68 Mon Sep 17 00:00:00 2001
From: Luca Berneking <luca@berneking.net>
Date: Fri, 11 Oct 2019 15:30:06 +0200
Subject: [PATCH] Remove deprecated arguments from helm chart deployment

Signed-off-by: Luca Berneking <luca@berneking.net>
---
 deploy/charts/cert-manager/README.md                 | 2 --
 deploy/charts/cert-manager/templates/deployment.yaml | 6 ------
 deploy/charts/cert-manager/values.yaml               | 2 --
 3 files changed, 10 deletions(-)

diff --git a/deploy/charts/cert-manager/README.md b/deploy/charts/cert-manager/README.md
index 34166422d..617f3f253 100644
--- a/deploy/charts/cert-manager/README.md
+++ b/deploy/charts/cert-manager/README.md
@@ -97,8 +97,6 @@ The following table lists the configurable parameters of the cert-manager chart
 | `tolerations` | Node tolerations for pod assignment | `[]` |
 | `ingressShim.defaultIssuerName` | Optional default issuer to use for ingress resources |  |
 | `ingressShim.defaultIssuerKind` | Optional default issuer kind to use for ingress resources |  |
-| `ingressShim.defaultACMEChallengeType` | Optional default challenge type to use for ingresses using ACME issuers |  |
-| `ingressShim.defaultACMEDNS01ChallengeProvider` | Optional default DNS01 challenge provider to use for ingresses using ACME issuers with DNS01 |  |
 | `prometheus.enabled` | Enable Prometheus monitoring | `true` |
 | `prometheus.servicemonitor.enabled` | Enable Prometheus Operator ServiceMonitor monitoring | `false` |
 | `prometheus.servicemonitor.namespace` | Define namespace where to deploy the ServiceMonitor resource | (namespace where you are deploying) |
diff --git a/deploy/charts/cert-manager/templates/deployment.yaml b/deploy/charts/cert-manager/templates/deployment.yaml
index 64c9ee84a..bcc1a676b 100644
--- a/deploy/charts/cert-manager/templates/deployment.yaml
+++ b/deploy/charts/cert-manager/templates/deployment.yaml
@@ -78,12 +78,6 @@ spec:
           {{- if .defaultIssuerGroup }}
           - --default-issuer-group={{ .defaultIssuerGroup }}
           {{- end }}
-          {{- if .defaultACMEChallengeType }}
-          - --default-acme-issuer-challenge-type={{ .defaultACMEChallengeType }}
-          {{- end }}
-          {{- if .defaultACMEDNS01ChallengeProvider }}
-          - --default-acme-issuer-dns01-provider-name={{ .defaultACMEDNS01ChallengeProvider }}
-          {{- end }}
           {{- end }}
           - --webhook-namespace=$(POD_NAMESPACE)
           - --webhook-ca-secret={{ include "webhook.rootCACertificate" . }}
diff --git a/deploy/charts/cert-manager/values.yaml b/deploy/charts/cert-manager/values.yaml
index 02aaf5d44..b9075cedb 100644
--- a/deploy/charts/cert-manager/values.yaml
+++ b/deploy/charts/cert-manager/values.yaml
@@ -97,8 +97,6 @@ ingressShim: {}
   # defaultIssuerName: ""
   # defaultIssuerKind: ""
   # defaultIssuerGroup: ""
-  # defaultACMEChallengeType: ""
-  # defaultACMEDNS01ChallengeProvider: ""
 
 prometheus:
   enabled: true