diff options
| author | Célestin Matte <dev@cmatte.me> | 2023-09-18 21:20:46 +0200 |
|---|---|---|
| committer | Célestin Matte <launchpad@cmatte.me> | 2023-10-03 08:10:24 +0000 |
| commit | eec8b05914878f52858f37bbf675994e4e80d74b (patch) | |
| tree | e73db76c73cc0d129b486c2fc7d99e6d9e4bed0b /kolla_ansible | |
| parent | 65079b4dc18283975e4fd38fddf8c553d78f36e1 (diff) | |
| download | kolla-ansible-eec8b05914878f52858f37bbf675994e4e80d74b.tar.gz kolla-ansible-eec8b05914878f52858f37bbf675994e4e80d74b.zip | |
kolla_address: check correct error return value
Unlike other methods such as resolve(), get() does not return an Undefined object, but None.
This removes 4 ansible-lint warnings in various files calling kolla_address.
Closes-Bug: #2038281
Change-Id: I591a50512a954210f951c40a350ed4b9e1fc48ae
Diffstat (limited to 'kolla_ansible')
| -rw-r--r-- | kolla_ansible/kolla_address.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kolla_ansible/kolla_address.py b/kolla_ansible/kolla_address.py index 12dff1ad1..4538e75a2 100644 --- a/kolla_ansible/kolla_address.py +++ b/kolla_ansible/kolla_address.py @@ -48,7 +48,7 @@ def kolla_address(context, network_name, hostname=None): raise FilterError("'inventory_hostname' variable is unavailable") hostvars = context.get('hostvars') - if isinstance(hostvars, Undefined): + if hostvars is None or isinstance(hostvars, Undefined): raise FilterError("'hostvars' variable is unavailable") host = hostvars.get(hostname) |