openstack/kolla-ansible
Code Issues Proposed changes

Merge "loadbalancer: remove support for haproxy_processes"

Browse Source
This commit is contained in:
Zuul 2023-08-18 10:28:23 +00:00 committed by Gerrit Code Review
commit 6af1754071
3 changed files with 9 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
ansible/roles/loadbalancer/defaults/main.yml
View File

@ -52,9 +52,7 @@ keepalived_traffic_mode: "multicast"
# Extended global configuration, optimization options.
haproxy_max_connections: 40000
haproxy_threads: 1
# More than 1 process is deprecated by HAProxy, prefer haproxy_threads
haproxy_processes: 1
haproxy_process_cpu_map: "no"
haproxy_thread_cpu_map: "no"
# Matches the mariadb 10000 max connections limit
haproxy_defaults_max_connections: 10000

8
ansible/roles/loadbalancer/templates/haproxy/haproxy_main.cfg.j2
View File

@ -6,14 +6,8 @@ global
daemon
log {{ syslog_server }}:{{ syslog_udp_port }} {{ syslog_haproxy_facility }}
maxconn {{ haproxy_max_connections }}
nbproc {{ haproxy_processes }}
nbthread {{ haproxy_threads }}
{% if (haproxy_processes | int > 1) and (haproxy_process_cpu_map | bool) %}
{% for cpu_idx in range(0, haproxy_processes) %}
cpu-map {{ cpu_idx + 1 }} {{ cpu_idx }}
{% endfor %}
{% endif %}
{% if (haproxy_threads | int > 1) and (haproxy_process_cpu_map | bool) %}
{% if (haproxy_threads | int > 1) and (haproxy_thread_cpu_map | bool) %}
cpu-map auto:1/all 0-63
{% endif %}
stats socket /var/lib/kolla/haproxy/haproxy.sock group kolla mode 660{% if haproxy_socket_level_admin | bool %} level admin{% endif %}

7
releasenotes/notes/haproxy-drop-processes-f6951f7b995e0694.yaml Normal file
View File

@ -0,0 +1,7 @@
---
upgrade:
- |
Configuring HAProxy nbproc setting via ``haproxy_processes`` and
``haproxy_process_cpu_map`` variables has been dropped since
threads are the recommended way to scale CPU performance since 1.8.
Please use ``haproxy_threads`` and ``haproxy_thread_cpu_map`` instead.