refactor(config): use jinja to construct data then pretty print it

this way is better as you don't have to worry about the doc changing unless theres a new key.

!17

templates/k3s-config.yaml.j2

@@ -6,85 +6,200 @@
 # Dont edit this file directly as it will be overwritten.
 #
 
-{% if Kubernetes_Master | default(false) -%}cluster-cidr: "{{ KubernetesPodSubnet }}"
+{%- if inventory_hostname in groups['kubernetes_master'] -%}
+  {%
 
-{% if 
+  set kube_apiserver_arg = [
+        "audit-log-path=/var/lib/rancher/k3s/server/logs/audit.log",
+        "audit-log-maxage=" + kube_apiserver_arg_audit_log_maxage | string,
+        "audit-policy-file=/var/lib/rancher/k3s/server/audit.yaml",
+      ]
+
+
+  -%}
+    {%
+    set servers_config = {
+        "cluster-cidr": KubernetesPodSubnet,
+        "disable": [
+          "traefik"
+        ],
+        "disable-network-policy": true,
+        "etcd-snapshot-retention": kubernetes_etcd_snapshot_retention | int,
+        "etcd-snapshot-schedule-cron":  kubernetes_etcd_snapshot_cron_schedule | string,
+        "flannel-backend": "none",
+        "service-cidr": KubernetesServiceSubnet
+      }
+    -%}
+
+    {%- if 
           kubernetes_config.cluster.domain_name is defined
             and
           kubernetes_config.cluster.domain_name | default('') != ''
     -%}
-        cluster-domain: {{ kubernetes_config.cluster.domain_name }}
-    {%- endif %}
 
-cluster-init: true
+        {%- set servers_config = servers_config | combine({
-disable-network-policy: true
+          "cluster-domain": kubernetes_config.cluster.domain_name
-disable:
+        }) -%}
-  {% if nfc_kubernetes_enable_metallb | default(false) or not nfc_kubernetes_enable_servicelb | default(false) -%}- servicelb{% endif +%}
-  - traefik
-etcd-snapshot-retention: {{ kubernetes_etcd_snapshot_retention | int }}
-etcd-snapshot-schedule-cron: "{{ kubernetes_etcd_snapshot_cron_schedule }}"
-flannel-backend: none
-kube-apiserver-arg:
-  - audit-log-path=/var/lib/rancher/k3s/server/logs/audit.log
-  - audit-log-maxage={{ kube_apiserver_arg_audit_log_maxage }}
-  - audit-policy-file=/var/lib/rancher/k3s/server/audit.yaml
-  # - admission-control-config-file=/var/lib/rancher/k3s/server/psa.yaml
-  {% if kubernetes_config.cluster.oidc.enabled | default(false) | bool -%}
-  - oidc-issuer-url={{ kubernetes_config.cluster.oidc.issuer_url }}
-  - oidc-client-id={{ kubernetes_config.cluster.oidc.client_id }}
-  - oidc-username-claim={{ kubernetes_config.cluster.oidc.username_claim }}
-{% if kubernetes_config.cluster.oidc.oidc_username_prefix | default('') != '' -%}  - oidc-username-prefix={{ kubernetes_config.cluster.oidc.oidc_username_prefix }}{% endif %}
-  - oidc-groups-claim={{ kubernetes_config.cluster.oidc.groups_claim }}
-{% if kubernetes_config.cluster.oidc.groups_prefix | default('') != '' %}  - oidc-groups-prefix={{ kubernetes_config.cluster.oidc.groups_prefix }}{% endif %}
-{% endif %}
-{% endif %}
 
-kubelet-arg:
+    {%- endif -%}
-  - system-reserved=cpu={{ kubelet_arg_system_reserved_cpu }},memory={{ kubelet_arg_system_reserved_memory }},ephemeral-storage={{ kubelet_arg_system_reserved_storage }}
-{% if host_external_ip | default('') %}node-external-ip: "{{ host_external_ip }}"{% endif %}
 
-# node-ip: this needs to be set to internal ip if host has external ip
+    {%- if kubernetes_config.cluster.oidc.enabled | default(false) | bool -%}
 
-node-name: {{ inventory_hostname }}
+      {%- 
+        set kube_apiserver_arg = kube_apiserver_arg + [
+        "oidc-client-id=" + kubernetes_config.cluster.oidc.client_id,
+        "oidc-groups-claim=" + kubernetes_config.cluster.oidc.groups_claim,
+        "oidc-issuer-url=" + kubernetes_config.cluster.oidc.issuer_url,
+        "oidc-username-claim=" + kubernetes_config.cluster.oidc.username_claim
+      ] -%}
 
-{% if 
+      {%- if kubernetes_config.cluster.oidc.oidc_username_prefix | default('') != '' -%}
-  groups[kubernetes_config.cluster.group_name] | default([]) | list | length > 0
+
--%}{% if k3s_installed.rc == 0 -%}
+        {%- set kube_apiserver_arg = kube_apiserver_arg + [
-server: 
+          "oidc-username-prefix=" + kubernetes_config.cluster.oidc.oidc_username_prefix
-  {% for cluster_node in groups[kubernetes_config.cluster.group_name] +%}
+        ] -%}
-  {% if
+
-      cluster_node in groups['kubernetes_master'] 
+      {%- endif -%}
+
+      {%- if kubernetes_config.cluster.oidc.groups_prefix | default('') != '' -%}
+
+        {%- set kube_apiserver_arg = kube_apiserver_arg + [
+          "oidc-groups-prefix=" + kubernetes_config.cluster.oidc.groups_prefix
+        ]
         -%}
-        - https://
+
+      {%- endif -%}
+
+    {%- endif -%}
+
+
+    {%- if (
+      nfc_kubernetes_enable_metallb | default(false) 
+        or 
+      not nfc_kubernetes_enable_servicelb | default(false)
+    ) -%}
+
+      {%- set disable = servers_config.disable + [ "servicelb" ] -%}
+
+      {%
+      set servers_config = servers_config | combine({
+          "disable": disable 
+        })
+      -%}
+
+
+    {%- endif -%} 
+
+    {%- if (
+      not nfc_kubernetes_enable_metallb | default(false)
+        and
+      nfc_kubernetes_enable_servicelb | default(false)
+    ) -%}
+      
+      {%- set servers_config = servers_config | combine({
+        "servicelb-namespace": kubernetes_config.cluster.networking.service_load_balancer_namespace | default('kube-system')
+      }) -%}
+
+    {%- endif -%}
+
+
+{# Combine Remaining Server Objects #}
+
+{%
+  set servers_config = servers_config | combine({
+    "kube-apiserver-arg": kube_apiserver_arg
+  })
+-%}
+
+{%- endif -%}
+{# Eof Server Nodes #}
+
+{# SoF All Nodes #}
+
+{%
+
+  set all_nodes_config = {
+    "kubelet-arg": [
+      "system-reserved=cpu=" + kubelet_arg_system_reserved_cpu  + ",memory=" + kubelet_arg_system_reserved_memory + 
+      ",ephemeral-storage=" + kubelet_arg_system_reserved_storage
+    ],
+    "node-name": inventory_hostname
+  }
+
+-%}
+
+
+{%- if groups[kubernetes_config.cluster.group_name] | default([]) | list | length > 0 -%}
+  
+  {%- if k3s_installed.rc == 0 -%}
+
+    {%- set ns = namespace(server=[]) -%}
+
+    {%- for cluster_node in groups[kubernetes_config.cluster.group_name] -%}
+
+      {%- if cluster_node in groups['kubernetes_master'] -%}
+
         {%- if hostvars[cluster_node].host_external_ip is defined -%}
 
-          {{ hostvars[cluster_node].host_external_ip }}
+          {%- set server_node = hostvars[cluster_node].host_external_ip -%}
 
         {%- else -%}
 
-          {{ hostvars[cluster_node].ansible_host }}
+          {%- set server_node = hostvars[cluster_node].ansible_host -%}
 
         {%- endif -%}
-        :6443
-    {%- endif %}
 
-  {% endfor +%}
+        {%- set ns.server = (ns.server | default([])) + [
-  {% elif 
+          "https://" + server_node + ":6443"
+        ] -%}
+
+      {%- endif -%}
+
+    {%- endfor -%}
+
+    {%- set all_nodes_config = all_nodes_config | combine({
+      "server": ns.server,
+    }) -%}
+
+  {%- elif 
     kubernetes_config.cluster.prime.name != inventory_hostname
       and
     k3s_installed.rc == 1
-  %}
+  -%}
 
-server:
+    {%- set server = (server | default([])) + [
-  - https://{{ hostvars[kubernetes_config.cluster.prime.name].ansible_host }}:6443
+      "https://" + hostvars[kubernetes_config.cluster.prime.name].ansible_host + ":6443"
-  {%- endif %}
+    ] -%}
 
-{%- endif %}
+    {%- set all_nodes_config = all_nodes_config | combine({
+      "server": server,
+    }) -%}
 
-{% if Kubernetes_Master | default(false) | bool -%}
+  {%- endif -%}
-{% if nfc_kubernetes_enable_metallb | default(false) or not nfc_kubernetes_enable_servicelb | default(false) -%}
-  servicelb-namespace: {{ kubernetes_config.cluster.networking.service_load_balancer_namespace | default('kube-system') }}
-{% endif %}
 
-service-cidr: "{{ KubernetesServiceSubnet }}"
+{%- endif -%}
-{% endif %}
+
+
+
+{%- if host_external_ip | default('') -%}
+  
+  {%- set all_nodes_config = all_nodes_config | combine({
+    "node-external-ip": host_external_ip,
+   }) -%}
+
+{%- endif -%}
+
+{# EoF All Nodes #}
+
+
+{%- if inventory_hostname in groups['kubernetes_master'] -%}
+
+  {%- set servers_config = servers_config | combine( all_nodes_config ) -%}
+
+  {{ servers_config | to_nice_yaml(indent=2) }}
+
+{%- else -%}
+
+  {{ all_nodes_config | to_nice_yaml(indent=2) }}
+
+{%- endif -%}