From 17e38ad8c62ac8f1c8f983e63181f69879a45d85 Mon Sep 17 00:00:00 2001
From: Jiri Mencak <jmencak@redhat.com>
Date: Tue, 3 Oct 2017 19:57:01 +0200
Subject: Separate tuned daemon setup into a role.

Currently, profiles for the tuned daemon are set only for
OpenShift node(s).  This excludes the OpenShift loadbalancer.
As a result, ARP cache limits on loadbalancers are not raised.
This causes problems with HA setups where loadbalancers serve
1k+ OpenShift nodes.

This commit ensures the openshift-control-plane role is applied
to loadbalancers, masters and OpenShift infra nodes.  Regular
OpenShift worker nodes get the openshift-node profile.

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1498213
---
 roles/openshift_node/tasks/config.yml |  4 ----
 roles/openshift_node/tasks/tuned.yml  | 41 -----------------------------------
 2 files changed, 45 deletions(-)
 delete mode 100644 roles/openshift_node/tasks/tuned.yml

(limited to 'roles/openshift_node/tasks')

diff --git a/roles/openshift_node/tasks/config.yml b/roles/openshift_node/tasks/config.yml
index e3898b520..e5fcaf9af 100644
--- a/roles/openshift_node/tasks/config.yml
+++ b/roles/openshift_node/tasks/config.yml
@@ -111,9 +111,5 @@
     msg: Node failed to start please inspect the logs and try again
   when: node_start_result | failed
 
-- name: Setup tuned
-  include: tuned.yml
-  static: yes
-
 - set_fact:
     node_service_status_changed: "{{ node_start_result | changed }}"
diff --git a/roles/openshift_node/tasks/tuned.yml b/roles/openshift_node/tasks/tuned.yml
deleted file mode 100644
index 425bf6a26..000000000
--- a/roles/openshift_node/tasks/tuned.yml
+++ /dev/null
@@ -1,41 +0,0 @@
----
-- name: Check for tuned package
-  command: rpm -q tuned
-  args:
-    warn: no
-  register: tuned_installed
-  changed_when: false
-  failed_when: false
-
-- name: Tuned service setup
-  block:
-  - name: Set tuned OpenShift variables
-    set_fact:
-      openshift_tuned_guest_profile: "{{ 'atomic-guest' if openshift.common.is_atomic else 'virtual-guest' }}"
-      tuned_etc_directory: '/etc/tuned'
-      tuned_templates_source: '../templates/tuned'
-
-  - name: Ensure directory structure exists
-    file:
-      state: directory
-      dest: '{{ tuned_etc_directory }}/{{ item.path }}'
-    with_filetree: '{{ tuned_templates_source }}'
-    when: item.state == 'directory'
-
-  - name: Ensure files are populated from templates
-    template:
-      src: '{{ item.src }}'
-      dest: '{{ tuned_etc_directory }}/{{ item.path }}'
-    with_filetree: '{{ tuned_templates_source }}'
-    when: item.state == 'file'
-
-  - name: Make tuned use the recommended tuned profile on restart
-    file: path=/etc/tuned/active_profile state=absent
-
-  - name: Restart tuned service
-    systemd:
-      state: restarted
-      daemon_reload: yes
-      name: tuned
-
-  when: tuned_installed.rc == 0 | bool
-- 
cgit v1.2.3