addl PR review updates

caktus · Feb 25, 2020 · 8732472 · 8732472
1 parent 0a9d9d0
commit 8732472
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/defaults/main.yml b/defaults/main.yml
@@ -7,3 +7,4 @@ k8s_letsencrypt_email: ''
 
 k8s_echotest_state: present
 k8s_echotest_hostname: 'echotest.caktus-built.com'
+k8s_echotest_letsencrypt_issuer: 'letsencrypt-staging'
diff --git a/tasks/main.yml b/tasks/main.yml
@@ -1,7 +1,7 @@
 ---
 
 # cert-manager install process adapted from:
-# https://docs.cert-manager.io/en/latest/getting-started/install/kubernetes.html#installing-with-regular-manifests
+# https://cert-manager.io/docs/installation/kubernetes/#installing-with-regular-manifests
 - name: Create namespace for cert-manager
   k8s:
     context: "{{ k8s_context }}"

diff --git a/templates/echotest.yml b/templates/echotest.yml
@@ -45,7 +45,7 @@ metadata:
   namespace: echoserver
   annotations:
     kubernetes.io/ingress.class: "nginx"
-    certmanager.k8s.io/cluster-issuer: "letsencrypt-production"
+    certmanager.k8s.io/cluster-issuer: "{{ k8s_echotest_letsencrypt_issuer }}"
 spec:
   tls:
   - hosts:
Original file line number	Diff line number	Diff line change
Expand Up		@@ -7,3 +7,4 @@ k8s_letsencrypt_email: ''

		k8s_echotest_state: present
		k8s_echotest_hostname: 'echotest.caktus-built.com'
		k8s_echotest_letsencrypt_issuer: 'letsencrypt-staging'