openstack/openstack-ansible
Code Issues Proposed changes

Merge "Be consistent with haproxy group name"

Browse Source
This commit is contained in:
Zuul 2018-07-12 02:37:10 +00:00 committed by Gerrit Code Review
commit a3bb070fd6
4 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
inventory/group_vars/galera_all.yml
View File

@ -32,7 +32,7 @@ galera_disable_privatedevices: "{{ not is_metal }}"
# the load balancers, and the galera nodes. # the load balancers, and the galera nodes.
galera_monitoring_allowed_source: >- galera_monitoring_allowed_source: >-
{{ {{
groups['galera_all'] | union(groups['haproxy_all']) groups['galera_all'] | union(groups['haproxy'])
| map('extract', hostvars, 'ansible_host') | map('extract', hostvars, 'ansible_host')
| list | list
| join(' ') ~ ' 127.0.0.1' | join(' ') ~ ' 127.0.0.1'

2
inventory/group_vars/keystone_all.yml
View File

@ -29,7 +29,7 @@ keystone_cache_servers: "{{ memcached_servers.split(',') }}"
keystone_service_in_ldap: "{{ service_ldap_backend_enabled }}" keystone_service_in_ldap: "{{ service_ldap_backend_enabled }}"
# Hosts allowed to override remote IP with X-Forwarded-For # Hosts allowed to override remote IP with X-Forwarded-For
keystone_set_real_ip_from: "{{ groups['haproxy_all'] | map('extract', hostvars, 'container_address') | list }}" keystone_set_real_ip_from: "{{ groups['haproxy'] | map('extract', hostvars, 'container_address') | list }}"
# Ensure that the package state matches the global setting # Ensure that the package state matches the global setting
keystone_package_state: "{{ package_state }}" keystone_package_state: "{{ package_state }}"

6
playbooks/healthcheck-hosts.yml
View File

@ -49,7 +49,7 @@
dest: /tmp/osa-master-requirements dest: /tmp/osa-master-requirements
- name: Ensure settings are not wrong with the usual suspects issues before trying to deploy infra - name: Ensure settings are not wrong with the usual suspects issues before trying to deploy infra
hosts: haproxy_all hosts: haproxy
gather_facts: yes gather_facts: yes
tasks: tasks:
- name: Checking that the LB vips are well configured. - name: Checking that the LB vips are well configured.
@ -72,7 +72,7 @@
- "{{ haproxy_keepalived_external_interface }}" - "{{ haproxy_keepalived_external_interface }}"
- "{{ haproxy_keepalived_internal_interface }}" - "{{ haproxy_keepalived_internal_interface }}"
when: when:
- groups['haproxy_all'] | length > 1 - groups['haproxy'] | length > 1
- name: Checking that vip address is well formed - name: Checking that vip address is well formed
assert: assert:
@ -83,4 +83,4 @@
- "{{ haproxy_keepalived_internal_vip_cidr }}" - "{{ haproxy_keepalived_internal_vip_cidr }}"
- "{{ haproxy_keepalived_external_vip_cidr }}" - "{{ haproxy_keepalived_external_vip_cidr }}"
when: when:
- groups['haproxy_all'] | length > 1 - groups['haproxy'] | length > 1

4
playbooks/healthcheck-infrastructure.yml
View File

@ -18,7 +18,7 @@
# Ensuring Load Balancer behavior # Ensuring Load Balancer behavior
- name: Ensuring haproxy runs - name: Ensuring haproxy runs
hosts: haproxy_all hosts: haproxy
gather_facts: no gather_facts: no
tasks: tasks:
- name: Check if host can connect to keepalived ping IP - name: Check if host can connect to keepalived ping IP
@ -28,7 +28,7 @@
- name: Checking if keepalived is running - name: Checking if keepalived is running
command: "pgrep keepalived" command: "pgrep keepalived"
changed_when: false changed_when: false
when: groups['haproxy_all'] | length > 1 when: groups['haproxy'] | length > 1
# Fails if HAProxy is not running # Fails if HAProxy is not running
- name: Recording haproxy stats as a way to ensure haproxy runs - name: Recording haproxy stats as a way to ensure haproxy runs