3504694b4a
When a dynamic_address_gather_filter is set and the inventory data does not generate the dynamic inventory variable container_networks[network_address]['bridge'], the task will fail, as even though dynamic_address_gather_filter is defined the jinja template will still try to evaluate container_networks[network_address]['bridge']. This patch fixes this by setting a default value if container_networks[network_address]['bridge'] is not defined Change-Id: I39082bee2435781d87f653e66fd55a92397c5572
36 lines
1.6 KiB
YAML
36 lines
1.6 KiB
YAML
---
|
|
# Copyright 2017, Rackspace US, Inc.
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License");
|
|
# you may not use this file except in compliance with the License.
|
|
# You may obtain a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS,
|
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
# See the License for the specific language governing permissions and
|
|
# limitations under the License.
|
|
|
|
- name: Gathering facts for {{ network_address }} interface on metal hosts
|
|
setup:
|
|
gather_subset: "!all,network"
|
|
filter: "{{ dynamic_address_gather_filter | default(('ansible_' ~ default_network_interface_name) | replace('-','_')) }}"
|
|
vars:
|
|
default_network_interface_name: "{{ container_networks[network_address]['bridge'] | default }}"
|
|
when:
|
|
- is_metal
|
|
- (dynamic_address_gather_filter is defined) or (default_network_interface_name | length > 0)
|
|
|
|
- name: Set IP to use for {{ network_address }}
|
|
set_fact: "{{ network_address }}={{ hostvars[inventory_hostname] | json_query(query) }}"
|
|
vars:
|
|
query: "{{ is_metal | ternary(metal_query, non_metal_query) }}"
|
|
non_metal_query: "container_networks.{{ network_address }}.address || ansible_host"
|
|
find_bridge: "container_networks.{{ network_address }}.bridge"
|
|
metal_query: "ansible_facts.\"{{ hostvars[inventory_hostname] | json_query(find_bridge) | replace('-','_') }}\".ipv4.address || {{ non_metal_query }}"
|
|
tags:
|
|
- common-address
|
|
- always
|