Installing node exporter and filebeat as daemonsets in custom namespa…

…ces (hitachienergy#1839) * Installing daemonsets in custom namespaces * Helm Chart name added to uninstall role * Upgrade process added * Including vars moved on the top of task * Added more strictly condiction * Obtaining helm releases in different way * Format improved * Kubeconfig env moved to playbook level * Changelog updated Co-authored-by: Robert Pudłowski <[email protected]>
rafzei · Nov 26, 2020 · a97089b · a97089b
1 parent 446a313
commit a97089b
Show file tree

Hide file tree

Showing 10 changed files with 126 additions and 27 deletions.
diff --git a/CHANGELOG-0.9.md b/CHANGELOG-0.9.md
@@ -11,6 +11,7 @@
 - [#1835](https://github.com/epiphany-platform/epiphany/issues/1835) - Automated tests may give false negative result for PGAudit
 - [#1409](https://github.com/epiphany-platform/epiphany/issues/1409) - custom_image_registry_address setting is not implemented
 - [#1280](https://github.com/epiphany-platform/epiphany/issues/1280) - [RHEL] Pgpool not showing Replication State
+- [#1833](https://github.com/epiphany-platform/epiphany/issues/1833) - DaemonSets of Node Exporter and Filebeat deploy in default namespace
 
 ### Updated
 

diff --git a/core/src/epicli/data/common/ansible/playbooks/roles/filebeat/defaults/main.yml b/core/src/epicli/data/common/ansible/playbooks/roles/filebeat/defaults/main.yml
@@ -1,3 +1,5 @@
 ---
 filebeat_helm_chart_file_name: filebeat-7.9.2.tgz
 filebeat_version: "7.9.2"
+# Use custom namespace for logging charts such as filebeat in case of k8s as cloud service.
+logging_chart_namespace: epi-logging
diff --git a/...icli/data/common/ansible/playbooks/roles/filebeat/tasks/install-filebeat-as-daemonset.yml b/...icli/data/common/ansible/playbooks/roles/filebeat/tasks/install-filebeat-as-daemonset.yml
@@ -26,7 +26,7 @@
 
     - name: Install Filebeat using custom Helm chart (custom-chart-values.yml)
       command: |
-        helm upgrade --install \
+        helm -n {{ logging_chart_namespace }} upgrade --install \
           -f {{ download_directory }}/custom-chart-values.yml \
           {{ specification.helm_chart_name }} \
-          {{ download_directory }}/{{ filebeat_helm_chart_file_name }}
+          {{ download_directory }}/{{ filebeat_helm_chart_file_name }} --create-namespace
diff --git a/...li/data/common/ansible/playbooks/roles/filebeat/tasks/uninstall-filebeat-as-daemonset.yml b/...li/data/common/ansible/playbooks/roles/filebeat/tasks/uninstall-filebeat-as-daemonset.yml
@@ -0,0 +1,9 @@
+---
+
+- name: Uninstall Helm release "{{ specification.helm_chart_name }}"
+
+  delegate_to: localhost
+  become: false
+  run_once: true
+
+  command: helm uninstall {{ specification.helm_chart_name }}
diff --git a/core/src/epicli/data/common/ansible/playbooks/roles/node_exporter/defaults/main.yml b/core/src/epicli/data/common/ansible/playbooks/roles/node_exporter/defaults/main.yml
@@ -4,3 +4,5 @@ exporter:
   service:
     description: "Service that runs Prometheus Node Exporter"
     name: prometheus-node-exporter
+# Use custom namespace for monitoring charts such as node exporter in case of k8s as cloud service.
+monitoring_chart_namespace: epi-monitoring
diff --git a/...common/ansible/playbooks/roles/node_exporter/tasks/install-node-exporter-as-daemonset.yml b/...common/ansible/playbooks/roles/node_exporter/tasks/install-node-exporter-as-daemonset.yml
@@ -36,16 +36,16 @@
         - name: Install Node-Exporter's Helm chart (with custom values.yaml)
           delegate_to: localhost
           shell: |
-            helm upgrade --install \
+            helm -n {{ monitoring_chart_namespace }} upgrade --install \
               -f {{ download_directory }}/{{ specification.helm_chart_name }}_values.yaml \
               {{ specification.helm_chart_name }} \
-              {{ download_directory }}/{{ exporter_chart_file_name }}
+              {{ download_directory }}/{{ exporter_chart_file_name }} --create-namespace
 
     - when: not helm_chart_values_bool
     # ELSE
       block:
         - name: Install Node-Exporter's Helm chart (with default values.yaml)
           shell: |
-            helm upgrade --install \
+            helm -n {{ monitoring_chart_namespace }} upgrade --install \
               {{ specification.helm_chart_name }} \
-              {{ download_directory }}/{{ exporter_chart_file_name }}
+              {{ download_directory }}/{{ exporter_chart_file_name }} --create-namespace
diff --git a/...mmon/ansible/playbooks/roles/node_exporter/tasks/uninstall-node-exporter-as-daemonset.yml b/...mmon/ansible/playbooks/roles/node_exporter/tasks/uninstall-node-exporter-as-daemonset.yml
@@ -0,0 +1,9 @@
+---
+
+- name: Uninstall Helm release "{{ specification.helm_chart_name }}"
+
+  delegate_to: localhost
+  become: false
+  run_once: true
+
+  command: helm uninstall {{ specification.helm_chart_name }}
diff --git a/core/src/epicli/data/common/ansible/playbooks/roles/upgrade/tasks/filebeat.yml b/core/src/epicli/data/common/ansible/playbooks/roles/upgrade/tasks/filebeat.yml
@@ -1,26 +1,62 @@
 ---
-- name: Filebeat | Get information about installed packages as facts
+- name: Filebeat as DaemonSet | Prepare fact about helm release
+  delegate_to: localhost
+  become: false
+  run_once: true
+  block:
+    - name: Get information about helm releases
+      command: helm list --output json
+      register: helm_list
+
+    - name: Check if filebeat release exist in default namespace and set fact
+      set_fact:
+        filebeat_release_exists: >-
+          {{ _names | ternary(true, false) }}
+      vars:
+        _names: >-
+          {{ helm_list.stdout | from_json 
+                              | map(attribute='name')
+                              | select('==', specification.helm_chart_name)
+                              | list }}
+
+- name: Reinstall Filebeat as DaemonSet from default namespace to custom namespace for "k8s as cloud service"
+  when:
+    - filebeat_release_exists
+    - k8s_as_cloud_service is defined
+    - k8s_as_cloud_service
+  block: 
+    - name: Include uninstall task for Filebeat as DaemonSet in default namespace for "k8s as cloud service"
+      include_role:
+        name: filebeat
+        tasks_from: uninstall-filebeat-as-daemonset.yml
+
+    - name: Include install task for Filebeat as DaemonSet in custom namespace for "k8s as cloud service"
+      include_role:
+        name: filebeat
+        tasks_from: install-filebeat-as-daemonset.yml
+
+- name: Filebeat as System Service | Get information about installed packages as facts
   package_facts:
     manager: auto
   when: ansible_facts.packages is undefined
 
-- name: Filebeat | Test if filebeat package is installed
+- name: Filebeat as System Service | Test if filebeat package is installed
   assert:
     that: ansible_facts.packages['filebeat'] is defined
     fail_msg: filebeat package not found, nothing to update
     quiet: true
 
-- name: Filebeat | Print versions
+- name: Filebeat as System Service | Print versions
   debug:
     msg:
       - "Installed version: {{ ansible_facts.packages['filebeat'][0].version }}"
       - "Target version: {{ filebeat_version }}"
 
-- name: Update Filebeat
+- name: Update Filebeat as System Service
   when:
     - filebeat_version is version(ansible_facts.packages['filebeat'][0].version, '>=')
   block:
-    - name: Filebeat | Backup configuration file (filebeat.yml)
+    - name: Filebeat as System Service | Backup configuration file (filebeat.yml)
       copy:
         remote_src: true
         src: /etc/filebeat/filebeat.yml

diff --git a/core/src/epicli/data/common/ansible/playbooks/roles/upgrade/tasks/node-exporter.yml b/core/src/epicli/data/common/ansible/playbooks/roles/upgrade/tasks/node-exporter.yml
@@ -1,8 +1,4 @@
 ---
-- name: Node Exporter | Populate service facts
-  service_facts:
-  when: ansible_facts.services is undefined
-
 - name: Node Exporter | Include defaults from node_exporter role
   include_vars:
     file: roles/node_exporter/defaults/main.yml
@@ -11,32 +7,72 @@
   include_vars:
     file: roles/node_exporter/vars/main.yml
 
-- name: Node Exporter | Test if node exporter service is configured
+- name: Node Exporter as DaemonSet | Prepare fact about helm release
+  delegate_to: localhost
+  become: false
+  run_once: true
+  block:
+    - name: Get information about helm releases
+      command: helm list --output json
+      register: helm_list
+
+    - name: Check if node exporter release exist in default namespace and set fact
+      set_fact:
+        node_exporter_release_exists: >-
+          {{ _names | ternary(true, false) }}
+      vars:
+        _names: >-
+          {{ helm_list.stdout | from_json 
+                              | map(attribute='name')
+                              | select('==', specification.helm_chart_name)
+                              | list }}
+
+- name: Reinstall Node Exporter as DaemonSet from default namespace to custom namespace for "k8s as cloud service"
+  when:
+    - node_exporter_release_exists
+    - k8s_as_cloud_service is defined
+    - k8s_as_cloud_service
+  block: 
+    - name: Include uninstall task for Node Exporter as DaemonSet in default namespace for "k8s as cloud service"
+      include_role:
+        name: node_exporter
+        tasks_from: uninstall-node-exporter-as-daemonset.yml
+
+    - name: Include install task for Node Exporter as DaemonSet in custom namespace for "k8s as cloud service"
+      include_role:
+        name: node_exporter
+        tasks_from: install-node-exporter-as-daemonset.yml
+
+- name: Node Exporter as System Service | Populate service facts
+  service_facts:
+  when: ansible_facts.services is undefined
+
+- name: Node Exporter as System Service | Test if node exporter service is configured
   assert:
     that: "'{{ exporter.service.name }}.service' in ansible_facts.services"
     fail_msg: "{{ exporter.service.name }} service is not found"
     quiet: true
 
-- name: Node Exporter | Set exporter_file_name fact
+- name: Node Exporter as System Service | Set exporter_file_name fact
   set_fact:
     exporter_file_name: "node_exporter-{{ exporter.version }}.linux-amd64.tar.gz"
 
-- name: Node Exporter | Collect version
+- name: Node Exporter as System Service | Collect version
   shell: >-
     /opt/node_exporter/node_exporter --version 2>&1
   register: exporter_current_version_out
 
-- name: Node Exporter | Set exporter_current_version fact
+- name: Node Exporter as System Service | Set exporter_current_version fact
   set_fact:
     exporter_current_version: "{{ exporter_current_version_out.stdout_lines[0] | regex_replace('^.*version ([0-9.]+) .*$', '\\1') }}"
 
-- name: Node Exporter | Print version
+- name: Node Exporter as System Service | Print version
   debug:
     msg:
       - "Installed version: {{ exporter_current_version }}"
       - "Target version: {{ exporter.version }}"
 
-- name: Node Exporter | Upgrade block
+- name: Node Exporter as System Service | Upgrade block
   when: exporter.version != exporter_current_version
   block:
     - name: Node Exporter | Download binaries
@@ -46,25 +82,25 @@
       vars:
         file_name: "{{ exporter_file_name }}"
 
-    - name: Node Exporter | Stop exporter
+    - name: Node Exporter as System Service | Stop exporter
       systemd:
         name: "{{ exporter.service.name }}.service"
         state: stopped
 
-    - name: Node Exporter | Remove /opt/node_exporter directory
+    - name: Node Exporter as System Service | Remove /opt/node_exporter directory
       file:
         path: /opt/node_exporter
         state: absent
 
-    - name: Node Exporter | Create empty /opt/node_exporter directory
+    - name: Node Exporter as System Service | Create empty /opt/node_exporter directory
       file:
         path: /opt/node_exporter
         owner: root
         group: node_exporter
         mode: '0750'
         state: directory
 
-    - name: Node Exporter | Unpack binary
+    - name: Node Exporter as System Service | Unpack binary
       unarchive:
         remote_src: true
         src: "{{ download_directory }}/{{ exporter_file_name }}"
@@ -76,15 +112,15 @@
         group: node_exporter
       check_mode: false
 
-    - name: Node Exporter | Update systemd service configuration
+    - name: Node Exporter as System Service | Update systemd service configuration
       template:
         src: roles/node_exporter/templates/prometheus-node-exporter.service.j2
         dest: "/etc/systemd/system/{{ exporter.service.name }}.service"
         owner: root
         group: root
         mode: '0644'
 
-    - name: Node Exporter | Start exporter
+    - name: Node Exporter as System Service | Start exporter
       systemd:
         daemon_reload: true
         enabled: true

diff --git a/core/src/epicli/data/common/ansible/playbooks/upgrade.yml b/core/src/epicli/data/common/ansible/playbooks/upgrade.yml
@@ -108,6 +108,8 @@
         name: upgrade
         tasks_from: filebeat
       when: groups.logging is defined # do not upgrade if there is legacy Elasticsearch (v6)
+  environment:
+    KUBECONFIG: "{{ kubeconfig.local }}"
 
 - hosts: logging
   become: true
@@ -165,3 +167,5 @@
     - import_role:
         name: upgrade
         tasks_from: node-exporter
+  environment:
+    KUBECONFIG: "{{ kubeconfig.local }}"