Merge pull request #5645 from jmencak/tuned-role-lb

Automatic merge from submit-queue.

Separate tuned daemon setup into a role.

Also adding support for the loadbalancer.  It is important to raise ARP cache limits on HA setups with loadbalancers serving 1k+ nodes.

playbooks/common/openshift-loadbalancer/config.yml

@@ -27,6 +27,7 @@
   roles:
   - role: os_firewall
   - role: openshift_loadbalancer
+  - role: tuned
 
 - name: Load Balancer Install Checkpoint End
   hosts: localhost

playbooks/common/openshift-master/config.yml

@@ -198,6 +198,7 @@
     openshift_master_default_registry_value: "{{ hostvars[groups.oo_first_master.0].l_default_registry_value }}"
     openshift_master_default_registry_value_api: "{{ hostvars[groups.oo_first_master.0].l_default_registry_value_api }}"
     openshift_master_default_registry_value_controllers: "{{ hostvars[groups.oo_first_master.0].l_default_registry_value_controllers }}"
+  - role: tuned
   - role: nuage_ca
     when: openshift_use_nuage | default(false) | bool
   - role: nuage_common

playbooks/common/openshift-node/configure_nodes.yml

@@ -13,4 +13,5 @@
   roles:
   - role: os_firewall
   - role: openshift_node
+  - role: tuned
   - role: nickhammond.logrotate

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 }}"

roles/tuned/defaults/main.yml

@@ -0,0 +1,3 @@
+---
+tuned_etc_directory: '/etc/tuned'
+tuned_templates_source: '../templates'

roles/tuned/meta/main.yml

@@ -0,0 +1,13 @@
+---
+galaxy_info:
+  author: Jiri Mencak
+  description: Restart the tuned daemon if present and make it use the recommended profile
+  company: Red Hat, Inc.
+  license: Apache License, Version 2.0
+  min_ansible_version: 2.3
+  platforms:
+  - name: EL
+    versions:
+    - 7
+  categories:
+  - cloud

roles/openshift_node/tasks/tuned.yml

@@ -12,8 +12,6 @@
   - 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:

roles/openshift_node/templates/tuned/recommend.conf

@@ -1,8 +1,11 @@
-[openshift-node]
-/etc/origin/node/node-config.yaml=.*region=primary
-
 [openshift-control-plane,master]
 /etc/origin/master/master-config.yaml=.*
 
 [openshift-control-plane,node]
 /etc/origin/node/node-config.yaml=.*region=infra
+
+[openshift-control-plane,lb]
+/etc/haproxy/haproxy.cfg=.*
+
+[openshift-node]
+/etc/origin/node/node-config.yaml=.*