openstack/openstack-ansible
Code Issues Proposed changes

Remove dynamic-address-fact call which is no longer required

Browse Source 
I don't believe this is necessary anymore since 'management_ip'
was introduced into openstack_user_config.

Whether 'management_ip' is set or not, 'management_address' is
already set in the inventory, and if dynamic-address-fact runs
here it can confusingly override the value of 'management_address'.

Change-Id: I0565eeffd599cc90119420edc18ed37aea80b0db
This commit is contained in:
Andrew Bonney 2023-08-02 09:30:56 +01:00
parent d1e30257ae
commit d0aeb6c5bb
1 changed files with 0 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
playbooks/common-playbooks/nova.yml
View File

@ -57,13 +57,6 @@
- "nova_api_os_compute" - "nova_api_os_compute"
- "nova_console" - "nova_console"
- name: Determine management bridge IP address
include_tasks: ../common-tasks/dynamic-address-fact.yml
vars:
network_address: "management_address"
tags:
- always
- name: Configure container - name: Configure container
include_tasks: "../common-tasks/os-{{ container_tech | default('lxc') }}-container-setup.yml" include_tasks: "../common-tasks/os-{{ container_tech | default('lxc') }}-container-setup.yml"
when: not is_metal when: not is_metal