From 94c9c83aa6000bc013846e5adc974f61d5d4a69a Mon Sep 17 00:00:00 2001
From: Jon <jonathon.lockwood@networkedweb.com>
Date: Wed, 4 Dec 2024 04:54:19 +0930
Subject: [PATCH 1/2] feat(kubernetes): dont install cni by default

ref: #34
---
 roles/nfc_kubernetes/defaults/main.yml      |  2 ++
 roles/nfc_kubernetes/tasks/k3s/install.yaml | 10 ++++++++++
 2 files changed, 12 insertions(+)

diff --git a/roles/nfc_kubernetes/defaults/main.yml b/roles/nfc_kubernetes/defaults/main.yml
index fc8cf29..11b2f3f 100644
--- a/roles/nfc_kubernetes/defaults/main.yml
+++ b/roles/nfc_kubernetes/defaults/main.yml
@@ -38,6 +38,8 @@ nfc_role_kubernetes_configure_firewall: true
 
 nfc_role_kubernetes_etcd_enabled: false
 
+nfc_role_kubernetes_install_cni: false
+
 nfc_role_kubernetes_install_olm: false
 
 nfc_role_kubernetes_install_helm: true
diff --git a/roles/nfc_kubernetes/tasks/k3s/install.yaml b/roles/nfc_kubernetes/tasks/k3s/install.yaml
index 5e8dcef..0927323 100644
--- a/roles/nfc_kubernetes/tasks/k3s/install.yaml
+++ b/roles/nfc_kubernetes/tasks/k3s/install.yaml
@@ -17,6 +17,8 @@
   register: file_calico_yaml_metadata
   when: >
     nfc_role_kubernetes_prime | bool
+      and
+    nfc_role_kubernetes_install_cni | default(false) | bool
 
 
 - name: Check for calico Operator deployment manifest
@@ -25,6 +27,8 @@
   register: file_calico_operator_yaml_metadata
   when: >
     nfc_role_kubernetes_prime | bool
+      and
+    nfc_role_kubernetes_install_cni | default(false) | bool
 
 
 - name: Install dependent packages
@@ -523,6 +527,8 @@
           {{
             nfc_role_kubernetes_prime | bool
               and
+            nfc_role_kubernetes_install_cni | default(false) | bool
+              and
             (
               (
                 not file_calico_operator_yaml_metadata.stat.exists
@@ -605,6 +611,8 @@
     nfc_role_kubernetes_prime | bool
       and
     not nfc_role_kubernetes_cluster_upgraded | default(false) | bool
+      and
+    nfc_role_kubernetes_install_cni | default(false) | bool
 
 
 - name: Install MetalLB Operator
@@ -724,6 +732,8 @@
   when: >
     nfc_role_kubernetes_prime | bool
       and
+    nfc_role_kubernetes_install_cni | default(false) | bool
+      and
     kubernetes_config.cluster.networking.encrypt | default(false) | bool
       and
     not nfc_role_kubernetes_cluster_upgraded | default(false) | bool
-- 
2.49.0


From c993a0e3fe11b0d93b75dd87a65a37d89417029c Mon Sep 17 00:00:00 2001
From: Jon <jonathon.lockwood@networkedweb.com>
Date: Thu, 5 Dec 2024 12:15:11 +0930
Subject: [PATCH 2/2] feat(kubernetes): dont install helm controller by default

ref: #34
---
 .../nfc_kubernetes/templates/k3s-config.yaml.j2 | 17 +++++++++++++++--
 1 file changed, 15 insertions(+), 2 deletions(-)

diff --git a/roles/nfc_kubernetes/templates/k3s-config.yaml.j2 b/roles/nfc_kubernetes/templates/k3s-config.yaml.j2
index 765bfee..b0dca9d 100644
--- a/roles/nfc_kubernetes/templates/k3s-config.yaml.j2
+++ b/roles/nfc_kubernetes/templates/k3s-config.yaml.j2
@@ -100,8 +100,7 @@
         })
       -%}
 
-
-    {%- endif -%} 
+    {%- endif -%}
 
     {%- if (
       not nfc_kubernetes_enable_metallb | default(false)
@@ -116,6 +115,20 @@
     {%- endif -%}
 
 
+    {%- if (
+      nfc_role_kubernetes_install_helm | default(false)
+    ) -%}
+
+      {%- set disable = servers_config.disable + [ "helm-controller" ] -%}
+
+      {%
+      set servers_config = servers_config | combine({
+          "disable": disable 
+        })
+      -%}
+
+
+    {%- endif -%} 
 {# Combine Remaining Server Objects #}
 
 {%
-- 
2.49.0