Skip to content

Commit

Permalink
Deprecate kolla_internal_address variable
Browse files Browse the repository at this point in the history
The "kolla_internal_address" variable is not documented or defined
anywhere.  When "kolla_internal_vip_address" is undefined, the error
message is about "kolla_internal_address", which will confuse operators.

This change deprecates "kolla_internal_address", and adds a default
value for "kolla_internal_vip_address" when "kolla_internal_address" is
undefined.

Change-Id: I09694b38420ea67896bb8cf4ffd7ce6f131af10e
Closes-Bug: #1864206
  • Loading branch information
chenxingc authored and markgoddard committed Aug 17, 2020
1 parent d1e5de2 commit b4603d9
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
2 changes: 1 addition & 1 deletion ansible/group_vars/all.yml
Expand Up @@ -32,7 +32,7 @@ kolla_base_distro: "centos"
# Valid options are [ binary, source ]
kolla_install_type: "binary"

kolla_internal_vip_address: "{{ kolla_internal_address }}"
kolla_internal_vip_address: "{{ kolla_internal_address | default('') }}"
kolla_internal_fqdn: "{{ kolla_internal_vip_address }}"
kolla_external_vip_address: "{{ kolla_internal_vip_address }}"
kolla_same_external_internal_vip: "{{ kolla_external_vip_address == kolla_internal_vip_address }}"
Expand Down
@@ -0,0 +1,6 @@
---
deprecations:
- |
The variable ``kolla_internal_address`` is deprecated. This variable is
used only as the default value for ``kolla_internal_vip_address``, and is
not documented.

0 comments on commit b4603d9

Please sign in to comment.