Skip to content

Commit a83d119

Browse files
committed
breaking: remove cert-manager support from helm chart
Right now we have experimental support for Cert Manager installation in the chart. As part of this, we store Cert Manager CRDs in the "crds" folder. If you install the chart with Helm, everything works fine, resources are only installed if they don't exist in the cluster. But if we use Argo, we always install CRDs even if they already exist by overwriting them. Since we can't get around this limitation, we decided to remove Cert Manager support from the chart. We recommend using the clusterctl plugin for local development and testing that retains support for installing Cert Manager. In the case of the chart, users must manage Cert Manager installation themselves.
1 parent 22b3a28 commit a83d119

File tree

5 files changed

+0
-79
lines changed

5 files changed

+0
-79
lines changed

Makefile

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -480,7 +480,6 @@ release-manifests: $(KUSTOMIZE) $(RELEASE_DIR) ## Builds the manifests to publis
480480
release-chart: $(HELM) $(KUSTOMIZE) $(RELEASE_DIR) $(CHART_DIR) $(CHART_PACKAGE_DIR) ## Builds the chart to publish with a release
481481
cp -rf $(ROOT)/hack/charts/cluster-api-operator/. $(CHART_DIR)
482482
$(KUSTOMIZE) build ./config/chart > $(CHART_DIR)/templates/operator-components.yaml
483-
$(ROOT)/hack/inject-cert-manager-chart-version.sh $(CERT_MANAGER_VERSION)
484483
$(HELM) package $(CHART_DIR) --app-version=$(HELM_CHART_TAG) --version=$(HELM_CHART_TAG) --destination=$(CHART_PACKAGE_DIR)
485484

486485
.PHONY: release-staging

hack/charts/cluster-api-operator/Chart.yaml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,3 @@ description: Cluster API Operator
44
type: application
55
version: 0.0.0
66
appVersion: "0.0.0"
7-
dependencies:
8-
- name: cert-manager
9-
version: "0.0.0"
10-
repository: https://charts.jetstack.io
11-
condition: cert-manager.enabled

hack/charts/cluster-api-operator/templates/cert-manager.namespace.yaml

Lines changed: 0 additions & 8 deletions
This file was deleted.

hack/charts/cluster-api-operator/values.yaml

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,4 @@
11
---
2-
# Cert Manager options
3-
# Full list of supported values is available at https://artifacthub.io/packages/helm/cert-manager/cert-manager
4-
cert-manager:
5-
enabled: false
6-
fullnameOverride: "cert-manager"
7-
namespace: "cert-manager"
8-
installCRDs: true
92
# ---
103
# Cluster API provider options
114
core: ""

hack/inject-cert-manager-chart-version.sh

Lines changed: 0 additions & 58 deletions
This file was deleted.

0 commit comments

Comments
 (0)