**Environment**:
* OS (e.g. from /etc/os-release): Ubuntu 18.04.5 LTS
* Kolla-Ansible version: stable/ussuri (55ffe1f8f)
* Ansible version: 2.9.21
**Bug Report**
What happened:
We encountered this error during a Train -> Ussuri upgrade:
```
TASK [haproxy : Stopping all slave keepalived containers] ***************************************************************************************************
fatal: [os-ctrl-0]: FAILED! => {"msg": "The conditional check 'kolla_internal_vip_address not in addresses' failed. The error was: error while evaluating conditional (kolla_internal_vip_address not in addresses): {{ hostvars[inventory_hostname]['ansible_' + api_interface].get(key, []) | map(attribute='address') | list }}: 'ansible.vars.hostvars.HostVarsVars object' has no attribute 'ansible_os-api-internal'\n\nThe error appears to be in '/usr/local/share/kolla-ansible/ansible/roles/haproxy/tasks/upgrade.yml': line 6, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- name: Stopping all slave keepalived containers\n ^ here\n"}
```
Our api_interface name here is "os-api-internal" which resulted in the variable name "ansible_os-api-internal"
This paragraph in the Ansible 3 documentation states that dashes need to be replaced by underscores when accessing these variables, and it seems to work the same way in 2.9:
https://docs.ansible.com/ansible/3/reference_appendices/faq.html#how-do-i-access-a-variable-name-programmatically
(note that the `replace` parameters seem to be interchanged in the doc)
Based on a quick search this should also affect prechecks and ovs deployment.
Related fix proposed to branch: master
Review: https:/